✨ feature: add GNB component #43
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
1 task
icon 컴포넌트 props를 이용하여 다양한 크기,색깔의 새로운 icon을 생성할 수있게 변경하였습니다.
Closed
hwangyena
reviewed
Nov 26, 2022
| * className을 통해 tailwindCSS를 이용할 수 있습니다. | ||
| */ | ||
|
|
||
| export interface InternalSvgProps { |
There was a problem hiding this comment.
아이콘 내에 props들이 많이 바뀌지 않을 것 같은데, 그냥 icons 내에 모든 파일 내용 나두고 props로 각자 조절할 수 있게 바꾸는 것은 어떨까요?
There was a problem hiding this comment.
따로 common svg 컴포넌트를 사용하지않고 아래의 PR에서 예나님이 사용하셨던것처럼 svg를 사용하자고 하신게 맞으실까요?
| return ( | ||
| <nav className="w-[375px]"> | ||
| <ul className="flex w-full justify-evenly"> | ||
| <NavigationBarList items={NAV_LIST_INFO} /> |
There was a problem hiding this comment.
굳이 props 없이 <NavigationBarList /> 컴포넌트 내에 NAV_LIST_INFO를 나둬도 되지않을까요?
There was a problem hiding this comment.
네 그 방법이 더 좋을 거 같아요! 컴포넌트 내에 NAV_LIST_INFO를 사용하는 쪽으로 변경하겠습니다~!
hwangyena
approved these changes
Nov 26, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What's Changed?
Svg컴포넌트를 만들었습니다.Svg컴포넌트를 만들었습니다.xmlns을 매번 적을 필요가 없습니다.width,height,viewbox를 설정할 수 있습니다.color를 이용하여 간단하게 색상을 변경할 수 있습니다.GNB에 필요한 각Icon컴포넌트를 만들었습니다.Profile,TalentRegister,TalentSearch아이콘이 제작되었습니다.IconAnchor컴포넌트를 만들었습니다.Icon이 포함된next/link컴포넌트를 생성할 수 있습니다.Anchor컴포넌트의 모든prop을 사용할 수 있습니다.className을 통해 커스텀 스타일링을 적용할 수 있습니다.NavigationBarList컴포넌트를 만들었습니다.padding값이 필요해보입니다. ( 추후 논의가 필요합니다. )style은 이후 디자인에 따라 변경될 예정입니다.uuid의 경우merge이후 예나님이 만들어주신util함수 이용하여 변경 예정입니다.Navigation컴포넌트를 만들었습니다.Navigation아이템들은NAV_LIST_INFO에 의해 결정됩니다.Screenshots
Related to any issues?
Dependency Changes
Test Code