-
Notifications
You must be signed in to change notification settings - Fork 7
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
[SP2] 프로젝트 서비스 이용 가능 및 창업중 정보 보여주기 #254
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.
수과셨어요~!~!
export default function ServiceInfo({ text }: ServiceInfoProps) { | ||
return ( | ||
<S.Info> | ||
<Image src={icEllipseGreen} alt="" width={5} height={5} /> |
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.
svg를 ReactComponent로 넣거나 div에 border-radius 50% 주어서 하는 것은 어떠신가용?!?!
alt를 비우지 않을 방법을 생각하다가 생각난 것입니다.! 번거로우시담 지금도 좋아용
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.
svg를 리액트 컴포넌트로 넣는 방식으로 가겠습니다!
@@ -44,7 +45,13 @@ export default function ProjectCard({ project }: { project: ProjectType }) { | |||
</S.ProjectHeader> | |||
<S.ProjectSummary>{project.summary}</S.ProjectSummary> | |||
</S.ProjectContent> | |||
{/* TODO: 서비스 이용 가능 및 창업 정보와 멤버 정보 추가 필요 */} | |||
<S.ProjectDetail> |
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.
ProjectDetail과 ServiceDetail의 차이는 무엇인가용??
- 뎁스를 줄여 하나로 만들거나,
- 위치상 ProjectDetail을 Footer 등으로 변경하고, ServiceDetail을 ProjectDetail로 변경하여도 좋을 것 같습니다!!!
But.. 사연이 있다면 이대로도 정말 좋아요!
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.
ProjectDetail
이 서비스 이용 가능, 창업 중 + 멤버가 들어가는 부분이고 ServiceDetail
은 서비스 이용 가능, 창업 중 정보가 들어가는 부분인데 네이밍이 너무 어렵네요.. ProjectDetail
은 ProjectFooter
로 바꿀 생각인데 ServiceDetail
도 뭔가 더 적합한 이름이 있을까요?
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.
네이밍이 까다롭네요..! 현재 서비스 상태를 나타낸다는 의미에서 ServiceDetail
-> ProjectServiceStatus
는 어떨까요?
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.
확인했습니다 !!
@@ -44,7 +45,13 @@ export default function ProjectCard({ project }: { project: ProjectType }) { | |||
</S.ProjectHeader> | |||
<S.ProjectSummary>{project.summary}</S.ProjectSummary> | |||
</S.ProjectContent> | |||
{/* TODO: 서비스 이용 가능 및 창업 정보와 멤버 정보 추가 필요 */} | |||
<S.ProjectDetail> |
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.
네이밍이 까다롭네요..! 현재 서비스 상태를 나타낸다는 의미에서 ServiceDetail
-> ProjectServiceStatus
는 어떨까요?
Summary
Screenshot