Feat: 마이피드 리스트 비공개/공개 토글기능 및 Ver3.0 디자인 구현 #256
Conversation
* feat: layout 컴포넌트의 metadata 수정 - 리스티웨이브 한글 타이틀 추가 및 description 수정 * Feat: 마이리스트, 콜라보리스트 페이지에 Dynamic 메타데이터 설정 * Feat: 리스트 상세 페이지에 Dynamic 메타데이터 설정 * Refactor: 메타데이터 별도 상수로 분리 * Feat: sitemap 설정 및 도메인 url 별도 상수로 분리 * Feat: 트렌딩리스트에 해당하는 리스트의 상세페이지가 sitemap에 추가되도록 설정 * Feat: robots.txt 설정 * Fix: sitemap.ts 트랜딩리스트 호출 방법을 fetch API로 수정 * Fix: sitemap.ts 트랜딩리스트 호출 로직 제거
* feat: 라벨 및 카테고리 클릭시 검색페이지로 이동 * fix: 리스트 상세조회, 카테고리 전체조회 API 수정에 따라 타입 변경
ParkSohyunee
requested review from
Eugene-A-01,
kanglocal,
Nahyun-Kang and
seoyoung-min
|
@ParkSohyunee is attempting to deploy a commit to the Eujin Ahn's projects Team on Vercel. A member of the Team first needs to authorize it. |
There was a problem hiding this comment.
✔️ Ver2.0 파일입니다.
There was a problem hiding this comment.
✔️ Ver2.0 파일입니다.
src/lib/constants/toastMessage.ts
Outdated
| deleteListSuccess: '리스트를 삭제했어요.', | ||
| deleteListError: '리스트 삭제에 실패했어요. 다시 시도해주세요.🥲', | ||
| visibilityListSuccess: '리스트 변경사항을 저장했어요.', | ||
| visibilityListError: '리스트 변경사항 저장에 실패했어요. 다시 시도해주세요.🥲', |
There was a problem hiding this comment.
리스트 삭제, 비공개/공개 시 토스트 메세지를 적용함에 따라 문구를 생각해 보았는데 괜찮은지 궁금합니다!
There was a problem hiding this comment.
-
실패메시지들이 길이가 긴 것 같아서 걱정되는 점이 있습니다.
폰 화면 크기에서 중간에 줄바꿈 되거나 하지는 않는지 확인해보면 좋을 것 같습니다!!
만약 중간에 줄이 넘어가거나 한다면 "리스트 삭제에 실패했어요 (줄바꿈하고) 다시시도해주세요" 같이 줄을 직접 나눠주는 방법이나, 아예 글자수 줄이기 (리스트 삭제 실패🥲 등으로 글자수 줄여버리기) 고려하면 좋을 것 같아요~ -
비공개/공개 메시지는 리스트 변경사항 저장 보다는
리스트 공개 여부 변경 완료 / 리스트 공개 여부 변경 실패 등의 메시지면 더 직관적일 것 같아요!
There was a problem hiding this comment.
오! 유진님 너무 좋은 의견 감사합니다.🥹
글자수를 줄이는 방법으로 토스트 메세지를 수정하였고, 비공개/공개 여부도 변경사항 대신 직관적인 메세지로 변경하였는데 더 직관적으로 보이네요✨👍
| position: 'sticky', | ||
| top: '1rem', | ||
| top: '-1px', // observer API로 sticky 영역이 stuck 상태임을 감지하기 위해 |
There was a problem hiding this comment.
이렇게도 감지할 수 있군요,, 여러 수 배워갑니다 소현님 !!
|
|
||
| if (type !== ('success' || 'error' || 'warning')) { | ||
| if (type === 'default') { | ||
| toast(txt, toastOption); | ||
| } else { | ||
| toast[type](txt, toastOption); |
There was a problem hiding this comment.
토스트 조건문을 수정하여, 토스트 타입에 따른 아이콘이 올바르게 보이도록 수정하였습니다.
- success
- error
- warning
- default (아이콘 없는 토스트 메세지 형태)
개요
작업 사항
참고 사항 (optional)
관련 이슈
1. 스크롤 시 리스트 정보 영역 고정 UI
2. 리스트가 있다는 것을 보여주는 상, 하단부 그라데이션 스타일 적용 고민 및 해결 방법
stuck(붙어있음)을 감지하여 그라데이션을 적용top: '-1px'을 줌으로써 달라붙는 sticky 요소에 벗어나는 순간을 체크할 수 있다. 참고스크린샷
스크롤 전
스크롤 중
스크롤 끝
리뷰어에게