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: 가이드 페이지 구현 #134

Merged
merged 8 commits into from
Jun 7, 2024
Merged

feat: 가이드 페이지 구현 #134

merged 8 commits into from
Jun 7, 2024

Conversation

hae-on
Copy link
Contributor

@hae-on hae-on commented May 22, 2024

Issue

✨ 구현한 기능

스크린샷 2024-05-22 오후 4 45 49

가이드 페이지를 구현하였습니다.
왕펑이 만든 useMenu custom hook 재사용했슴니당.

최상단에 있는 bar 이름은 material-ui에 있는 stepper의 이름을 참고했습니다!

📢 논의하고 싶은 내용

1. 이거 연결을 어디다가 할까요?

보통 홈페이지에 첫 진입하면 뜨지 않아요...?????
일단은 가이드 배너에 연결시켜놨습니다!
스크린샷 2024-05-22 오후 4 42 18
(요기에,,,!)

2. 높이

스크린샷 2024-05-22 오후 4 44 13

피그마 보면, 각 step마다 글자와 이미지 사이의 높이가 다르더라구요!
이거를 어떻게 하죵...?
(지금은 세 화면 다 높이 똑같음)
젤 쉬운 방법은 props로 높이를 받는건데, 뭔가 부자연스러운 느낌이라....
다른 방법 아시는 분 있나용?!?!

🎸 기타

x

⏰ 일정

  • 추정 시간 : 20분
  • 걸린 시간 : 1시간

Copy link

🔗 배포된 storybook 주소: https://65f015a4aed45406385006ee-suswxoaeut.chromatic.com/

Copy link

🔗 배포된 storybook 주소: https://65f015a4aed45406385006ee-opanqpyawv.chromatic.com/

Copy link
Member

@xodms0309 xodms0309 left a comment

Choose a reason for hiding this comment

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

슬랙 알람을 못봐서 이 PR을 지금봤네요..ㅠㅠ 고생했어요 해옹! 전 가이드 배너에 연결하는거 좋은 것 같아요 그리고 높이는 다 통일하면 좋지 않을까요...? 제기준 높이가 그렇게 중요해보이지 않음..!

return (
<div className={contentWrapper}>
<div className={titleWrapper}>
<Text weight="bold" color="yellow" style={{ fontSize: '12px' }}>
Copy link
Member

Choose a reason for hiding this comment

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

이거 caption3 사이즈 아닌가용?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

🦅👁️

Comment on lines 21 to 23
<Text size="display1" weight="semiBold" color="sub" style={{ whiteSpace: 'pre-wrap', textAlign: 'center' }}>
{description}
</Text>
Copy link
Member

Choose a reason for hiding this comment

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

style을 className으로 추가해주면 더 좋을것 같아용

import { container, stepper, wrapper } from './stepper.css';

interface StepperProps {
selectedStepper: string;
Copy link
Member

Choose a reason for hiding this comment

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

이 selectedStepper을 string으로 관리하는 이유가 있나요? 다른 곳 보면 parseInt를 하거나 밑에서 String으로 변환시키는 것 같은데 number로 관리하면 안그래도 되지 않나 싶어서요!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

이게 useTabMenu 훅이 string을 받고 있어서 저렇게 썼던 거 같아요!
number로 받게 하고, 사용처에서

const selectedStepper = parseInt(selectedTabMenu);

요렇게 묶는 방식으로 변경했슴다!

Copy link
Contributor

@Leejin-Yang Leejin-Yang left a comment

Choose a reason for hiding this comment

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

진짜 앱 같네요.!
첫진입때 뜨는게 맞아보이는데, 로컬스토리지에 처음 방문하면 값을 저장해둬서 구분하는거 어떤가요?
그리고 배너에 연결하는거 굿굿
높이는 동일하면 좋아보입니다..🫠

<button
className={isSelected ? stepper['active'] : stepper['default']}
type="button"
value={String(index)}
Copy link
Contributor

Choose a reason for hiding this comment

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

value는 뽑아서 쓸 때 아마 string으로 변환될거에요

Copy link

github-actions bot commented Jun 5, 2024

🔗 배포된 storybook 주소: https://65f015a4aed45406385006ee-qtfmzhggsw.chromatic.com/

@hae-on
Copy link
Contributor Author

hae-on commented Jun 5, 2024

@Leejin-Yang
왕펑씨 첫 진입시 온보딩 페이지로 진입하도록 구현했는데 확인해주세요!
원래 로컬스토리지 키도 불리언일 때, is- 접두사 붙는거였나요?? 기억 안남,,,

Copy link
Member

@xodms0309 xodms0309 left a comment

Choose a reason for hiding this comment

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

localStorage key값은 딱히 컨벤션 안정한듯?! 고생했어요!

@hae-on hae-on merged commit 914bbb2 into feat/v2 Jun 7, 2024
2 of 3 checks passed
@hae-on hae-on deleted the feat/issue-129 branch June 7, 2024 00:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

가이드 페이지 연결
3 participants