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

refactor: 마크업 수정 및 기존 코드를 배열로 리팩토링했습니다. #270

Merged
merged 5 commits into from
Aug 28, 2024

Conversation

summermong
Copy link
Collaborator

@summermong summermong commented Aug 28, 2024

🤔 어떤 문제가 발생했나요?

  • 지영님이 발견하신 것처럼 앱 버전 하단 backgroundColor가 비어있었습니다.
  • setting 페이지에는 listItem 컴포넌트를 사용하는데 동일한 컴포넌트를 계속 사용하는 것보다 배열로 관리하는 게 더 나은 것 같아 수정했습니다.

🎉 어떻게 해결했나요?

  • flex 1로 랩핑했습니다.
  • 배열로 만들고 그 다음 divider 여부를 boolean으로 체크해 map으로 순회했습니다.

📷 이미지 첨부 (Option)

2024-08-28.5.53.44.mov

⚠️ 유의할 점! (Option)

  • 변경된 파일 중 탈퇴 로직 구현을 위해 사용하는 부분이 있습니다. 해당 부분은 수정중으로 완료되면 console도 지울 예정입니다.
    setting 페이지만 봐주세용~

@summermong summermong added the refactor # Refactor code label Aug 28, 2024
@king2dwellsang king2dwellsang added the Review Plz🙏 # Review is not yet complete label Aug 28, 2024
Copy link
Member

@Jungjjeong Jungjjeong left a comment

Choose a reason for hiding this comment

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

console.log는 나중에 지울 수 있게 TODO로 달아주시면 좋을 것 같아요

const containerStyles = css({
display: 'flex',
flexDirection: 'column',
minHeight: '100vh',
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
minHeight: '100vh',
minHeight: '100dvh',

사파리 대응한 스타일 수정입니다

@king2dwellsang king2dwellsang added Approved 🆗 # Review is completed and removed Review Plz🙏 # Review is not yet complete labels Aug 28, 2024
@summermong summermong merged commit cf20462 into main Aug 28, 2024
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Approved 🆗 # Review is completed refactor # Refactor code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants