Fix&Refact/ 카드 컴포넌트 레이아웃 정리, 검색기록 삭제 수정, 로그인페이지 에러메시지 출력 조건 수정 - Githubissues

PinTogether / frontend

pintogether web frontend dev

https://pintogether-frontend.vercel.app

1 stars 1 forks source link

Fix&Refact/ 카드 컴포넌트 레이아웃 정리, 검색기록 삭제 수정, 로그인페이지 에러메시지 출력 조건 수정 #144

Closed jwo1024 closed 7 months ago

jwo1024 commented 7 months ago

🛠️ 변경 사항

보고받은 에러 수정

search log 삭제시 보여지는 배열에서도 삭제, SearchLogRenderer 파일 분리

css 수정 & refactor

PinCard 깨지는 레이아웃 정리
PlaceCard icon 위치 정렬 정리
collectionPage의 긴 태그가 들어왔을때를 고려하여 수정

기존 컬렉션 페이지 버튼 수정 사항

"핀 보기" -> "핀 장소"
"핀 리뷰 같이 보기" -> "핀 리뷰"
페이지 기본 보기 상태는 "핀 리뷰" 로 변경하였습니다.
보여지는 화면 상태관리 enum으로 리펙토링

로그인

로그인 시도 전에도 에러메시지 나오는 것 수정, message 이벤트로 주고받는 내용물 enum 으로 변경
- 변경 전에는 모든 message 이벤트를 받고 있어서, react 개발자 도구가 보내는 메시지를 받아도 로그인 실패 여부를 확인함.
- source 를 설정해주어 login-popup 에서 보내는 메시지인지 확인하도록 수정.

📝 주의 사항 & 리뷰 요청

⚡️ Issue number & Link

📸 ScreenShot

핀 카드

수정 전
수정 후

컬렉션 조회 페이지

수정 전
수정 후

vercel[bot] commented 7 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Updated (UTC)
pintogether-frontend	✅ Ready (Inspect)	Visit Preview	Apr 4, 2024 6:37pm

UMGGG commented 7 months ago

바뀐거 너무 깔끔하고 좋습니당 근데 핀 카드에 있던 세가지 버튼들은 없어지는건가요?

jwo1024 commented 7 months ago

바뀐거 너무 깔끔하고 좋습니당 근데 핀 카드에 있던 세가지 버튼들은 없어지는건가요?

앗 안없어집니다 ! 예시 카드에 옵선을 다르게 줘버렸네요,, 아래 이미지의 2번째 카드 처럼 들어갑니닷