feat: 로그인 페이지 ui #24
feat: 로그인 페이지 ui #24
Conversation
| export const LargeClose = ({ className, ...props }: SvgProps) => { | ||
| return ( | ||
| <svg | ||
| xmlns="http://www.w3.org/2000/svg" |
There was a problem hiding this comment.
color를 없애면 해당 아이콘의 색을 바꿀수 있는 방법이 있을까요? props를 fill에 적용시킬 수 있는 방법이 있을까요?
There was a problem hiding this comment.
오 좋습니다! 속성 추가해서 넣어주었습니다.
사용하는 밖에서 선언하시면 적용 가능합니다.
| export default function DateFormatter(publishedAt: string): string { | ||
| export function DateFormatter(publishedAt: string): string { | ||
| return new Intl.DateTimeFormat('ko-KR', { | ||
| year: 'numeric', | ||
| month: '2-digit', |
There was a problem hiding this comment.
혹시 shared가 아니라 domain에 넣으신 이유를 알 수 있을까요?
figma를 보면 날짜가 필요한 곳이 popular뿐만이 아닌 것 같아서용
There was a problem hiding this comment.
또 어디에 있나요??
저는 해당 도메인에만 사용되는 유틸함수로 알고 있어서 옮겼습니다.
|
|
||
| useEffect(() => { | ||
| setFormedQuestions(isTyped ? answers : `${answers}입니다.`); | ||
| }, []); | ||
| }, [answers, isTyped]); |
There was a problem hiding this comment.
의존성을 추가하신 이유가 있을까용?
isTyped와 answers가 동적으로 변하게 쓰실 곳이 있을까요?
isTyped가 의존성 배열에 들어가게 되면 userinfo 페이지는 의도데로 돌아가지 않아서요!
There was a problem hiding this comment.
다시 수정해서 올려놨습니다!
| const routeMap = { | ||
| const _routeMap = { | ||
| home: '/', | ||
| login: '/login', | ||
| setting: '/setting', |
There was a problem hiding this comment.
오 혹시 이름은 이렇게 변경하신 이유는 뭘까용?
| import clsx from 'clsx'; | ||
|
|
||
| const SIZE = { | ||
| larage: '16', | ||
| large: 'h-20', | ||
| medium: 'h-16', | ||
| xlsmall: 'h-12', | ||
| lsmall: 'h-10', | ||
| xsmall: 'h-8', | ||
| } as const; | ||
|
|
||
| interface SpacingProps { | ||
| size: keyof typeof SIZE; | ||
| size?: keyof typeof SIZE; | ||
| className?: string; | ||
| } | ||
|
|
||
| export default function Spacing({ size }: SpacingProps) { | ||
| return <div className={`h-${SIZE[size]}`} />; | ||
| export default function Spacing({ size = 'medium', className }: SpacingProps) { | ||
| return <div className={clsx(SIZE[size], className)} />; | ||
| } |
There was a problem hiding this comment.
고정 값을 넣으면 다른 화면 크기에 따라 responsive 하게 바뀔 수 있나용?
전 그게 고민이었는데 만약 가능하다면 정말 좋은 방법 같습니다
There was a problem hiding this comment.
responsive하게 만들기 위해서는 gap으로 조정해서 하는게 젤 좋습니다!
지금은 일단 이렇게 적용하였습니다
이슈 번호
작업한 목록을 작성해 주세요
스크린샷
2025-02-07.1.41.17.mov
pr 포인트나 궁금한 점을 작성해 주세요
연관된 issue: #undefined