-
Notifications
You must be signed in to change notification settings - Fork 6
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
페이지 UI: 검색 페이지 #104
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
고생하셨습니다!! 제가 Dropdown 컴포넌트를 엉망으로 만들어놨네요 ㅠㅠ
이따 다시 리팩토링 하겠습니다!
src/app/search/page.tsx
Outdated
|
||
import classNames from 'classnames/bind'; | ||
|
||
import ProductArticle from '@/components/shared/product-article/ProductArticle'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
import ProductArticle from '@/components/shared/product-article/ProductArticle'; | |
import ProductArticle from '@shared/product-article/ProductArticle'; |
There was a problem hiding this comment.
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')}> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수정했습니다!
src/app/search/page.tsx
Outdated
const dropdownRef = useRef<HTMLInputElement>(null); | ||
|
||
const [selectedLabel, setSelectedLabel] = useState(options[0].label); | ||
const [isFilterDrawerOpen, setIsFilterDrawerOpen] = useState(false); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
const [isFilterDrawerOpen, setIsFilterDrawerOpen] = useState(false); | |
const [isOpenFilterDrawer, setIsOpenFilterDrawer] = useState(false); |
동사가 먼저 오는 게 좋아보여요!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
넵!
src/app/search/page.tsx
Outdated
<main className={cx('mainContainer')}> | ||
<SearchBar /> | ||
<div className={cx('filterWrapper')}> | ||
<span>{`총 ${productArticleData.length}개`}</span> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Text 컴포넌트 사용해주세요!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
넵!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
고생하셨습니다!
해당 사항 (중복 선택)
설명
Key Changes
How it Works
To Reviewers
검색페이지 제작 도중에 dropdown 컴포넌트를 수정하였습니다. (클릭된 라벨이 선택되도록)
검색페이지에 사용되는 dropdown option 중에 위반제품순이 있는데 글자수가 길어서 레이아웃이 깨지고 있습니다. 이부분을 글자 크기대로 늘릴지 아니면 글자가 잘리도록 할지 고민됩니다! 🤔
이후에 filter drawer ui 제작 예정입니다.