prefragrance / prefragrance-client

향으로 취향을 공유하고 나의 취향을 찾는 소셜미디어, 취향의 프론트엔드 저장소
0 stars 0 forks source link

[Fix] 검색바 및 검색모달 수정 #17

Closed jellysoo97 closed 2 years ago

jellysoo97 commented 2 years ago

주요 수정사항

constants/color 값 수정했습니다. 제가 보이는대로 수정하긴 했는데 혹시나 color값 이상하게 나오는거 수정해주세요~

jellysoo97 commented 2 years ago

Sep-24-2022 23-03-26

백이랑 api 통신할 때 자꾸 CORS 에러가 떠서 이것만 해결하면 검색 기능 남은 건 아래 2개 입니다.

zizonyoungjun commented 2 years ago

확인했습니다.

HiimKwak commented 2 years ago
image
  1. 검색어 모달 - 검색어 탭 currentTab과 otherTab 간 차이를 좀 더 명확히 구분하고자 font-weight값을 선택적으로 부여했습니다. font-size도 1.2rem은 너무 큰 것 같아서 다시 1rem으로 돌려놓았습니다.(+ 제 생각엔 box-shadow 디자인이 좀 더 괜찮은 것 같은데 잘 구별이 안됐었나요? 지금 green 색상의 언더바도 나쁘진 않습니다만 제 생각엔 green 컬러의 언더바보단 green 컬러의 언더 박스 쉐도우가 더 모던틱하고 이쁘지 않을까 싶습니다.!)
  2. 검색어 블럭에 들어가는 돋보기 아이콘의 명도를 0.5로 낮췄습니다(기존 컬러는 너무 쨍했음).

그리고 제가 발견한 일요일 스프린트 때 논의해볼 사항 여기다 기록하겠습니다.

  1. 검색어 모달이 사라지는 경우 - 현재 풀리퀘의 사라짐 범위보다 이전 코드의 사라짐 범위가 더 좋지 않을까요?
  2. 검색창 input에 클릭을 할 시 무조건 검색어 모달이 닫혔다 다시 열리는 현상
  3. 검색어가 아무것도 없었음에도 최근 검색어엔 저장되는 현상

위 2번과 3번의 이해를 돕고자 아래 움짤을 첨부합니다.

Q

jellysoo97 commented 2 years ago

마지막 issue는 새로 pr 열겠습니다~