Skip to content

Conversation

@saseungmin
Copy link
Collaborator

@saseungmin saseungmin commented Nov 21, 2020

🚀 진행 사항

  • 스터디 소개 페이지 꾸미기

11

- Improve css style for study introduce page
Comment on lines +94 to +119
<IntroduceHeaderWrapper>
<h1>{title}</h1>
<button type="button">신청하기</button>
</IntroduceHeaderWrapper>
<ModeratorWrapper>
{`🙋‍♂️ ${moderatorId}`}
</ModeratorWrapper>
<IntroduceReferenceWrapper>
<IntroduceReference>
<label htmlFor="application-status">신청 현황 :</label>
<span id="application-status">
{`${participants.length} / ${personnel}`}
</span>
</IntroduceReference>
<IntroduceReference>
<label htmlFor="apply-end">접수 마감 일자 :</label>
<span id="apply-end">{applyEndDate}</span>
</IntroduceReference>
</IntroduceReferenceWrapper>
<IntroduceContentTitle>
소개
</IntroduceContentTitle>
{/* TODO: dangerouslySetInnerHTML으로 변경하기 */}
<IntroduceContent>
{contents}
</IntroduceContent>
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

  • Refactoring이 필요할 듯 하다.
  • 컴포넌트 분리하기.

Comment on lines +24 to +38
button {
display: inline-flex;
align-items: center;
margin: .5rem 0 .5rem 0;
padding: 0.25rem 5rem;
font-size: 1.5em;
line-height: 0;
font-family: 'Gamja Flower', cursive;
border-radius: 0.4rem;
border: none;
background: ${palette.teal[5]};
color: white;
&:hover{
background: ${palette.teal[4]};
}
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

  • 버튼은 공통으로 분리하기 (TODO)

@saseungmin saseungmin added the Improve 기능 개선 및 추가 사항 label Nov 21, 2020
@saseungmin saseungmin mentioned this pull request Nov 21, 2020
11 tasks
@saseungmin saseungmin merged commit 93c5001 into CodeSoom:main Nov 22, 2020
@saseungmin saseungmin deleted the simple-decorate-study-introduce branch July 5, 2021 19:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Improve 기능 개선 및 추가 사항

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant