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 AboutAusgCon page to activities tab #98

Merged
merged 5 commits into from
Jun 15, 2024
Merged

Feat AboutAusgCon page to activities tab #98

merged 5 commits into from
Jun 15, 2024

Conversation

grayashh
Copy link
Member

  • 모바일 뷰 지원 마감 섹션 일부 수정
  • activities 페이지 안 bigchat | ausgcon 탭 추가
  • ausgcon 영상 임베드 캐러셀 추가

. . 이 PR을 마무리로 2024 홈페이지 리뉴얼로 넘어가야할 것 같습니다,,! (clear legacy)

@grayashh grayashh requested a review from Geun-Oh June 15, 2024 11:08
@grayashh grayashh self-assigned this Jun 15, 2024
@grayashh grayashh enabled auto-merge (squash) June 15, 2024 11:08
Copy link
Member

@Geun-Oh Geun-Oh left a 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'>(
Copy link
Member

Choose a reason for hiding this comment

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

요기는 나중에 활동들 더 추가되면 따로 유니온 타입으로 빼버려도 될 듯 (리뉴얼에서)

Copy link
Member Author

Choose a reason for hiding this comment

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

리뉴얼 디자이너 구해주세요 <-

Comment on lines +65 to +70
<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>
Copy link
Member

Choose a reason for hiding this comment

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

나중에는 svg도 싹 빼버리는 것도 갠춘할지도..? 굳이인가 (의견만 메모)

Copy link
Member Author

Choose a reason for hiding this comment

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

원래는 뺐었는데,, 일단 아이콘이 라이브러리로 관리되고 있지 않아서 추후 리뉴얼에서는 아이콘은 라이브러리로 관리할 듯!

Copy link
Member

Choose a reason for hiding this comment

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

태강쓰가 짱입니다.

@grayashh grayashh merged commit 64f075e into main Jun 15, 2024
1 check passed
@grayashh grayashh deleted the feat/activities branch June 15, 2024 11:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants