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カスタマイズ依頼

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

モバイルボタン「目次へ(#toc)」に...
 
共有:
通知
すべてクリア

[解決済] モバイルボタン「目次へ(#toc)」について

9 投稿
2 ユーザー
10 Reactions
2,166 表示
ろこ
(@lococo)
Prominent Member Registered
結合: 7年前
投稿: 830
Topic starter  

モバイルボタンの目次リンクコマンドによるメニューボタンについてです。

不具合というほどではないですが、目次タグが生成されないページで目次リンクのメニューボタンをクリックするとJSエラー(Uncaught TypeError)が出るので、表示しないように修正しておいた方がよいかもしれません。

https://github.com/yhira/cocoon/blob/master/tmp/mobile-toc-button.php#L13


   
edit and わいひら reacted
わいひら
(@yhira)
Illustrious Memberサイト Admin
結合: 7年前
投稿: 17211
 

僕のテスト環境では、本文中に目次のないページでは、ボタンは表示されていないようです。

もしかして、サイトのどこかに、ウィジェットもしくはショートコードで、目次を表示させる設定をしているページでしょうか(Cocoon設定「目次」有効以外で目次表示設定をしているページ)。


   
ろこ
(@lococo)
Prominent Member Registered
結合: 7年前
投稿: 830
Topic starter  

全てのパターンをチェックしたわけではありませんが、Cocoon設定で表示させる目次のみの場合でも設定によってはボタンが表示されてしまうようです。

例えば、設定項目にある「表示条件」で見出しが4つ以上ある場合に表示するようにしたとき、見出しが3つのページでは目次は表示されませんが、目次ボタンは表示されます。

このパターンのみで考えると、少なくとも最初に挙げた判定内の以下の部分

includes_string($content, '<h2')

https://github.com/yhira/cocoon/blob/master/lib/toc.php#L335

に問題があると思います。


   
edit and わいひら reacted
わいひら
(@yhira)
Illustrious Memberサイト Admin
結合: 7年前
投稿: 17211
 

なるほど、そういう条件なんですね。
ありがとうございます。

該当部分を修正してみました。
https://github.com/yhira/cocoon


   
ろこ
(@lococo)
Prominent Member Registered
結合: 7年前
投稿: 830
Topic starter  

https://github.com/yhira/cocoon/commit/d0787a0655c799733cb3aa98f90c202388901899

こちらの修正では不具合に対応できていないようです。

 

まず、前提として

is_singular() && !is_plugin_fourm_page() &&
(
is_toc_visible() &&
is_the_page_toc_visible() &&
(
(is_single() && is_single_toc_visible()) ||
(is_page() && is_page_toc_visible())
)
)

ここまでが表示されたページで目次生成がオンになっているかどうかを判定しています。

上記の判定に加え、目次生成時に生成条件を満たしているかどうかの判定が必要になるかと思います。
つまり、Cocoon設定で言う「表示条件」と「目次表示の深さ」を考慮した判定になります。

&& includes_string($content, '<h2')

こちらだと、「H2タグがひとつでも含まれていれば表示する」という判定になり、考慮すべき設定条件にマッチしていませんでした。
ここに、「目次表示の深さ」に含まれるHタグが「表示条件」以上使用されている判定が必要なのではないでしょうか。


   
edit and わいひら reacted
ろこ
(@lococo)
Prominent Member Registered
結合: 7年前
投稿: 830
Topic starter  

[補足]
1.「目次表示の深さ」に合わせて目次リンク生成とともに数をカウント
https://github.com/yhira/cocoon/blob/master/lib/toc.php#L136

2.カウントした数と「表示条件」で設定した数を比較し、最終的に生成するかどうかを判定
https://github.com/yhira/cocoon/blob/master/lib/toc.php#L182


   
edit and わいひら reacted
わいひら
(@yhira)
Illustrious Memberサイト Admin
結合: 7年前
投稿: 17211
 

すいません。
本日のフォーラム返信の数をこなすため、つい動作確認を怠っていました。
以下のように修正してみました。
https://github.com/yhira/cocoon/commit/de085a9b41b5eed5d9aba2addb452f8b1c22758f
とりあえず、見出しの表示数とモバイルボタンに関する不具合は修正できたように思います。


   
ろこ
(@lococo)
Prominent Member Registered
結合: 7年前
投稿: 830
Topic starter  

確認しました。
「表示条件」と「目次表示の深さ」の組み合わせについては不具合が解消されたようです。

修正ありがとうございました。


   
edit and わいひら reacted
わいひら
(@yhira)
Illustrious Memberサイト Admin
結合: 7年前
投稿: 17211
 

ご確認ありがとうございます。
詳しい原因まで教えていただき感謝申し上げます。


   
共有:

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

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

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

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

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

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

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

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