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

Update 2024 Winter Contents #14

Merged
merged 7 commits into from
Mar 13, 2024
Merged

Update 2024 Winter Contents #14

merged 7 commits into from
Mar 13, 2024

Conversation

2093ab
Copy link
Member

@2093ab 2093ab commented Mar 12, 2024

  • Update 2024 SUAPC
  • Update 2024 Hall Of Fame
  • Update Camp Contest not held
  • Add Header backdrop-filter

@2093ab 2093ab requested a review from witch-factory March 12, 2024 07:20
Copy link
Contributor

@witch-factory witch-factory left a comment

Choose a reason for hiding this comment

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

고생하셨습니다. 캠콘 페이지 말씀드린 부분만 괜찮으시면 수정하시고 머지하셔도 될 것 같습니다!

@@ -54,6 +54,7 @@ const Header = styled(({ className }) => {
</header>
);
})`
backdrop-filter: blur(1.5rem);
Copy link
Contributor

Choose a reason for hiding this comment

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

어차피 이후 많은 수정이 있겠지만 이왕 헤더 바꾼거 글씨 크기도 조금만 키우죠. DesktopTopbarItem, MobileTopbarItem글씨 크기를 1rem으로 바꾸기만 하는 걸로...

title={"이번 시즌에는 Camp Contest가 진행되지 않았습니다"}
content={"다음 시즌을 기대해 주세요"}
/>
)}
Copy link
Contributor

Choose a reason for hiding this comment

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

그래도 지금 캠프 콘테스트 페이지에서 가장 먼저 보이는 게 2024 Winter일 텐데 콘테스트 일자, 소개, 출제경향은 남겨두는 게 좋지 않을까요? 콘테스트 일자에 이번 시즌에는 Camp Contest가 진행되지 않았습니다를 써두고 하단에 적당한 말 써두면 좋지 않을까 합니다.

@@ -163,7 +163,7 @@ const Suapc = () => {
<ArchiveButton
disabled={!Boolean(currentSeasonData.fileNames.solutionPdf)}
href={
(currentSeasonIdx !== 0 ? "https://api.suapc.kr/" : "/res/") +
(currentSeasonIdx > 1 ? "https://api.suapc.kr/" : "/res/") +
Copy link
Contributor

Choose a reason for hiding this comment

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

이건 지금 시즌에는 api를 이용해서 사진 가져오는 대신 public/res 폴더 사진 쓰려고 그런 게 맞죠?

Copy link
Member Author

Choose a reason for hiding this comment

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

네넵 pdf랑 포스터를 서버에 못 올려둬서 일단 이렇게 바꿨어요

Copy link
Contributor

Choose a reason for hiding this comment

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

포스터 ㄷㄷ 갓은채

@2093ab 2093ab merged commit f3b98b9 into icpc-sinchon:main Mar 13, 2024
1 check passed
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