Skip to content

refactor: jQuery を削除 #10

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Aug 6, 2025
Merged

refactor: jQuery を削除 #10

merged 1 commit into from
Aug 6, 2025

Conversation

mucho613
Copy link
Collaborator

@mucho613 mucho613 commented Aug 6, 2025 •
edited
Loading

概要

  • 2024年版の実装に含まれていた、古い jQuery のコードを削除しました
  • jQuery の利用場所が 0 になったため、 内での jQuery 読み込みも削除

jQuery の利用箇所

対応する HTML 要素が無い部分もあるため推測も含みます

  • スクロール位置によって、「ページトップへジャンプ」ボタンの出し分け
  • navbar のメニュー制御
  • ページ内スクロールのスムーススクロール化（ただし、コメントアウトしてあった）

@mucho613 mucho613 self-assigned this Aug 6, 2025
@mucho613 mucho613 changed the title update: jQuery を削除 refactor: jQuery を削除 Aug 6, 2025
Fukusan64
Fukusan64 approved these changes Aug 6, 2025
View reviewed changes
Copy link
Collaborator

@Fukusan64 Fukusan64 left a comment

Choose a reason for hiding this comment

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

影響範囲は山村さんの対応範囲に全部入ってそうなので大丈夫かなと思います。

@mucho613 mucho613 merged commit df296c4 into main Aug 6, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Fukusan64 Fukusan64 Fukusan64 approved these changes

@kp54 kp54 Awaiting requested review from kp54

@takusandayooo takusandayooo Awaiting requested review from takusandayooo

Assignees

@mucho613 mucho613

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mucho613 @Fukusan64