Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

192 front task 공유 일정 페이지 일정 후보 모달 및 후보 등록 #208

Conversation

kimyoungyin
Copy link
Contributor

@kimyoungyin kimyoungyin commented Apr 26, 2024

설명

테스트

스크린샷 2024-05-08 오전 11 27 04

refactor: move shared styled components to ScheduleModal.Shared.styles.js and use them in ScheduleModal and ScheduleProposalModal
…e it's triggering condtion with ui-slice
@kimyoungyin kimyoungyin requested a review from sikkzz April 26, 2024 07:12
@kimyoungyin kimyoungyin self-assigned this Apr 26, 2024
@kimyoungyin kimyoungyin added the enhancement New feature or request label Apr 26, 2024
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

스크린샷 2024-05-06 오후 10 21 28

추천받기 버튼 width가 어색한데 아래 등록하기 버튼이랑 맞추는건 어떠세요?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

저는 이 폼 자체를 제출하는 거랑 버튼을 같은 디자인으로 가면 안 될 것 같아서 저렇게 했었는데 이런 측면에서 생각해봐도 변경하는게 나을까요? 일단 지수님한테도 여쭤볼게요

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

지수님하고 얘기 나눠보니까 너비만 수정하기로 했습니다. disabled 상태에서도 높이 차이가 있으니 색 구분이 되지 않을까 싶어 그대로 가려고 합니다.

@sikkzz
Copy link
Contributor

sikkzz commented May 6, 2024

개인일정쪽 schedulModal이랑 EditedProposalForm, ScheduleProposalModal 각각 기능이 비슷한 함수들이 있지 않나요? 이걸 좀 모듈화 시켜서 사용하는게 좋아 보이는데 어떠신가요

@sikkzz
Copy link
Contributor

sikkzz commented May 6, 2024

다른 계정에서 공유 일정 작동 안하는거도 확인 다시 부탁드려요

@kimyoungyin
Copy link
Contributor Author

다른 계정에서 공유 일정 작동 안하는거도 확인 다시 부탁드려요

해결: 관련 노션 링크

@kimyoungyin kimyoungyin closed this May 8, 2024
@kimyoungyin kimyoungyin reopened this May 8, 2024
@kimyoungyin
Copy link
Contributor Author

개인일정쪽 schedulModal이랑 EditedProposalForm, ScheduleProposalModal 각각 기능이 비슷한 함수들이 있지 않나요? 이걸 좀 모듈화 시켜서 사용하는게 좋아 보이는데 어떠신가요

저도 이 생각으로 시도를 조금 해봤었는데, 각자 다루는 form value들이 약간씩 달라서 조금 힘들더군요.. 다시 보고 분리하면 좋겠다 싶은 부분은 진행해볼게요

@kimyoungyin kimyoungyin requested a review from sikkzz May 17, 2024 07:00
@kimyoungyin kimyoungyin merged commit 3e378dd into 179-front-task-공유-일정-페이지-구현 Jun 17, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants