Skip to content

Conversation

@doyou1
Copy link
Contributor

@doyou1 doyou1 commented Apr 24, 2024

Issues 번호 :

Closes #64

변경, 추가된 코드(설명 등)

  • static 폰트인 Pretendard-Regular는 삭제하고 font-weight로 제어
  • 다 굵게 했는데, 적절한지 확인해주세욤

코드 주의점

스크린샷

image

Summary by CodeRabbit

- Style: "Pretendard-Regular" 폰트를 "Pretendard"로 대체하고 굵기를 700으로 설정했습니다.

@doyou1 doyou1 requested a review from Jung-eunwoo April 24, 2024 06:47
@doyou1 doyou1 self-assigned this Apr 24, 2024
@github-actions
Copy link

github-actions bot commented Apr 24, 2024

Image description CodeRabbit

개요:

전체적으로 "Pretendard-Regular" 폰트를 "Pretendard"로 대체하고 굵기를 700으로 설정했습니다.

상세 정보:

파일 변경 내용
src/assets/typography.scss "Pretendard-Regular" 폰트를 "Pretendard"로 대체하고 굵기를 700으로 설정함.

Uplevel your code reviews with CodeRabbit Pro

CodeRabbit Pro

If you like this project, please support us by purchasing the Pro version. The Pro version has advanced context, superior noise reduction and several proprietary improvements compared to the open source version. Moreover, CodeRabbit Pro is free for open source projects.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Image description CodeRabbit

Commits Files that changed from the base of the PR and between 2fd4909 and 5b0fc8c commits.
Files selected (1)
  • src/assets/typography.scss (1)
Files ignored due to filter (5)
  • src/assets/fonts/Pretendard-Regular.eot
  • src/assets/fonts/Pretendard-Regular.svg
  • src/assets/fonts/Pretendard-Regular.ttf
  • src/assets/fonts/Pretendard-Regular.woff
  • src/assets/fonts/Pretendard-Regular.woff2
Review comments generated (2)
  • Review: 1
  • LGTM: 1

Tips

Chat with Image description CodeRabbit Bot (@coderabbitai)

  • Reply on review comments left by this bot to ask follow-up questions. A review comment is a comment on a diff or a file.
  • Invite the bot into a review comment chain by tagging @coderabbitai in a reply.

Code suggestions

  • The bot may make code suggestions, but please review them carefully before committing since the line number ranges may be misaligned.
  • You can edit the comment made by the bot and manually tweak the suggestion if it is slightly off.

Pausing incremental reviews

  • Add @coderabbitai: ignore anywhere in the PR description to pause further reviews from the bot.

Comment on lines 1 to 9
body * {
font-family: "Pretendard-Regular";
--el-font-family:"Pretendard-Regular";
@font-face {
font-family: "Pretendard-Regular";
src: url("./fonts/Pretendard-Regular.ttf") format("truetype");
src: url("./fonts/Pretendard-Regular.eot"); /* IE9 Compat Modes */
src: url("./fonts/Pretendard-Regular.eot?#iefix") format("embedded-opentype"), /* IE6-IE8 */
url("./fonts/Pretendard-Regular.woff2") format("woff2"), /* Super Modern Browsers */
url("./fonts/Pretendard-Regular.woff") format("woff"), /* Pretty Modern Browsers */
url("./fonts/Pretendard-Regular.svg#svgFontName") format("svg"); /* Legacy iOS */
}
}
font-family: "Pretendard";
font-style: normal;
font-weight: 700;
--el-font-family:"Pretendard";
}

/** 새로 추가된 폰트(!important는 element-plus component에도 적용하기 위해서) */
.cp-text-head-1 {

Choose a reason for hiding this comment

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

Image description CodeRabbit

Note: This review was outside of the patch, but no patch was found that overlapped with it. Original lines [22-22]

폰트 굵기를 700으로 변경한 것은 올바른 조치입니다.

-    font-family: "Pretendard";
+    font-family: "Pretendard-Regular";

@Jung-eunwoo
Copy link
Contributor

image

이 부분에서 흰글자가 기존 굵기로 얄아질 수 있을까요??

그 부분 외에는 완벽합니다!!! bbb

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.

design init 페이지 글자 굵기가 얇아짐

3 participants