-
Notifications
You must be signed in to change notification settings - Fork 8
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
[Style] CSS 스타일 개선 #1571
[Style] CSS 스타일 개선 #1571
Conversation
[deploy] 파티 모집 서비스 배포
[deploy] 파티 모집 버그 피드백 반영 후 배포
[test-deploy] 아젠다 기능 구현 테스트
[test-deploy] 빌드 conflict 해결 및 프로필 css조정
[deploy] agenda 배포판
아젠다 endpoint update
[deploy] update yml for agenda endpoint
- 업적 이미지 마우스 hover 효과 적용 : 해당 업적에 대한 내용 UI로 보이게 설정 - 업적 이미지 박스 1.5개 -> 2.5개 보이게 height 늘리기
- 내 프로필이 페이지 아닐 경우 / 참여중 탭 삭제 - 데이터 (참여중, 개최중, 참여기록, 개최기록) 없을 시, 텍스트 띄워주기
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.
굳굳 호버되는 부분이 아주 좋네요! 머지 안된 부분만 확인 부탁드려요~
@@ -48,7 +48,7 @@ const Agenda: NextPage = () => { | |||
> | |||
진행중 | |||
</button> | |||
{' | '} | |||
{'|'} |
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.
기존 deploy된 부분이 merge가 안 된것 같습니다! 확인부탁드려요~
sx={{ | ||
color: activeTab === 'current' ? '#af71ff' : 'black', | ||
'@media (min-width: 961px)': { | ||
fontSize: '1rem', // 폰트 크기 변경 | ||
}, |
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.
이 부분은 mui 코드여서 클래스로 따로 빼서 처리가 안되는 것으로 알고 있어요 😅
@@ -45,6 +45,7 @@ | |||
|
|||
.announcementContent { |
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.
공지사항 부분도 개행 처리가 된건가요?!
<div className={styles.historyEmpty}>아젠다 기록이 없습니다.</div> | ||
<div className={styles.historyEmpty}> | ||
아젠다 기록이 없습니다. 새로운 아젠다를 시작해보세요! | ||
</div> |
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.
LGTM..
@@ -37,7 +37,7 @@ export default function AgendaDescription({ agendaData }: AgendaProps) { | |||
<div className={styles.midContainer}> | |||
<div className={styles.descriptionItem}> | |||
<h3>대회 설명</h3> | |||
<span>{agendaContent}</span> | |||
<span className={styles.preWrap}>{agendaContent}</span> |
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.
굿굿
📌 개요
💻 작업사항
개행 처리 기능 추가 / 대회 설명, 공지사항 내용 작성 -> 개행 처리 안되어 있는 것 개선
index 첫 페이지 / 버튼 클릭 모션 개선, 모집 중 아젠다 없을 시 UI 개선
프로필 페이지 / 아젠다 프로필 카드 개선 (업적 설명 마우스 hover처리)
프로필 페이지 / 아젠다 리스트 UI 개선 (내 프로필 아닐 시- 참여중 안보이게 개선, 리스트 없을 시 텍스트 추가)