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

Refactor/browse view #92

Merged
merged 4 commits into from
Dec 7, 2022
Merged

Refactor/browse view #92

merged 4 commits into from
Dec 7, 2022

Conversation

jinwoong16
Copy link
Collaborator

@jinwoong16 jinwoong16 commented Dec 6, 2022

📕 Issue Number

Close #91

📙 작업 내역

구현 내용 및 작업 했던 내역

  • BrowseCell의 corner radius를 15로 지정
  • BrowseCell의 배경색을 Max Ligth Blue로 지정
  • QuestCell의 배경색을 투명으로 변경
  • BrowseCell의 헤더의 폰트를 볼드체로 변경
  • QuestCell의 폰트색상을 Max Violet으로 변경

📘 작업 유형

  • 신규 기능 추가
  • 버그 수정
  • 리펙토링
  • 문서 업데이트

📋 체크리스트

  • Merge 하는 브랜치가 올바른가?
  • 코딩컨벤션을 준수하는가?
  • PR과 관련없는 변경사항이 없는가?
  • 내 코드에 대한 자기 검토가 되었는가?
  • 변경사항이 효과적이거나 동작이 작동한다는 것을 보증하는 테스트를 추가하였는가?
  • 새로운 테스트와 기존의 테스트가 변경사항에 대해 만족하는가?

📝 PR 특이 사항

PR을 볼 때 주의깊게 봐야하거나 말하고 싶은 점

  • 배경색이 좀 더 산뜻한 색이 있을까요. 탭바의 색상이랑 안맞는 것 같기도 하고,.,.



@sprituz
Copy link
Member

sprituz commented Dec 6, 2022

좋습니다

@Jeonhui
Copy link
Member

Jeonhui commented Dec 6, 2022

음 아니면 탭바 색을 맥스 발색으로 바꿔볼까요? Browse 자체는 잘 어울리는데 탭바가 너무 튀는색이라 그런것 같네요

@jinwoong16
Copy link
Collaborator Author

jinwoong16 commented Dec 6, 2022

max dark yello로 하면 이런 느낌입니다. 국민투표해야겠네요.

@wickedRun
Copy link
Collaborator

깔끔한것 같습니당!

@jinwoong16 jinwoong16 merged commit e00332d into develop Dec 7, 2022
@jinwoong16 jinwoong16 deleted the refactor/BrowseView branch December 7, 2022 08:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

BrowseView refactor
4 participants