-
Notifications
You must be signed in to change notification settings - Fork 6
[FE,BE] Setting calendar startday #110
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
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- 7 경우의 수이기 때문에 따로 빼서 관리
- 시작요일을 변화시키면 달력에도 적용되도록 하였다.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
📕 Issue Number
Close #105
📙 작업 내역
account book의 start day를 읽어서, calendar에 알맞은 header와 body를 그리도록함
util에 template을 만들어 calendar body를 그릴때 사용하였다.
동적으로 할당할 수 있지만 경우의 수가 7가지 뿐 이고, 각각에 디자인 효과를 주기에는 오히려 나눠져서 관리되는 것도 괜찮을 것 같았다.
calendar Header또한 7가지 경우의 수이기 때문에, 별도로 컴포넌트를 분리 한 후 start day에 따라 다르게 그려주도록 하였다.
마찬가지로 각 요일에 대한 디자인등을 할 때 더 편리할 것 같아서 동적으로 할당하지 않았다.
추후 리팩토링 가능
setting page의 calendar탭을 구현하였다.
시작요일을 설정하는 컨텐츠를 위치시켰다.
이후 GMT 설정란이 생길 수 있어서 우선 꽉차게 구현하지 않았다.
라디오 버튼에 따라 calendar header가 어떻게 변할지 시각적으로 확인할 수 있도록 하였다.
save버튼 클릭시 바로 적용하는게 아니라 modal형태로 한번 되묻도록 하였다.
setting탭 save modal 분리
변경 사항을 저장할 것인지 되묻는 모달이 여러군데서 사용될 수 있을 것 같아서 분리하였다.
modal을 끄는 setSaveModal, 변경데이터인 updateData, save ok버튼 클릭시 행해질 함수인 saveAction을 prop으로 받는다.
멘토님 멘셔닝
@boostcamp-2020/accountbook_mentor