Feature/jaino/#126 #143
Feature/jaino/#126 #143
Conversation
…re/jaino/#95 # Conflicts: # app/src/main/java/com/wap/wapp/MainActivity.kt # feature/survey/src/main/java/com/wap/wapp/feature/survey/answer/SurveyAnswerForm.kt
…re/jaino/#95 # Conflicts: # app/src/main/java/com/wap/wapp/MainActivity.kt # feature/management/src/main/java/com/wap/wapp/feature/management/ManagementScreen.kt
…를 불러오는 게 아닌 인자로 넘긴 상태로 변경되도록 수정
… 현재 및 다음 상태 인자로 넘겨주기
jeongjaino
added
🚀진호🚀
| modifier = Modifier.padding(top = 16.dp, start = 20.dp, end = 20.dp), | ||
| modifier = Modifier.padding(vertical = 16.dp, horizontal = 20.dp), | ||
| ) | ||
|
|
||
| EventRegistrationContent( | ||
| eventRegistrationState = currentEditState, | ||
| modifier = Modifier.padding(top = 50.dp, start = 20.dp, end = 20.dp), | ||
| modifier = Modifier.padding(horizontal = 20.dp), |
There was a problem hiding this comment.
진호상 여기 지금 설문 등록 / 수정이랑 일정 등록 / 수정 마진을 맞춰 놨는데,
제가 일부러 약간 띄워놓긴 했거든요,
근데 지금 다시 보니께 진호님이 설정해놓은 것 처럼 좁혀놓은 것도 좋은 것 같아서,
혹시 제가 해놓은 대로 넓히시는 게 좋으시면 이 부분을 넓혀 주시고,
진호님이 하신대로 좁힌게 좋으시면 수정 부분 Margin에 맞게 등록 부분도 Vertical 마진 맞춰주시면 좋을 것 같아요 !!!!!!!
There was a problem hiding this comment.
해당 부분이
제가 이전 카톡에서 언급한 바깥쪽 패딩을 안쪽 패딩으로 옮기는 작업 중에 발생한 것 같아요
근데 중요한게 병합하면서, 수정한 패딩들이 다 사라졌어요 하하,, 그래서 아마 패딩이 안맞는 부분이 있는걸꺼에요
저도 다시 돌려보면서 안맞는 부분 수정할게요
There was a problem hiding this comment.
진호상 여기 지금 설문 등록 / 수정이랑 일정 등록 / 수정 마진을 맞춰 놨는데,
제가 일부러 약간 띄워놓긴 했거든요,
근데 지금 다시 보니께 진호님이 설정해놓은 것 처럼 좁혀놓은 것도 좋은 것 같아서,
혹시 제가 해놓은 대로 넓히시는 게 좋으시면 이 부분을 넓혀 주시고,
진호님이 하신대로 좁힌게 좋으시면 수정 부분 Margin에 맞게 등록 부분도 Vertical 마진 맞춰주시면 좋을 것 같아요 !!!!!!!
다시 50dp 맞춰놓겠습니다 🙏
There was a problem hiding this comment.
40dp로 바꿨습니다.
SurveyForm, Event 모두 적용시켜놨습니다 하하
tgyuuAn
added
✏️수정 요청✏️
|
진호상 고생했어요 흐흐흐흐 퇴근 후 바로 작업이라니 ... 🥲🥲 |
tgyuuAn
added
🌟머지 해주세요🌟
1. 📄 관련된 이슈 및 소개
#126 행사 등록페이지 이전 버튼 구현
2. 🔥 변경된 점
3. ✅ 꼭 확인해줬으면 하는 부분
행사뿐만아니라 설문에도 로직이 동일하게 추가되어있어요
Content에서 이동하기를 원하는 State를 위로 올려서, ViewModel에서 전환하도록 구현했어요
보시다가 모르겠는 부분 말씀주세요
4. 📸 스크린샷(선택)
5. 💡알게된 혹은 궁금한 사항들