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

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

AMP ページでの class 属性の...
 
共有:
通知
すべてクリア

[解決済] AMP ページでの class 属性の重複

6 投稿
2 ユーザー
5 Reactions
2,395 表示
(@akira)
Famed Member Registered
結合: 7年前
投稿: 1657
Topic starter  

AMP ページにて、同じタグ内で class 属性が重複しています。

Cocoon 公式サイトのどのページでも、W3C の HTML 検証チェッカーで重複エラー(Duplicate attribute class)を確認できます。

おそらく画像周りでしか重複エラーは出ていないと思いますので、↓ の $class_attr が 2 つあるのが原因かもしれません。

https://github.com/yhira/cocoon/blob/master/lib/amp.php#L371


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

ご連絡ありがとうございます。
どうやら、前回「AMP画像にID付加の要望」があったときに、classが後ろのほうに既にあるのに気づかず「classがないじゃないか!」と思ってしまって重複追加してしまったみたいです。
https://github.com/yhira/cocoon/commit/19c04cda0c5b421071e719e82272dae895de9b0f#diff-6ca2ddffd0691f770167fc4038707ae9
後方のclassを1つ削除しておきました。
------------------------------------------------
https://github.com/yhira/cocoon
最新ファイルをダウンロードする場合は、上記ページのダウンロードボタンからzipファイルをダウンロードしてください。

FTPでのアップデート方法はこちら。
https://wp-cocoon.com/ftp-update/
------------------------------------------------
確かに、トピックの報告の画像にもclassが2つ表示されています。

完全なポカです ? ? 

教えていただき、助かりました。


   
Akira reacted
(@akira)
Famed Member Registered
結合: 7年前
投稿: 1657
Topic starter  

ありがとうございます。 class 属性の重複の解消を確認しました。

ただ、申し訳ありません。私の見落としがありました。

AMP ページの検索フォーム(検索ウィジェット)の form タグで target="_top" も重複していました。

おそらく原因は、まず searchform.php で target="_top" の指定があります。

https://github.com/yhira/cocoon/blob/master/searchform.php#L13

その上で、lib / amp.php で target="_top" を追加されているからかなと思います。

https://github.com/yhira/cocoon/blob/4f258447acc8c25c44c077d08c44e63801ad36ab/lib/amp.php#L225


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

その部分も確かに…。
詳細は覚えていないですがおそらく僕が、何かの拍子に追加してしまったんだと思います。
テンプレート部分の target="_top"を削除しておきました。
https://github.com/yhira/cocoon/blob/cec9274e386a0fa97804b908ea1067880c96c291/searchform.php#L13
更新ファイル。
https://github.com/yhira/cocoon/


   
Akira reacted
(@akira)
Famed Member Registered
結合: 7年前
投稿: 1657
Topic starter  

ありがとうございます。お手数をおかけしました。


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

ご確認ありがとうございます!


   
共有:

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

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

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

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

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

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

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

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