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

[ Navigation Menu Custom ] Navigation Block Support #1155

Merged
merged 8 commits into from
Jan 9, 2025

Conversation

kurudrive
Copy link
Member

@kurudrive kurudrive commented Jan 8, 2025

チケットへのリンク / 変更の理由(元のissueがあればリンクを貼り付ければOK)

WordPress の wp_nav_menu() や ナビゲーションブロックで出力されるナビゲーションのメニュー項目は、表示中のページに該当するメニュー項目にはカレントクラスが付与される事になっている。
しかしながら、

wp_nav_menu() の場合(既に対応済み)

  • 投稿トップのメニュー項目は、カスタム投稿タイプを表示中にもカレントクラスが付与されてしまう
  • カスタム投稿タイプのトップのメニュー項目があっても、カスタム投稿タイプ関連のページを表示中にカレントクラスが付かない

ナビゲーションブロックの場合(今回のプルリクで対応)

  • 投稿トップのメニュー項目は、投稿関連のページを表示中にもカレントクラスが付与されない
  • カスタム投稿タイプのトップのメニュー項目があっても、カスタム投稿タイプ関連のページを表示中にカレントクラスが付かない

という問題があるため、表示中のページに該当するメニュー項目にカレントクラスを付与する処理を追加する。

ここでいう関連ページとは、

  • 投稿タイプアーカイブ
  • 投稿詳細ページ
  • 投稿タイプに紐づいたカスタムタクソノミー(ターム)アーカイブ
  • 投稿タイプの年月日アーカイブ

を指します

https://www.youtube.com/watch?v=T6iP2g3BYG4

@kurudrive kurudrive merged commit 7aa1cb8 into master Jan 9, 2025
3 checks passed
@kurudrive kurudrive deleted the update/deal-with-active-flag-to-nav-block branch January 9, 2025 12:03
@kurudrive
Copy link
Member Author

[ Navigation Menu Custom ] メニュー項目のURL末尾に / ナシで登録されてる場合に投稿タイプを検出しない不具合修正
#1156

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.

1 participant