-
Notifications
You must be signed in to change notification settings - Fork 1
31 change UI #33
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
base: main
Are you sure you want to change the base?
31 change UI #33
Conversation
title: 'Bibimbap', | ||
description: 'A delicious Korean dish', | ||
title: 'coduck', | ||
description: 'coduck~', |
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.
이 부분은 아직 placeholder로 두고 나중에 수정 되는 부분인 거죠?
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.
우선은 coduck으로 남겨둬도 괜찮을 것 같긴 해요
다만 package.json
에서는 아직 bibimbap-frontend
라고 되어 있어서 아직 불일치가 존재하는 것 같아요. README.md
, LICENSE
에서도 비슷한 이슈가 존재하고 있어요
{
"name": "bibimbap-frontend",
"version": "0.1.0",
"private": true,
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.
모든 페이지가 의도대로 작성하는 것으로 보이고, 디자인 변경 사항도 전부 적용된 것으로 보여요. 다만, 아직 bibimbap
의 내용이 남아있는 부분이 있어 comment로 남겨둬요
<div className='min-h-screen bg-gradient-to-br from-gray-50 to-gray-100'> | ||
<div className='container px-4 py-6 mx-auto'> | ||
<div className='flex items-center justify-between mb-6'> | ||
<div> | ||
<h1 className='text-3xl font-bold text-gray-900 flex items-center gap-3'> |
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/problems/page.tsx
의 41번 라인부터 45번 라인까지 똑같은 css 코드가 있어서, 이런 부분은 나중에 찾아서 공통으로 묶어서 리팩토링 하면 좋을 것 같아요
/* globals.css */
.page-background {
@apply min-h-screen bg-gradient-to-br from-gray-50 to-gray-100
}
.page-container {
@apply container px-4 py-6 mx-auto;
}
.page-header {
@apply flex items-center justify-between mb-6;
}
.page-title {
@apply text-3xl font-bold text-gray-900 flex items-center gap-3;
}
.page-title-icon {
@apply h-8 w-8 text-primary;
}
title: 'Bibimbap', | ||
description: 'A delicious Korean dish', | ||
title: 'coduck', | ||
description: 'coduck~', |
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.
우선은 coduck으로 남겨둬도 괜찮을 것 같긴 해요
다만 package.json
에서는 아직 bibimbap-frontend
라고 되어 있어서 아직 불일치가 존재하는 것 같아요. README.md
, LICENSE
에서도 비슷한 이슈가 존재하고 있어요
{
"name": "bibimbap-frontend",
"version": "0.1.0",
"private": true,
비빔밥 -> 코덕 이름 변화 후 전반적인 대표색, 로고, 메인 페이지 및 관련 css, 이에 따른 하위 페이지 변경 작업에 따른 코드입니다.
구체적인 변경 사항은 다음과 같습니다.
관련 작업을 위한 ui 파일 추가 입니다.