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/#115] 메인 페이지 퍼블리싱 #118

Merged
merged 12 commits into from
Jan 18, 2025
Merged

Conversation

chaeneey
Copy link
Collaborator

@chaeneey chaeneey commented Jan 17, 2025

📌 관련 이슈번호


체크리스트

  • 🎋 base 브랜치를 develop 브랜치로 설정했나요?
  • 🖌️ PR 제목은 형식에 맞게 잘 작성했나요?
  • 🏗️ 빌드는 성공했나요? (yarn build)
  • 🧹 불필요한 코드는 제거했나요? e.g. console.log
  • 🙇‍♂️ 리뷰어를 지정했나요?
  • 🏷️ 라벨은 등록했나요?

✅ Key Changes

이번 PR에서 작업한 내용을 간략히 설명해주세요

  1. CategoryCard 만들기
    category 정보를 받아서, CATEGORY_COMPONENT 객체에서 text, hashtag, icon 내용을 가져오는 형태로 구현했습니다.

  2. HomePage 만들기
    피그마 보고 화면 뚝딱뚝딱 그렸습니다.
    구현 결과는 용량이슈로 동영상이 안올라가서 카톡으로 파일 보내겠습니다 !-!


📢 To Reviewers


📸 스크린샷 or 실행영상

Copy link
Collaborator

@zzz-myam zzz-myam left a comment

Choose a reason for hiding this comment

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

뚝딱뚝딱 금방 잘 그려내는 채연님의 멋진 실력에 큰 박수!!!!!!!!👏👏👏

category: CategoryType;
}

const CategoryCard = ({ category }: CategoryCardProps) => {
Copy link
Collaborator

Choose a reason for hiding this comment

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

다시 한 번 꼭꼭 잘 기억하고, 찾아보지 않아도 사용할 수 있도록 이해하는 계기가 되었습니다! 늘 깔꼼 정리 잘하시는 채연님 최고최고

Copy link
Collaborator

@youtheyeon youtheyeon left a comment

Choose a reason for hiding this comment

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

진짜 깔끔하당.. 요소가 많은데 엄청 가독성이 좋네요 수고하셨습니다!!! 💛


export const categoryIconWrapper = style({
position: 'absolute',
top: '1rem',
Copy link
Collaborator

Choose a reason for hiding this comment

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

보이는 것에서 차이는 없겠지만 개인적으로 top보다 bottom을 기준으로 하는 게 더 자연스러워 보입니다...!

{
width: '100%',
gap: '4.2rem',
padding: '3.3rem 0 7.3rem',
Copy link
Collaborator

Choose a reason for hiding this comment

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

사소한 부분이기는 한데.. 피그마 상으로 7.3rem 부분이 7.4rem으로 표기되어 있는 듯 합니다!

Comment on lines 68 to 72
{isLogin ? (
<h1 className={mainTextStyle}>{user.userName} 님,</h1>
) : (
<h1 className={mainTextStyle}>안녕하세요!</h1>
)}
Copy link
Collaborator

Choose a reason for hiding this comment

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

<h1 className={mainTextStyle}>
  {isLogin ? `${user.userName} 님,` : '안녕하세요!'}
</h1>

이렇게 하면 좀 가독성이 떨어질까요..? 흠..🥺

Comment on lines 91 to 93
<>
<CategoryCard category={category} />
</>
Copy link
Collaborator

Choose a reason for hiding this comment

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

혹시 빈 태그 (<>) 없애면 오류날까요? 안 난다면 없애는 게 좋을 것 같습니다!

gap: '0.4rem',
width: '100%',
height: '8.7rem',
padding: '1.4rem 1rem',
Copy link
Collaborator

Choose a reason for hiding this comment

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

padding: '1rem' 으로 수정 부탁드립니다!


export const categoryTextStyle = style([
vars.fonts.head02_b_20,
flexGenerator('row', 'flex-start'),
Copy link
Collaborator

Choose a reason for hiding this comment

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

여기는 flexGenerator() 빼도 괜찮지 않나요?

flexGenerator('row', 'flex-start'),
{
color: vars.colors.black,
width: '100%',
Copy link
Collaborator

Choose a reason for hiding this comment

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

width: '100%도 빼도 괜찮을것 같기도 하고 테스트해보고 괜찮으면 빼주세요!

import { DesignCard } from '@components';
import { CATEGORY } from '@constants';
import CategoryCard from '@pages/home/components/CategoryCard/CategoryCard';
import { MainKeyVisual } from 'src/assets/images';
Copy link
Collaborator

Choose a reason for hiding this comment

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

이거 추후에 이미지 파일들은 경로를 통일해줘야 할 것 같아요!


export const mainContentContainer = style([
flexGenerator('column', 'flex-start', 'flex-start'),
{
Copy link
Collaborator

Choose a reason for hiding this comment

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

gap: '0.4rem' 추가해주세요!

Comment on lines 28 to 33
export const mainTextContainer = style([
flexGenerator('column', 'flex-start', 'flex-start'),
{
gap: '0.4rem',
},
]);
Copy link
Collaborator

Choose a reason for hiding this comment

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

안쓰는 스타일은 지워주세요!

</div>
</section>

<section className={subSectionContainer}>
Copy link
Collaborator

Choose a reason for hiding this comment

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

여기는 그럼 main태그로 바꿔주면 더 좋을거 같아요~

Copy link
Collaborator

@thisishwarang thisishwarang left a comment

Choose a reason for hiding this comment

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

메인페이지 너무 예쁘네요 고생 많으셨습니다!
몇가지 수정 부탁드리고, 인기 스토어 랭킹 부분 추가 부탁드립니다~

@thisishwarang thisishwarang merged commit 1e4e64f into develop Jan 18, 2025
3 checks passed
@thisishwarang thisishwarang deleted the feat/#115/main-page branch January 18, 2025 20:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feat] 메인 페이지 퍼블리싱
4 participants