-
Notifications
You must be signed in to change notification settings - Fork 5
[운영] SEO 및 성능 개선 #148
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
[운영] SEO 및 성능 개선 #148
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
이 PR은 SEO 및 성능 개선을 위해 메타 태그 추가, 불필요한 CDN 제거, script 태그에 async/defer 속성 추가, 사용하지 않는 코드 제거, 그리고 href 속성이 없는 a 태그를 button 태그로 변경하는 작업을 수행합니다.
- 메타 태그 추가와 스크립트 비동기 처리로 페이지 로딩 최적화
- a 태그를 button 태그로 변경하여 웹 크롤러 접근성 개선
- 불필요한 CDN 및 사용하지 않는 CSS 코드 제거
Reviewed Changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| tests/table-contents.test.ts | 테스트 케이스를 a 태그에서 button 태그로 업데이트 |
| src/styles/style.css | 전역 및 Table 관련 기본 스타일를 재구성 및 불필요한 코드 제거 |
| src/scripts/table-contents.ts | 링크 요소를 a 에서 button 으로 변경하고 관련 클래스 추가 |
| src/scripts/breadcrumb.ts | Breadcrumb 관련 인터페이스 및 번역 데이터 업데이트 |
| src/data/breadcrumb.json | Breadcrumb 데이터 구조를 기존 객체에서 세그먼트 기반으로 확장 |
| index.html | 메타 태그 추가, 스크립트 속성 조정 및 접근성 속성 보완 |
| .vscode/settings.json | CSS lint 경고 무시 설정 추가 |
Comments suppressed due to low confidence (2)
src/scripts/table-contents.ts:47
- [nitpick] 변수명 'link'는 HTML 버튼 요소를 생성하는 데 사용되므로, 'button'과 같이 더 명확한 이름으로 변경하는 것을 고려해보세요.
const link = document.createElement('button');
src/styles/style.css:51
- 전역 스타일 관련 코드가 제거되었습니다. 이 변경이 디자인 가이드와 일관되는지 재검토하여 의도치 않은 UI 변경이 없도록 확인해주세요.
}
close #139
close #140
close #142
close #143
close #144