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

[FE] 은하 색상 변경 #300

Merged
merged 10 commits into from
Dec 10, 2023
Merged

[FE] 은하 색상 변경 #300

merged 10 commits into from
Dec 10, 2023

Conversation

KimGaeun0806
Copy link
Member

📎 이슈번호

📃 변경사항

  • 은하의 색을 변경했습니다.

    • 저희 서비스 UI와 잘 어우러지도록 파랑-보라빛이 돌도록 조합했습니다.
    • 동민님 준섭님의 동의 하에 진행햇습니당
    • 이의가잇다면 코멘트부탁드립니다
  • 은하의 별 개수를 6000개에서 4000개로 변경했습니다.

  • 처음 우주에 도착했을 때의 카메라 position을 변경했습니다.

    • 은하의 모양이 한 눈에 보이도록 조금 더 멀리 배치했습니다.
  • 글쓰기 모달 디자인 변경

    • 모달이 너무 커진 관계로 요소 간 margin 값을 줄였습니다.
    • 이미지의 크기를 줄였습니다.
  • api 호출에서 에러나는 부분을 해결했습니다.

  • nickName이 결졍되기 전에 api가 호출되는 부분이 있어서, nickName이 falsy한 값일 때는 실행되지 않도록 했습니다.

  • 로그인 모달에 '예시 은하 구경하기' 버튼을 추가했습니다.

    • 기존의 둘러보기 버튼을 대체합니다.
    • 예시 은하로 이동합니다.
  • 공유링크 에러를 해결했습니다.

🫨 고민한 부분

  • 은하 색 조합...
    • 많은 고민 끝에 두 분의 동의를 얻어 결정햇슴다. 별론것같기도하고

📌 중점적으로 볼 부분

  • 은하의 색

🎇 동작 화면

image
image
image

💫 기타사항

  • 현재 guest로 접근하면 은하 형태를 불러오는 api에서 에러가 나고 있습니다.
  • 로그인하지 않아도 하단바에 로그아웃 버튼이 생기는 문제가 있습니다. 이건 곧 수정해서 바로 PR 올리겠습니다.

- 이미지 사이즈 축소
- margin값 조정
- home 카메라 포지션 변경
- 은하 색 변경
- 은하 별 갯수 축소
- nickName이 불러와지기 전에 api 호출되는 것을 막음
- 로그인창의 둘러보기를 예시 은하 구경하기 버튼으로 변경
- 공유링크 이상하게 뜨는 문제 해결
@KimGaeun0806 KimGaeun0806 added ✨ Feature 기능 개발 🎨 HTML&CSS 마크업 & 스타일링 🐛 BugFix 에러 해결 FE 프론트엔드 labels Dec 10, 2023
@KimGaeun0806 KimGaeun0806 self-assigned this Dec 10, 2023
@KimGaeun0806
Copy link
Member Author

너무.. 보랏빛이 도는 것 같기도 하고요? 좀 더 흰색인게 좋으려나요

@bananaba @SongJSeop @qkrwogk @MinboyKim

MinboyKim
MinboyKim previously approved these changes Dec 10, 2023
Copy link
Member

@MinboyKim MinboyKim left a comment

Choose a reason for hiding this comment

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

고생하셨어요 전 보라색도 이쁜 것 같습니다!

Copy link
Collaborator

@bananaba bananaba left a comment

Choose a reason for hiding this comment

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

galaxy api 오류 해결 완료됐습니다

- 닉네임 없이 링크를 요청하지 않도록 수정했다
@bananaba bananaba self-requested a review December 10, 2023 14:36
@KimGaeun0806 KimGaeun0806 merged commit ef0ddaa into fe-develop Dec 10, 2023
@KimGaeun0806 KimGaeun0806 deleted the feature/edit-galaxy branch December 10, 2023 14:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 BugFix 에러 해결 FE 프론트엔드 ✨ Feature 기능 개발 🎨 HTML&CSS 마크업 & 스타일링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants