Feat: 마이피드 리스트 개별 삭제 기능 및 UI 구현 #253
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. |
| //댓글 삭제 api | ||
| const deleteList = async (listId: string | undefined) => { | ||
| const response = await axiosInstance.delete(`/lists/${listId}`); | ||
| return response.data; | ||
| await axiosInstance.delete(`/lists/${listId}`); | ||
| }; |
There was a problem hiding this comment.
@Nahyun-Kang @kanglocal 나현님, 현지님 해당 API의 주석이 댓글 삭제로 되어있어서 이 부분 제거 및 return 문 제거했습니다~! 리스트 상세페이지 삭제 기능과 연관있는 부분이라 참고 부탁드립니다~!
There was a problem hiding this comment.
✔️ 해당 파일은 Ver2.0 입니다.
There was a problem hiding this comment.
✔️ 해당 파일은 Ver2.0 입니다.
src/styles/font.css.ts
Outdated
| /** | ||
| * 아래부터는 피그마 확인하여 정리함 | ||
| TODO 최종 디자인시스템 확인하면, 위의 변수와 아래 변수 통합하기 | ||
| */ | ||
|
|
||
| // 타이틀 | ||
| export const TitleLarge = style({ | ||
| fontSize: '2.2rem', | ||
| fontWeight: '600', | ||
| lineHeight: '2.8rem', | ||
| letterSpacing: '0.088rem', | ||
| }); | ||
|
|
There was a problem hiding this comment.
👀 피그마 Component 디자인시스템 폰트 부분에 있는 모든 폰트를 넣었습니다! 따라서, 폰트 적용하실 때 해당 파일에 있는 폰트를 적용하시면 될 것 같습니다.
다만, 스타일 적용할 때 마이피드의 경우 없는 폰트가 많아서 우선 비슷한 값으로 적용하였습니다. 추후 디자인 확정되면 수정해야 할 것 같습니다.
There was a problem hiding this comment.
이게 아마 이전 ver1.0, ver2.0에서 은서님께서 사용하셨던 폰트들이었습니다!! ver3.0에서 관리하는 폰트는 아마 유진님께서 올려놔주신 것이 전부인 것 같습니다!
하지만 말씀해주신대로 다양한 폰트 스타일들이 적용되고 있어서,, 한 번 정리는 필요할 것 같습니다ㅠㅠ 우선 위 5가지 폰트 내에서 사용해보고 너무 달라져서 추가가 필요한 경우는 추가한 후 피그마에 기록하는 방향은 어떨까요?!
There was a problem hiding this comment.
서영님~ PR 상세히 확인해 주셔서 감사드립니다. ✨
- 서영님 말씀은 피그마 아래 부분이 ver1.0, ver2.0이라는 말씀이신거죠?!
[ 피그마 ]
- 그렇다면, 해당 부분은 다시 삭제하고, 대신 폰트 1.2rem, 1.1rem 처럼 자주 사용하지만 5개 중 없는 폰트 1개만 별도로 추가하고 추후 정리하는 방향으로 가도 괜찮을까요?!
// font.css.ts
import { style } from '@vanilla-extract/css';
export const Header = style({
fontSize: '2rem',
fontWeight: '700',
letterSpacing: '-0.6rem',
});
export const Subtitle = style({
fontSize: '1.8rem',
fontWeight: '700',
letterSpacing: '-0.54rem',
});
export const BodyBold = style({
fontSize: '1.6rem',
fontWeight: '600',
letterSpacing: '-0.54rem',
});
export const Body = style({
fontSize: '1.5rem',
fontWeight: '400',
letterSpacing: '-0.54rem',
});
export const Label = style({
fontSize: '1.4rem',
fontWeight: '400',
letterSpacing: '-0.42rem',
});
// 추가
export const LabelSmall = style({
fontSize: '1.2rem',
fontWeight: '300',
lineHeight: '1.6rem',
letterSpacing: '0.6rem',
});
There was a problem hiding this comment.
- 네, 소현님! 해당 시스템은 ver1,2용이 맞습니다!
- 저는 너무 좋다고 생각합니다! 추가해주셔서 감사해요 🙇♀️🤍
src/styles/font.css.ts
Outdated
| /** | ||
| * 아래부터는 피그마 확인하여 정리함 | ||
| TODO 최종 디자인시스템 확인하면, 위의 변수와 아래 변수 통합하기 | ||
| */ | ||
|
|
||
| // 타이틀 | ||
| export const TitleLarge = style({ | ||
| fontSize: '2.2rem', | ||
| fontWeight: '600', | ||
| lineHeight: '2.8rem', | ||
| letterSpacing: '0.088rem', | ||
| }); | ||
|
|
There was a problem hiding this comment.
이게 아마 이전 ver1.0, ver2.0에서 은서님께서 사용하셨던 폰트들이었습니다!! ver3.0에서 관리하는 폰트는 아마 유진님께서 올려놔주신 것이 전부인 것 같습니다!
하지만 말씀해주신대로 다양한 폰트 스타일들이 적용되고 있어서,, 한 번 정리는 필요할 것 같습니다ㅠㅠ 우선 위 5가지 폰트 내에서 사용해보고 너무 달라져서 추가가 필요한 경우는 추가한 후 피그마에 기록하는 방향은 어떨까요?!
| - [ ] 토스트 메세지 | ||
| - [ ] 리스트 공개, 비공개 기능 | ||
| */ | ||
|
|
개요
작업 사항
참고 사항 (optional)
스크린샷
리뷰어에게