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

[feat/CK-210] 로드맵 디테일 페이지의 UI를 개편한다. #173

Merged
merged 9 commits into from
Oct 1, 2023

Conversation

jw-r
Copy link
Collaborator

@jw-r jw-r commented Sep 30, 2023

📌 작업 이슈 번호

CK-210

✨ 작업 내용

  • RoadmaapDetail 컴포넌트 UI 개편
  • 모바일 환경 대응

💬 리뷰어에게 남길 멘트

모바일 환경에서는 사진과 로드맵의 메타 정보(난이도, 카테고리 등)가 보이지 않는 것을 확인할 수 있습니다.
이유는 다음과 같습니다.

  1. 주요 타겟 환경은 데스크톱이다
  2. 모바일에서도 어색하지 않게 동작해야한다
  3. 모바일에서도 주요 정보를 같은 편리함으로 보게 하고 싶다

왜인지 개행이 적용되지 않아서 조금 지저분해 보이는군요😭

PC 환경

스크린샷 2023-10-01 오전 12 35 26

모바일 환경

image

더 보기 버튼 활성화 상태 및 No Image

2023-10-01.12.39.07.mov

잘 부탁드립니다~!

@jw-r jw-r added feature 🔅 Improvements or additions to documentation FE 👨‍👨‍👧 FrontEnd labels Sep 30, 2023
@jw-r jw-r self-assigned this Sep 30, 2023
Copy link
Collaborator

@NaveOWO NaveOWO left a comment

Choose a reason for hiding this comment

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

우디 넘 수고했어요.... 반응형에 여러가지 상황을 고려해서 잘 만들어준 것 같아요..!!!!!👍👍👍 너무 잘해서 질투나니깐 어프루브 할께요! 선택적으로 수정사항 반영해주세욨

roadmapInfo: RoadmapDetailType;
};

const Introduction = ({ roadmapInfo }: IntroductionType) => {
Copy link
Collaborator

Choose a reason for hiding this comment

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

타입 네이밍 컨벤션은 InfoductionProps 에요!!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

반영완!

}, []);

const toggleExpand = () => {
setIsExpanded(!isExpanded);
Copy link
Collaborator

Choose a reason for hiding this comment

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

사소한 부분이지만 setIsExpanded((prev) => !prev) 가 더 정확한 표현인 것 같아요~!;

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

오메 매의 눈 굿굿!
반영완!

Copy link
Collaborator

@sh981013s sh981013s left a comment

Choose a reason for hiding this comment

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

너무 좋습니다! 야무지게 구현하셨네요 ㅎㅎ UI 적인 부분이라 진행해도 좋을 듯 싶어요! 😀

@jw-r jw-r merged commit b60c3e3 into develop-client Oct 1, 2023
1 check passed
@jw-r jw-r deleted the feature/CK-210 branch October 1, 2023 03:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FE 👨‍👨‍👧 FrontEnd feature 🔅 Improvements or additions to documentation
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants