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

Heading 컴포넌트 #30

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Heading 컴포넌트 #30

wants to merge 4 commits into from

Conversation

DaleSeo
Copy link
Contributor

@DaleSeo DaleSeo commented Jan 7, 2025

Heading 컴포넌트를 구현하였습니다.

Shot 2025-01-09 at 20 02 01

@DaleSeo DaleSeo linked an issue Jan 7, 2025 that may be closed by this pull request
@DaleSeo DaleSeo marked this pull request as ready for review January 9, 2025 02:14
@DaleSeo DaleSeo requested a review from a team as a code owner January 9, 2025 02:14
@DaleSeo DaleSeo marked this pull request as draft January 9, 2025 02:58
/** 굵기 */
weight?: FontWeight;
/** 명암비 */
muted?: boolean;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

원래 contrast: "low" | "high" 였는데 API를 살짝 바꾸었습니다. 좀 더 사용자 친화적일 것이라 생각합니다.

contrast: {
DEFAULT: {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

명암비가 높은 글자색을 훨씬 더 많이 사용할 것 같아서, 디폴트 글자색을 바꾸었습니다.

@DaleSeo DaleSeo marked this pull request as ready for review January 10, 2025 01:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Text, Heading Component
1 participant