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

페이지 UI: 검색 페이지 #104

Merged
merged 11 commits into from
Jan 21, 2024
Merged

페이지 UI: 검색 페이지 #104

merged 11 commits into from
Jan 21, 2024

Conversation

seoye0ng
Copy link
Collaborator

해당 사항 (중복 선택)

  • FEAT : 새로운 기능 추가 및 개선
  • TEST : 테스트 추가 및 리팩토링
  • FIX : 버그 수정
  • REFACTOR : 결과의 변경 없이 코드의 구조를 재조정
  • STYLE : 코드 스타일에 관련된 변경 사항
  • DOCS : 코드가 아닌 문서를 수정한 경우
  • REMOVE : 파일을 삭제하는 작업만 수행
  • RENAME : 파일 또는 폴더명을 수정하거나 위치(경로)를 변경
  • CHORE : 패키지 매니저 설정, 코드 수정 없이 설정 변경(eslint) 등 기타 사항

설명

Key Changes

image image

How it Works

To Reviewers

검색페이지 제작 도중에 dropdown 컴포넌트를 수정하였습니다. (클릭된 라벨이 선택되도록)
검색페이지에 사용되는 dropdown option 중에 위반제품순이 있는데 글자수가 길어서 레이아웃이 깨지고 있습니다. 이부분을 글자 크기대로 늘릴지 아니면 글자가 잘리도록 할지 고민됩니다! 🤔
image
이후에 filter drawer ui 제작 예정입니다.

@seoye0ng seoye0ng added ✨ Feature 기능 개발 🎨 Html&css 마크업 & 스타일링 labels Jan 21, 2024
@seoye0ng seoye0ng self-assigned this Jan 21, 2024
@seoye0ng seoye0ng requested a review from bottlewook as a code owner January 21, 2024 06:12
Copy link

vercel bot commented Jan 21, 2024

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

Name Status Preview Comments Updated (UTC)
f1-wash-pedia-fe ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 21, 2024 7:39am

Copy link
Collaborator

@bottlewook bottlewook left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!! 제가 Dropdown 컴포넌트를 엉망으로 만들어놨네요 ㅠㅠ
이따 다시 리팩토링 하겠습니다!


import classNames from 'classnames/bind';

import ProductArticle from '@/components/shared/product-article/ProductArticle';
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
import ProductArticle from '@/components/shared/product-article/ProductArticle';
import ProductArticle from '@shared/product-article/ProductArticle';

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

수정했습니다!

setSelectedLabel={setSelectedLabel}
/>
</div>
<div className={cx('productArticleContainer')}>
Copy link
Collaborator

Choose a reason for hiding this comment

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

ProductArticle 컴포넌트에 box-shadow 스타일 추가해주세요!

스크린샷 2024-01-21 오후 3 36 52

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 dropdownRef = useRef<HTMLInputElement>(null);

const [selectedLabel, setSelectedLabel] = useState(options[0].label);
const [isFilterDrawerOpen, setIsFilterDrawerOpen] = useState(false);
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
const [isFilterDrawerOpen, setIsFilterDrawerOpen] = useState(false);
const [isOpenFilterDrawer, setIsOpenFilterDrawer] = useState(false);

동사가 먼저 오는 게 좋아보여요!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

넵!

<main className={cx('mainContainer')}>
<SearchBar />
<div className={cx('filterWrapper')}>
<span>{`총 ${productArticleData.length}개`}</span>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Text 컴포넌트 사용해주세요!

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

@bottlewook bottlewook left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!

@seoye0ng seoye0ng merged commit 00dd486 into develop Jan 21, 2024
6 checks passed
@seoye0ng seoye0ng deleted the page-search branch January 21, 2024 07:43
@seoye0ng seoye0ng linked an issue Jan 21, 2024 that may be closed by this pull request
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feature 기능 개발 🎨 Html&css 마크업 & 스타일링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

페이지 UI: 검색 페이지
2 participants