Cocoonフォーラム

サイト内検索
書き込みの前に以下の3点をご確認ください。
  1. 1つのトピックにつき1つの質問を書き込んでください
  2. 不具合・カスタマイズ対象ページのURLを提示高速化を無効にしてください
  3. 該当部分のキャプチャ・環境情報とともに書き込んでいただけると助かります

何を書き込んだら良いか分からない場合は、以下のテンプレートをコピペしてご利用ください。

不具合・カスタマイズ対象ページのURL:

相談内容:

不具合の発生手順:

解決のために試したこと:

※文字だけでは正しく伝わらない可能性があるため、スクショ画像の添付もお願いします。
※高速化設定をしている場合は無効にしてください。
環境情報:

※↑こちらに「Cocoon設定 → テーマ情報」にある「環境情報」を貼り付けてください。

環境情報の取得方法はこちら。
https://wp-cocoon.com/theme-report/
高速化設定を無効にするにはこちら。
https://wp-cocoon.com/theme-trouble/

フォーラム利用ガイドリンク

  1. フォーラムガイドライン
  2. よくある質問と答え(FAQ)
  3. サポート対象外のケース
  4. 原因不明の不具合用トラブルシューティング
  5. トピックにHTMLを貼り付ける方法(推奨ツール:notepad.pw
  6. 真っ白画面でのエラーメッセージの確認方法
  7. ブラウザ環境チェックツール
  8. Cocoonカスタマイズ依頼

フォーラム質問後、問題等が解決した場合は結果を書き込んでいただけると幸いです。同様の問題で調べている方には、結果が一番気になる部分となります。

このフォーラムには初心者の方も多く参加されています。質問に不慣れな方も安心して相談できるよう、できるだけ穏やかな言葉でご案内いただけますと幸いです。

id属性に日本語メニュー名が反映される...
 
通知
すべてクリア

[解決済] id属性に日本語メニュー名が反映されることについて

11 投稿
3 ユーザー
7 Reactions
277 表示
大門未知子
(@chu-ya)
Illustrious Member Registered
結合: 4年前
投稿: 4374
トピックスターター  

----------------------------------------------
テーマ名:Cocoon
バージョン:2.9.0.12
カテゴリー数:18
タグ数:123
投稿数:1747
固定ページ数:12
ユーザー数:1
----------------------------------------------

メニュー生成時、wp_nav_menu によってメニュー名がそのまま id 属性として自動的に出力される。
メニュー名に日本語を使用した場合、2バイト文字はエンコードされた形で id 属性に反映される。

HTML5の仕様では、id や class 属性に日本語を使用すること自体に明確な制限はない。
しかし、意図しない形式の id 属性が出力される点については、望ましいとは言い難い。

そのため、不要な id 属性は出力しないよう制御するのが適切なのでは?

該当

  • navi.php
  • navi-footer.php
  • mobile-navi-button.php
  • mobile-header-custom-navi-buttons.php
  • mobile-footer-custom-navi-buttons.php

●改善

preg_replaceでid属性を削除しているが、
wp_nav_menuのパラメータで除去可能

'items_wrap' => '<ul class="%2$s">%3$s</ul>',

%1$s → ID(使わなければ出力されない)
%2$s → class
%3$s → <li> の中身

https://github.com/xserver-inc/cocoon/blob/68f4a7698e38ccbe611bbb66ced3d4ed84e9255b/tmp/mobile-navi-button.php#L58



エックスサーバー 高澤
(@s_takazawa)
Reputable Member Moderator
結合: 2年前
投稿: 194
 

@chu-ya さん

上記ご指摘いただき、ありがとうございます。

現時点で動作確認できておらず取り急ぎのご返信で恐縮ですが、ID属性の件ということで、内容のほうご確認させていただきます。



   
大門未知子
(@chu-ya)
Illustrious Member Registered
結合: 4年前
投稿: 4374
トピックスターター  

@s_takazawa さん

バグではないので、エックスサーバー社のコーディングルールとしてid属性の「日本語」どう考えるか?
という話です。

 


わいひら
(@yhira)
Illustrious Memberサイト Admin
結合: 8年前
投稿: 18449
 

@s_takazawa さん

お世話になっております。
体調も回復したので、確認がまだでしたらこちらの方で対応させていただきます。
よろしくお願いいたします。



エックスサーバー 高澤
(@s_takazawa)
Reputable Member Moderator
結合: 2年前
投稿: 194
 

@yhira

わいひらさん

お世話になっております。
体調が良くなられたとのことで、本当に良かったです!

本件に関して、承知いたしました。
お忙しいところ大変恐縮ですが、ご確認のほどよろしくお願いいたします。

引き続きよろしくお願いいたします。



   
エックスサーバー 高澤
(@s_takazawa)
Reputable Member Moderator
結合: 2年前
投稿: 194
 

@chu-ya さん

お世話になっております。

本件のコーディングルール等につきましては、わいひらさんのほうでご確認いただく運びとなりました。
ご承知おきいただけますと幸甚です。

引き続き、どうぞよろしくお願い申し上げます。



   
大門未知子
(@chu-ya)
Illustrious Member Registered
結合: 4年前
投稿: 4374
トピックスターター  

@yhira さん

余程のことがないとまず、該当メニューのid属性を参照することはないと思います。


●余談

アクセスログの参照先を見ると、メニュー名に「日本語」を使っている方が一定数います。
このため、スキンで、メニューにメモ欄を設けることを検討しており。

 



   
わいひら reacted
わいひら
(@yhira)
Illustrious Memberサイト Admin
結合: 8年前
投稿: 18449
 

投稿者:: @chu-ya

余程のことがないとまず、該当メニューのid属性を参照することはないと思います。

確かにああいったID名のCSSセレクターは実際僕も使いたくないです。
該当部分を修正させていただきました。
https://github.com/xserver-inc/cocoon/commit/fd0f1ef0f4d5de439cf7f10a7c22fc1c2a232700



   
大門未知子
(@chu-ya)
Illustrious Member Registered
結合: 4年前
投稿: 4374
トピックスターター  

@yhira さん

----------------------------------------------
テーマ名:Cocoon
バージョン:2.9.0.13
カテゴリー数:18
タグ数:124
投稿数:1749
固定ページ数:12
ユーザー数:1

----------------------------------------------

修正が漏れています。
既にメニューリストulからid属性は除去されているので、以下の部分のpreg_replaceは不要です。

preg_replaceでid属性を削除しているが、wp_nav_menuのパラメータで除去可能。

https://github.com/xserver-inc/cocoon/blob/fd0f1ef0f4d5de439cf7f10a7c22fc1c2a232700/tmp/mobile-navi-button.php#L60-L85



   
わいひら reacted
大門未知子
(@chu-ya)
Illustrious Member Registered
結合: 4年前
投稿: 4374
トピックスターター  

----------------------------------------------
テーマ名:Cocoon
バージョン:2.9.0.13
カテゴリー数:18
タグ数:124
投稿数:1748
固定ページ数:12
ユーザー数:1
----------------------------------------------
wp_nav_menuで作成されるメニューのULからid属性が削除されていることを確認しました。
本件をクローズとします。



   
わいひら reacted
わいひら
(@yhira)
Illustrious Memberサイト Admin
結合: 8年前
投稿: 18449
 

あ、すでにご確認いただいていたんですね。
ご確認いただきありがとうございます。



   
共有:

問題の解決に至った場合には、トピック冒頭の「解決済み」をクリックしていただけますと幸いです。

また、有用な回答があった場合は返信右下にある「いいね!」もご活用ください。回答者の励みになります。

「いいね!」機能はフォーラム登録者のみが利用できる機能です。

CC BY-ND 2.1)準じていれば(リンクを貼っていただければ)転載も自由です。カスタマイズ記事を書く際にコード等をコピペ利用していただいて構いません。

フォーラムの使い方がよくわからない場合は、テストトピックで自由にテストしていただいて構いません。

最近の書き込みはこちら。

詳細なカスタマイズ依頼をするならこちら。

タイトルとURLをコピーしました