Conversation
…campwm-2024/web33-Nocta into Feature/#127_탭_브라우징_피드백_반영
- pageId, isSidebar 를 객체로 받아오도록 수정 #127
…into Feature/#127_탭_브라우징_피드백_반영
| <Modal isOpen={isOpen} primaryButtonLabel="확인" primaryButtonOnClick={closeModal}> | ||
| <p> | ||
| 최대 {MAX_VISIBLE_PAGE}개의 페이지만 화면에 표시할 수 있습니다. | ||
| <br /> | ||
| 사용하지 않는 페이지는 닫아주세요. | ||
| </p> | ||
| </Modal> |
There was a problem hiding this comment.
모달 재사용하는 부분 굉장히 좋은 것 같습니다!
There was a problem hiding this comment.
확장성이 굉장히 좋아보입니다. 제 화면에서는 '있습니' 에서 <br/> 이 되어서
최대 nn개의 페이지만 표시할 수 있습니다. 로 줄이는건 어떨까요? 😂
There was a problem hiding this comment.
오 너무 좋습니다~!~! 안그래도 문구 맘에 안들었는데,, 감사합니다!
| import CloseIcon from "@assets/icons/close.svg?react"; | ||
| import ExpandIcon from "@assets/icons/expand.svg?react"; | ||
| import MinusIcon from "@assets/icons/minus.svg?react"; |
There was a problem hiding this comment.
svg 이미지를 컴포넌트로 사용하는 방식이네요!
| @@ -3,4 +3,7 @@ import { css } from "@styled-system/css"; | |||
| export const pageTitle = css({ | |||
| textStyle: "display-medium24", | |||
| color: "gray.500", | |||
| textOverflow: "ellipsis", | |||
| overflow: "hidden", | |||
| whiteSpace: "nowrap", | |||
There was a problem hiding this comment.
글자들이 밖으로 나가는 부분이 없어졌군요! 😄
There was a problem hiding this comment.
다만 placeholder 는 여전히 보이네요 ..!
There was a problem hiding this comment.
저,, 저건,, 일단 몰루,, 하겠습니다...
There was a problem hiding this comment.
webkit을 통해 우측 스크롤바 디자인을 변경해봐도 좋을것 같습니다!
There was a problem hiding this comment.
스크롤바 디자인 간단하게 수정해봤습니다!
2024-11-17.8.03.02.mov
| boxShadow: { duration: 0.2 }, | ||
| x: { type: "tween", duration: 0.03, ease: "linear" }, | ||
| y: { type: "tween", duration: 0.03, ease: "linear" }, | ||
| scale: { type: "spring", stiffness: 300, damping: 15 }, |
There was a problem hiding this comment.
묵직하게 움직이는 애니메이션이 빠르게 바뀌었군요. 지금이 UX적으로 반응이더 좋은 듯 하네요!
pnpm-lock.yaml
Outdated
There was a problem hiding this comment.
플러그인 설치처럼, 종속성 관련한 정보를 저장한다고 합니다! 그래서 이번에 svgr이라는 플러그인을 설치했는데, 그에 따라 pnpm-lock파일이 바뀌어서, 이걸 푸시안해주면 lint 에러가 뜨더라구요...
There was a problem hiding this comment.
아 .. 저를 newsStand에서 애먹게 했던 그 플러그인이군요 ㅋㅋ..
알려주셔서 감사합니다 !!
| @@ -16,11 +16,13 @@ export const navWrapper = css({ | |||
| gap: "md", | |||
| flexDirection: "column", | |||
| width: "100%", | |||
| overflowY: "scroll", | |||
There was a problem hiding this comment.
page가 잠깐 생길때 x축에도 스크롤이 발생하네요.
x축 스크롤은 안보이는게 처리하는게 좋을까요?
There was a problem hiding this comment.
오 좋은 것 같습니다!! 수정하겠습니다!
📝 변경 사항
🔍 변경 사항 설명
🙏 질문 사항
📷 스크린샷 (선택)
2024-11-17.12.35.30.mov
✅ 작성자 체크리스트