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

feat: add global css #30

Merged
merged 3 commits into from
Jun 11, 2024
Merged

feat: add global css #30

merged 3 commits into from
Jun 11, 2024

Conversation

sounmind
Copy link
Member

@sounmind sounmind commented Jun 11, 2024

  • close Create Global CSS #10

  • add global-styles.css file

    • a tag
    • font family
    • scroll behavior
    • box sizing
  • fix html wrong structure

    • remove duplicated body and head tags
    • format little code styles

- remove duplicated body and head tags
- format little code styles
@sounmind sounmind requested a review from a team as a code owner June 11, 2024 01:39
global-styles.css Outdated Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
Comment on lines +22 to -19
href="//spoqa.github.io/spoqa-han-sans/css/SpoqaHanSansNeo.css"
rel="stylesheet"
href="https://cdn.jsdelivr.net/gh/spoqa/spoqa-han-sans/css/SpoqaHanSansNeo.css"
Copy link
Contributor

Choose a reason for hiding this comment

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

웹폰트 내려받는 URL은 어떤 연유로 바꾸게 되는지 궁금하네용.

Copy link
Member Author

Choose a reason for hiding this comment

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

공식 홈페이지에서 권유하는 대로 설정해서 더 신뢰있다고 판단했습니당

Copy link
Contributor

Choose a reason for hiding this comment

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

처음에 저 jsDelivr URL은 어떻게 발견하신 거에요? 그냥 궁금해서 ㅎㅎ

Copy link
Member Author

Choose a reason for hiding this comment

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

@SamTheKorean 이 질문에 답해주실 수 있나요?! 제가 추가한 게 아니라서 ...

Copy link
Contributor

Choose a reason for hiding this comment

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

보통 공식홈페이지에서 가져오는데 그게 아니라면 chatgpt를 의심해봅니다..! 더 신중히 채크하도록 하겠습니다!

- set styles of
  - a tag
  - font family
  - scroll behavior
  - box sizing
@sounmind sounmind force-pushed the evan/add-global-styles branch from 8baa761 to 5ac7a6f Compare June 11, 2024 03:08
@sounmind sounmind requested a review from DaleSeo June 11, 2024 14:53
@sounmind sounmind merged commit e26853c into main Jun 11, 2024
@sounmind sounmind deleted the evan/add-global-styles branch June 11, 2024 22:37
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.

Create Global CSS
3 participants