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

Design: 커뮤니티 UI 구현 #19

Merged
merged 26 commits into from
May 15, 2024
Merged

Conversation

mung96
Copy link
Contributor

@mung96 mung96 commented May 12, 2024

#️⃣연관된 이슈

#13

📝작업 내용

추가된 라이브러리 & 플러그인

  • dayjs : 시간과 날짜를 편리하게 조작할수 있도록 도와주는 라이브러리 (타이머 구현시 사용)

다음 페이지들 UI 구현

  • 랭킹페이지
  • 친구페이지
  • 채팅페이지
  • 베팅페이지

🐛버그 있음

(한번만 확인해주세요. 안고쳐지면 이번 pr 머지하고 작업하면서 계속 찾아보겠습니다.)

  • 랭킹페이지에 그라데이션 색상이 세은이 코드와 충돌함.
// tailwind.config.js L39

      Ranking_Bar_Start: '#88A0E9',
      Ranking_Bar_End: '#C7A4F4',
    },    //1번

     backgroundImage: {
    gradient: 'linear-gradient(0.25turn,#3F70DD, #B377F3)',
    },   //2번

문제: 1번과 2번 동시 적용이 안됨.

  • 1번 구간 주석처리시 홈페이지 그라데이션 적용가능
  • 2번 구간 주석처리시 랭킹페이지 그라데이션 적용가능
  • 하지만 주석을 모두 풀었을때 동시 적용이 안됨

🔥 check your project preview here!

@mung96 mung96 self-assigned this May 12, 2024
@mung96 mung96 added the Design🎨 CSS 등 사용자 UI 디자인 변경 label May 12, 2024
@mung96 mung96 added the Feat✨ 새로운 기능 추가 label May 12, 2024
@mung96 mung96 added the Fix🐛 버그 수정 label May 12, 2024
Copy link
Contributor

@ChaMinXD ChaMinXD left a comment

Choose a reason for hiding this comment

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

이슈 문제는 아마 계층문제가 아닐지..?

@se-eun-park
Copy link
Contributor

현명리뷰
우선 코드 보기 전에, 버셀 먼저 확인했는데 해상도가 1920일 때, 화면 너비가 모자랍니다! 그리고 배경색도 figma와 다른 것 같아요. 개발자도구로 해상도 왔다갔다 하면서, 너비와 비율 유지되도록 코드 수정하면 좋을 것 같습니다.

Copy link
Contributor

@se-eun-park se-eun-park left a comment

Choose a reason for hiding this comment

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

UI 너비를 큰 해상도에서도 어느정도 호환되도록 바꿔주세요! Dark모드 적용 안된 폰트 컬러들도 꽤나 보입니당.

Comment on lines 42 to 44
// backgroundImage: {
// gradient: 'linear-gradient(0.25turn,#3F70DD, #B377F3)',
// },
Copy link
Contributor

Choose a reason for hiding this comment

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

    extend: {
      backgroundImage: {
        gradient: 'linear-gradient(0.25turn,#3F70DD, #B377F3)',
      },
    },

theme에서 extend로 컬러 위치를 옮겼더니 해결됐습니다. theme에 적용한 색들은 기본 컬러들을 덮어써서 등록한 컬러 외엔 사용할 수 없게 하기 때문에 (의도한 설정임) 발생한 문제같습니다!

Copy link
Contributor

@se-eun-park se-eun-park left a comment

Choose a reason for hiding this comment

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

바뀐거 확인했습니다! 수고하셨어용

@ChaMinXD
Copy link
Contributor

확인이요~

Copy link
Contributor

@ChaMinXD ChaMinXD left a comment

Choose a reason for hiding this comment

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

고생고생

@ChaMinXD ChaMinXD merged commit 04b07bd into dev May 15, 2024
1 check passed
@ChaMinXD ChaMinXD deleted the 13-design-커뮤니티-ui-구현 branch May 15, 2024 09:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Design🎨 CSS 등 사용자 UI 디자인 변경 Feat✨ 새로운 기능 추가 Fix🐛 버그 수정
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants