サイト内検索
Cocoonフォーラム
書き込みの前に以下の3点をご確認ください。
何を書き込んだら良いか分からない場合は、以下のテンプレートをコピペしてご利用ください。
不具合・カスタマイズ対象ページのURL:
相談内容:
不具合の発生手順:
解決のために試したこと:
※文字だけでは正しく伝わらない可能性があるため、スクショ画像の添付もお願いします。
※高速化設定をしている場合は無効にしてください。
環境情報:※↑こちらに「Cocoon設定 → テーマ情報」にある「環境情報」を貼り付けてください。
環境情報の取得方法はこちら。
→ https://wp-cocoon.com/theme-report/
高速化設定を無効にするにはこちら。
→ https://wp-cocoon.com/theme-trouble/
フォーラム利用ガイドリンク
- フォーラムガイドライン
- よくある質問と答え(FAQ)
- サポート対象外のケース
- 原因不明の不具合用トラブルシューティング
- トピックにHTMLを貼り付ける方法(推奨ツール:notepad.pw)
- 真っ白画面でのエラーメッセージの確認方法
- ブラウザ環境チェックツール
- Cocoonカスタマイズ依頼
フォーラム質問後、問題等が解決した場合は結果を書き込んでいただけると幸いです。同様の問題で調べている方には、結果が一番気になる部分となります。
Topic starter
2019年8月18日 15:35
お世話になっております。
tmp/mobile-logo-button.php のファイルですが、imgタグにwidth、heightが入っていないためAMPページでエラーになってしまいました。
//ロゴが存在する場合は画像
if ($logo_url): ?>
<img class="site-logo-image" src="<?php echo esc_url($logo_url); ?>" alt="<?php echo esc_attr(get_bloginfo('name')); ?>">
<?php else: ?>
そこで現在、下記の内容に上書きしてエラー回避できております。
<?php //モバイル用のホームボタン
/**
* Cocoon WordPress Theme
* @author: yhira
* @link: https://wp-cocoon.com/
* @license: http://www.gnu.org/licenses/gpl-2.0.html GPL v2 or later
*/
if ( !defined( 'ABSPATH' ) ) exit;
global $_MENU_CAPTION;
global $_MENU_ICON;
$icon_class = $_MENU_ICON ? $_MENU_ICON : 'logo-menu-icon'; ?>
<!-- ロゴボタン -->
<li class="logo-menu-button menu-button">
<a href="<?php echo esc_url(get_home_url()); ?>" class="menu-button-in">
<?php
$logo_url = get_the_site_logo_url();
//ロゴの幅設定
$site_logo_width = get_the_site_logo_width();
$width_attr = null;
$width_attr = ' width="'.$site_logo_width.'"';
//ロゴの高さ設定
$site_logo_height = get_the_site_logo_height();
$height_attr = null;
$height_attr = ' height="'.$site_logo_height.'"';
//ロゴが存在する場合は画像
if ($logo_url): ?>
<img class="site-logo-image" src="<?php echo esc_url($logo_url); ?>" alt="<?php echo esc_attr(get_bloginfo('name')); ?>" <?php echo $width_attr.$height_attr; ?> >
<?php else: ?>
<?php echo $_MENU_CAPTION ? $_MENU_CAPTION : get_bloginfo('name'); ?>
<?php endif; ?>
</a>
</li>
ご確認いただけたら幸いです。
わいひら reacted
2019年8月18日 19:23
ご連絡ありがとうございます。
一つ確認させてください。
現在のCocoonの仕様では、モバイルヘッダーボタンも、モバイルフッターボタンも以下のようにAMPページでは表示させていません。
https://github.com/yhira/cocoon/blob/0e4898e60c255e8fae66aedea6e95a5e50dcd4d8/tmp/mobile-header-menu-buttons.php#L10
https://github.com/yhira/cocoon/blob/0e4898e60c255e8fae66aedea6e95a5e50dcd4d8/tmp/mobile-footer-menu-buttons.php#L10
現在、AMPページでモバイルボタンが表示されているということでしょうか?
もしよろしければ、そのページのURLをいただけますでしょうか(冒頭コードで修正済みでも大丈夫です)。
Topic starter
2019年8月20日 09:51
お返事ありがとうございます。
申し訳ございません。AMPで使わなければ、問題にならないですよね ?
サイトURL(amp)は https://brainsym.com/?amp=1 です。
こちらのサイトですが、AMPページでもモバイルボタンが表示するように、カスタマイズしてました。
https://gist.github.com/arimichi/6a9f0713a2c2abc2230096f11ec0eb92
AMPでも、モバイルヘッダーボタンとモバイルフッターボタンを表示させておりますが、tmp/mobile-logo-button.php でwidthとheightを表示させるようにしたら、エラーは出ておりません。
なお、モバイルフッターボタンに、情報のリンクを設定すると、AMPでサイドバーが一式非表示になりました。
ご参考までに、当サイトですが、下記のものもカスタマイズしております。
tmpディレクトリ内、
appeal.php(画像を全体に表示させるため)、body-top.php(48行目、フロントページ以外のときクラスを入れるため)、footer-bottom.php(フッターロゴ画像削除)、mobile-navi-button.php(FB,lineの画像リンク挿入)
libディレクトリ内(jQuery Core 3.4.1にするため)
utils.php、page-settings/others-forms.php
わいひら reacted
2019年8月20日 19:53
申し訳ないですが、カスタマイズの影響まで考慮に入れて開発すると、果てがないので、カスタマイズした場合の不具合は、今回のように各自で対応していただくしかないかもしれません。
今回の修正は、AMPページで表示しないので、コードを複雑化しないという意味でも、親テーマでの追加は今のところしないと思います。
現在、ヘッダー・フッターモバイルボタンを高機能化したことにより、全ての機能をAMP対応させるには、結構な開発コストがかかります。
そこまで開発コストをかけてまで、AMPページでモバイルボタンを表示させることが必要かということを考えて、現在は非表示にしています。
元々、AMP自体、記事に素早くアクセスするため発足されたものなので、その目的自体は達しているかなと思い現在の仕様にしています。
https://japan.googleblog.com/2016/02/blog-post_25.html
なべみち reacted
問題の解決に至った場合には、トピック冒頭の「解決済み」をクリックしていただけますと幸いです。
また、有用な回答があった場合は返信右下にある「いいね!」もご活用ください。回答者の励みになります。
(CC BY-ND 2.1)準じていれば(リンクを貼っていただければ)転載も自由です。カスタマイズ記事を書く際にコード等をコピペ利用していただいて構いません。
フォーラムの使い方がよくわからない場合は、テストトピックで自由にテストしていただいて構いません。
最近の書き込みはこちら。
詳細なカスタマイズ依頼をするならこちら。