Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
arthurkimdev
approved these changes
Sep 10, 2024
Comment on lines
+47
to
+50
| <ul class="links"> | ||
| <li><a href="privacy.html">Privacy Policy</a></li> | ||
| <li><a href="faq.html">FAQ</a></li> | ||
| </ul> |
There was a problem hiding this comment.
Suggested change
| <ul class="links"> | |
| <li><a href="privacy.html">Privacy Policy</a></li> | |
| <li><a href="faq.html">FAQ</a></li> | |
| </ul> | |
| <nav aria-label="footer navigation"> | |
| <ul class="links"> | |
| <li><a href="privacy.html">Privacy Policy</a></li> | |
| <li><a href="faq.html">FAQ</a></li> | |
| </ul> | |
| <nav> |
지금도 충분히 시멘틱하게 잘 해주셨지만, 네비게이션의 경우 메뉴를 nav 태그와 접근성을 위한 aria-label 까지 사용해주면 더 좋겠습니다. 웹표준을 따라 작성하면 문서의 구조와 의미를 보다 더 정확하게 검색엔진과 스크린리더에 전달할 수 있으니까요. 😄
Comment on lines
+63
to
+65
| <a href="https://youtube.com/" target="_blank"> | ||
| <img src="img/ico-youtube.png" alt="Youtube"> | ||
| </a> |
There was a problem hiding this comment.
Suggested change
| <a href="https://youtube.com/" target="_blank"> | |
| <img src="img/ico-youtube.png" alt="Youtube"> | |
| </a> | |
| <a href="https://youtube.com/" target="_blank" rel="noopener noreferrer"> | |
| <img src="img/ico-youtube.png" alt="Youtube"> | |
| </a> |
target="_blank" 를 사용하여 새 탭이나 새 창에서 링크를 열 때, 열린 새 페이지는 window.opener 객체를 통해 원래 페이지에 접근할 수 있습니다. 이는 잠재적인 보안 위험으로 아래처럼 rel="noopener"를 사용하여 window.opener 참조를 제거를 통해 원래 페이지에 접근할 수 없도록 설정해야 합니다.
| <meta name="viewport" content="width=device-width, initial-scale=1.0"> | ||
| <title>FAQ | 판다마켓</title> | ||
| <!-- pretendard web-font: https://github.com/orioncactus/pretendard --> | ||
| <link rel="stylesheet" as="style" crossorigin href="https://cdn.jsdelivr.net/gh/orioncactus/pretendard@v1.3.9/dist/web/static/pretendard.min.css"> |
There was a problem hiding this comment.
Suggested change
| <link rel="stylesheet" as="style" crossorigin href="https://cdn.jsdelivr.net/gh/orioncactus/pretendard@v1.3.9/dist/web/static/pretendard.min.css"> | |
| <link rel="stylesheet" as="style" crossorigin href="https://cdn.jsdelivr.net/gh/orioncactus/pretendard@v1.3.9/dist/web/static/pretendard.min.css" media="print" onload="this.media='all'"> | |
웹 폰트를 사용할 때, 성능에 대해서도 고민을 하면 좋습니다. 😸
- media="print"
브라우저가 이 스타일시트를 즉시 로드하지 않습니다. 대신, 페이지의 주요 콘텐츠가 먼저 로드됩니다. 즉, 필요에 따라 지연로딩을 하게 설정할 수 있어요. - onload="this.media='all'
스타일시트가 완전히 로드된 후에 이 JavaScript 코드가 실행됩니다.
이렇게 하는 경우, 초기 페이지 로드 시간 단축할 수 있습니다. (즉, 폰트 스타일시트가 페이지 렌더링을 차단하지 않습니다.)
Comment on lines
+24
to
+35
| </div> | ||
| <!-- /.container --> | ||
| </header> | ||
|
|
||
| <section class="section"> | ||
| <div class="container"> | ||
| <h2>FAQ</h2> | ||
| </div> | ||
| <!-- /.container --> | ||
| </section> | ||
| <!-- /.section --> | ||
|
|
There was a problem hiding this comment.
불필요한 주석은 제거되면 좋겠습니다. 🙃
kiJu2
reviewed
Sep 24, 2024
Comment on lines
+40
to
+42
| <!-- /.container --> | ||
| </div> | ||
| <!-- / bottom-visual --> |
baeggmin
pushed a commit
that referenced
this pull request
Oct 21, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
요구사항
기본
체크리스트 [기본]
1920px이상이면 하늘색 배경색은 너비를 꽉 채우도록 채워지고, 내부 요소들의 위치는 고정되고, 여백만 커지도록 합니다.1920px보다 작아질 때, "판다마켓" 로고의 왼쪽 여백200px"로그인" 버튼의 오른쪽 여백200px이 유지되고, 화면의 너비가 작아질수록 두 요소간 거리가 가까워지도록 설정합니다.cursor: pointer로 설정합니다.체크리스트 [심화]
스크린샷
멘토에게