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

[Fix/#168] 캘린더 이전 날짜 선택 막기, 경고문구 디자인 수정. #181

Merged
merged 7 commits into from
Oct 2, 2023

Conversation

eastlaw80
Copy link
Contributor

@eastlaw80 eastlaw80 commented Sep 20, 2023

#168

todo

  • 캘린더 이전 날짜 선택 막기
  • 선택 안되는 이전날짜 글자색 변경. 오늘이 속한 이전달의 날짜는 아예 배경색과 동일
  • 캘린더 7일 선택 경고문구 흔들림 애니메이션 추가
  • 캘린더 7일 이상 선택시 자동 선택 해제

📌 내가 알게 된 부분

  • 조건부로 스타일드 컴포넌트에 애니메이션을 넣을 때는 백틱 문법을 사용한다.
    ${({ $dateLength }) => $dateLength > 7 && `animation: vibration 0.1s 5;`};
  • 방금 알았는데 마크다운에서 백틱을 코드입력에 넣고 싶으면 코드입력을 백틱 하나로 하지말고 3개로 하면된다.

📌 공유하고 싶은 부분

  • 없습니다.

📌 질문할 부분

  • 없습니다.

📌스크린샷

2023-09-20.3.35.48.mov

@eastlaw80 eastlaw80 added fix fix design about style labels Sep 20, 2023
@eastlaw80 eastlaw80 self-assigned this Sep 20, 2023
@vercel
Copy link

vercel bot commented Sep 20, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
asap ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 2, 2023 3:33pm

@eastlaw80 eastlaw80 added the 동헌 동헌이의 개발 라벨 label Sep 20, 2023
Copy link
Member

@simeunseo simeunseo left a comment

Choose a reason for hiding this comment

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

굳굳 ,,, 꽤나 복잡한 처리인데 완벽하네요 멋집니다!

Comment on lines +108 to +114
if (newDate.length > 7) {
setTimeout(() => {
setMeetingInfo((prev: MeetingInfo) => {
return { ...prev, availableDates: [] };
});
}, 1000);
}
Copy link
Member

Choose a reason for hiding this comment

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

오 7일 이상 선택하면 1초 뒤에 자동으로 선택이 풀리는거 맞나요?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

네 맞습니다. 7일 이상 선택하면, 1초 뒤에 자동으로 선택이 풀리고 그동안 경고문구가 빨간 색으로 변한뒤 잠시 글이 좌우로 흔들리는 애니메이션이 나옵니다.

Comment on lines 189 to +199
/* .bg-dark .rmdp-button:not(.rmdp-action-button):hover {
background-color: var(--rmdp-main1);
} */

.rmdp-day.rmdp-disabled {
color: var(--rmdp-grey6) !important;
}

.rmdp-day.rmdp-disabled.rmdp-deactive {
color: var(--rmdp-grey10) !important;
}
Copy link
Member

Choose a reason for hiding this comment

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

혹시 setDates.css를 따로 뺀 건 라이브러리 자체 css를 삭제하는 코드를 따로 분리하기 위해서인가요?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

아 네네 맞습니다. 라이브러리 자체 css 코드를 수정하는 것들이 많아서 너무 길어지길래 그러한 목적을 가진 css 코드만 따로 뺏습니다.

transform: translateX(0);
}
}
${({ $dateLength }) => $dateLength > 7 && `animation: vibration 0.1s 5;`};
Copy link
Member

Choose a reason for hiding this comment

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

animation : ${({ $dateLength }) => $dateLength > 7 ? vibration 0.1s 5; : none};
이전까지 별고민 없이 이런식으로 사용했던 것 같은데 동헌이가 쓴 것 같이 사용하는게 훨씬 알맞네요!!! 뭔가 백틱으로 애니메이션 속성 전체를 넣을 생각을 못했음!!

@sohyundoh
Copy link
Member

우와 이거 빨리 반영됐으면 좋겠다! FGI할때 이전 날짜 왜 선택되냐고 많이 혼남

@ljh0608
Copy link
Member

ljh0608 commented Oct 2, 2023

LGTM

@eastlaw80 eastlaw80 merged commit 2e28ccd into develop Oct 2, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
design about style fix fix size/M 동헌 동헌이의 개발 라벨
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants