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

[Style] CSS 스타일 개선 #1571

Merged
merged 22 commits into from
Oct 7, 2024
Merged

[Style] CSS 스타일 개선 #1571

merged 22 commits into from
Oct 7, 2024

Conversation

JeongRon
Copy link
Contributor

@JeongRon JeongRon commented Oct 5, 2024

📌 개요

  • CSS 스타일 개선

💻 작업사항

  • 개행 처리 기능 추가 / 대회 설명, 공지사항 내용 작성 -> 개행 처리 안되어 있는 것 개선
    스크린샷 2024-10-05 오후 1 28 06
    스크린샷 2024-10-05 오후 1 28 54

  • index 첫 페이지 / 버튼 클릭 모션 개선, 모집 중 아젠다 없을 시 UI 개선
    스크린샷 2024-10-05 오후 1 25 29

  • 프로필 페이지 / 아젠다 프로필 카드 개선 (업적 설명 마우스 hover처리)
    스크린샷 2024-10-05 오후 1 30 11

  • 프로필 페이지 / 아젠다 리스트 UI 개선 (내 프로필 아닐 시- 참여중 안보이게 개선, 리스트 없을 시 텍스트 추가)
    스크린샷 2024-10-05 오후 1 36 36

스크린샷 2024-10-05 오후 1 36 39

izone00 and others added 22 commits April 16, 2024 20:30
[deploy] 파티 모집 서비스 배포
[deploy] 파티 모집 버그 피드백 반영 후 배포
[test-deploy] 아젠다 기능 구현 테스트
[test-deploy] 빌드 conflict 해결 및 프로필 css조정
아젠다 endpoint update
[deploy] update yml for agenda endpoint
- 업적 이미지 마우스 hover 효과 적용 : 해당 업적에 대한 내용 UI로 보이게 설정
- 업적 이미지 박스 1.5개 -> 2.5개 보이게 height 늘리기
- 내 프로필이 페이지 아닐 경우 / 참여중 탭 삭제
- 데이터 (참여중, 개최중, 참여기록, 개최기록) 없을 시, 텍스트 띄워주기
@JeongRon JeongRon linked an issue Oct 5, 2024 that may be closed by this pull request
4 tasks
@JeongRon JeongRon self-assigned this Oct 5, 2024
@JeongRon JeongRon added the enhancement New feature or request label Oct 5, 2024
@JeongRon JeongRon changed the base branch from main to agenda October 5, 2024 04:39
@JeongRon JeongRon changed the base branch from agenda to main October 5, 2024 04:47
@JeongRon JeongRon changed the base branch from main to agenda October 5, 2024 04:49
Copy link
Contributor

@irenee-14 irenee-14 left a 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>
{' | '}
{'|'}
Copy link
Contributor

Choose a reason for hiding this comment

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

기존 deploy된 부분이 merge가 안 된것 같습니다! 확인부탁드려요~

Comment on lines +129 to +133
sx={{
color: activeTab === 'current' ? '#af71ff' : 'black',
'@media (min-width: 961px)': {
fontSize: '1rem', // 폰트 크기 변경
},
Copy link
Contributor

Choose a reason for hiding this comment

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

이부분은 클래스로 빼면 적용이 안되나요?! 가능하다면 부탁드립니다!

Copy link
Contributor Author

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 {
Copy link
Contributor

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>
Copy link
Collaborator

Choose a reason for hiding this comment

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

멘트 사소하지만 좋네여...

Copy link
Collaborator

@cweedlee cweedlee left a 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>
Copy link
Collaborator

Choose a reason for hiding this comment

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

굿굿

@cweedlee cweedlee merged commit cae0445 into agenda Oct 7, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Style] CSS 스타일 개선
4 participants