-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feature/#370 학습자료 모달에서 UI 수정 #383
base: develop
Are you sure you want to change the base?
The head ref may contain hidden characters: "Feature/#370-\uD559\uC2B5\uC790\uB8CC_\uBAA8\uB2EC\uC5D0\uC11C_UI_\uC218\uC815"
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
const IconBox = ({ leftIcon, content, rightIcon, handleClick, cursor = 'default' }: IconBoxProps) => { | ||
return ( | ||
<Flex align="center" gap="1" w="100%" h="40px" minH="40px" color="white" bg="orange_light" borderRadius="2xl"> | ||
<IconButton as="div" flexShrink="0" aria-label="" icon={leftIcon} size="icon_md" variant="transparent" /> | ||
<Text textStyle="bold_md" flex="auto" cursor="default" isTruncated> | ||
<Text textStyle="bold_md" flex="auto" cursor={cursor} isTruncated> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
다른 부분도 그렇고, default도 이렇게 직접 default라는 것을 명시해야 하나요? 말 그대로 기본값이라 그냥 둬도 된다고 생각이 들어서 명시해야 하는 이유가 궁금합니다!!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
아 저부분은 IconBox를 지금은 default 로 사용하지만 나중에는 pointer 나 text로 사용할 가능성도 있을 거라 생각되어서 저렇게 만들어 두었습니다!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
관련 이슈
작업 요약
작업 상세 설명
리뷰 요구 사항
3분
미리 보기