Skip to content
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

Merged
merged 3 commits into from
Nov 6, 2023

Conversation

solar3070
Copy link
Member

@solar3070 solar3070 self-assigned this Nov 3, 2023
@solar3070 solar3070 changed the title 프로젝트 카드 서비스 이용 가능 및 창업중 정보 보여주기 [SP2] 프로젝트 카드 서비스 이용 가능 및 창업중 정보 보여주기 Nov 3, 2023
Copy link
Member

@SeojinSeojin SeojinSeojin left a 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} />
Copy link
Member

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를 비우지 않을 방법을 생각하다가 생각난 것입니다.! 번거로우시담 지금도 좋아용

Copy link
Member Author

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>
Copy link
Member

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.. 사연이 있다면 이대로도 정말 좋아요!

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ProjectDetail이 서비스 이용 가능, 창업 중 + 멤버가 들어가는 부분이고 ServiceDetail은 서비스 이용 가능, 창업 중 정보가 들어가는 부분인데 네이밍이 너무 어렵네요.. ProjectDetailProjectFooter로 바꿀 생각인데 ServiceDetail도 뭔가 더 적합한 이름이 있을까요?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

네이밍이 까다롭네요..! 현재 서비스 상태를 나타낸다는 의미에서 ServiceDetail -> ProjectServiceStatus 는 어떨까요?

@solar3070 solar3070 changed the title [SP2] 프로젝트 카드 서비스 이용 가능 및 창업중 정보 보여주기 [SP2] 프로젝트 서비스 이용 가능 및 창업중 정보 보여주기 Nov 4, 2023
Copy link
Contributor

@f0rever0 f0rever0 left a 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>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

네이밍이 까다롭네요..! 현재 서비스 상태를 나타낸다는 의미에서 ServiceDetail -> ProjectServiceStatus 는 어떨까요?

@solar3070 solar3070 merged commit 81cca79 into develop Nov 6, 2023
1 check passed
@solar3070 solar3070 deleted the feat#253-project-detail-info branch November 6, 2023 07:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

프로젝트 서비스 이용 가능 및 창업중 정보 보여주기
3 participants