-
Notifications
You must be signed in to change notification settings - Fork 1
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 AboutAusgCon page to activities tab #98
Conversation
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 Activities = () => { | ||
const [currentTab, setCurrentTab] = useState<'bigchat' | 'study'>('bigchat'); | ||
const [currentTab, setCurrentTab] = useState<'bigchat' | 'ausgcon'>( |
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.
리뉴얼 디자이너 구해주세요 <-
<svg width={32} height={32} viewBox="0 0 532 532"> | ||
<path | ||
fill="currentColor" | ||
d="M355.66 11.354c13.793-13.805 36.208-13.805 50.001 0 13.785 13.804 13.785 36.238 0 50.034L201.22 266l204.442 204.61c13.785 13.805 13.785 36.239 0 50.044-13.793 13.796-36.208 13.796-50.002 0a5994246.277 5994246.277 0 0 0-229.332-229.454 35.065 35.065 0 0 1-10.326-25.126c0-9.2 3.393-18.26 10.326-25.2C172.192 194.973 332.731 34.31 355.66 11.354Z" | ||
/> | ||
</svg> |
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.
나중에는 svg도 싹 빼버리는 것도 갠춘할지도..? 굳이인가 (의견만 메모)
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.
태강쓰가 짱입니다.
. . 이 PR을 마무리로 2024 홈페이지 리뉴얼로 넘어가야할 것 같습니다,,! (clear legacy)