「メイド・イン・ヘブン」スキン適用中

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ロゴ(AMPロゴをSmartNe...
 
共有:
通知
すべてクリア

[解決済] AMPロゴ(AMPロゴをSmartNewsのロゴに変更する)

11 投稿
2 ユーザー
1 Reactions
2,266 表示
kodaira
(@kodaira)
Eminent Member Registered
結合: 5年前
投稿: 29
Topic starter  

お世話になっています。

内容は、下記ページの関連になりますが「解決済」として閉じられていたので、新トピックを立てさせていただきます。

ampのときのサイトロゴ表示について

 

この最後の部分で

 

AMPロゴが反映されなかった部分は少し気になりますが、正常表示されていますので、いったんこのままにしておこうと思います。

 

たしかに、AMPロゴはいつだったか?ずっと前から反映されないですね^^

ただ不具合なく運営できてたのと、書き込みもなかったのでそのままにしてました 

 

で、トピックが終わっていますが、

 

AMPロゴは、このまま反映されないのでしょうか?

 

 

具体的に書きます。

SmartNewsに申請中ですが、ロゴの色が背景と同化している(SmartNewsのロゴには背景色が適用されず白なので)と指摘されています。「AMP」で設定しているロゴであれば、SmartNewsの水準を満たすのですが、一般のロゴの方が表示されてしまいます。

 

ぺーじはこちら

https://seishonen.or.jp/honbu/

「AMP」で設定しているロゴはこちらです。

seishonen.or.jp/honbu/wp-content/uploads/2019/08/amplogo.png

 

AMPBenchで確認すると、「AMP」で設定しているロゴのほうが選ばれています。

(別添イメージの通り)

 

しかし、実際のAMPページでは「AMP」で設定しているロゴは表示されません。

https://seishonen.or.jp/honbu/?amp=1

 

よろしくお願いします。

 

 

 

 

This topic was modified 5年前 2回 by kodaira

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

しかし、実際のAMPページでは「AMP」で設定しているロゴは表示されません。

https://seishonen.or.jp/honbu/?amp=1

AMPロゴの機能は、設定ページで説明されている通りです。

Google検索結果に表示されるAMP用のロゴ画像を設定します。

AMPページのロゴに反映されるものではないですよ。
Google検索結果のパブリッシャーロゴとして表示されるものです。
そもそも、添付画像のように検索結果でカルーセル表示されるものでないと表示されませんよ。


   
kodaira
(@kodaira)
Eminent Member Registered
結合: 5年前
投稿: 29
Topic starter  

わいひら様

お世話になっています。

 

意味を間違っていたようで、失礼しました。

 

 

どこかのPHPの<channel> </channel>の間に、

 

<snf:logo>URL</snf:logo>

 

を記載する、みたいなことなのでしょうか。

 

もう少し勉強してみます。


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

そういったタグではなく、JSON-LDで構造化データとして出力しています。
https://github.com/yhira/cocoon/blob/2397451a07f13f4025baaf7a6172553fefb59eca/tmp/json-ld.php#L99


   
kodaira
(@kodaira)
Eminent Member Registered
結合: 5年前
投稿: 29
Topic starter  

わいひら様

 

お世話になっています。

もう少し教えてください。

 

上で教えていただいたことは

 

(1)cocoonの子テーマに「tmp」フォルダを作り、json-ld.phpをコピーして入れる。

(2)json-ld.phpのロゴ画像の部分(75行目もしくは101行目)の部分を、smartnews用に使いたいロゴのURLに書き換える。

 

という対応でいいのかな、と思いましたが、反映されず、違っていたようです。

 

修正記載位置が違っていたという単純な間違いでしょうか?

それとも、もっと、根本的に、複雑な対応が必要なのでしょうか?

 

よろしくお願いします。


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

JSON-LDを変更しても、SmartNewsには影響しないと思います(※SmartNewsの仕様を熟知しているわけではないので詳しいことはわかりません)。
SmartNewsに対応させるのであれば、/tmp/smartnews.phpをカスタマイズする必要があるかと思います。
https://github.com/yhira/cocoon/blob/8b4dc565a0cab7e350f0c3e5b289e7c762173102/tmp/smartnews.php


   
kodaira
(@kodaira)
Eminent Member Registered
結合: 5年前
投稿: 29
Topic starter  

わいひら様

お世話になっています。

 

結局、smartnews.phpの63行目あたり、

最初は、一般ロゴを読みに行くところ($logo_url = get_the_site_logo_url())を、

最初からampロゴを読みに行くように変えました($logo_url = get_amp_logo_image_url())。

 

その結果、smartnewsの審査担当者からもOKが出ました。

 

とりあえず解決ですが、

ここで質問させてください。

ampロゴもsmartnews用ロゴも、それぞれ基準が厳しいですが、

ampロゴとsmartnewsロゴでは、サイズの縦横比さえ気をつければ、

デザイン的に求められている基準は似通っています。

Cocoonの仕様では、最初は一般ロゴを見に行くようになっていますが、

これをsmartnewsを意識するものにすると、透明地(つまり白系)に濃色にせざるを得ず、一般ロゴ自体が、かなり制約を受けてしまいます。

最初から、ampロゴ→smartnews、では、いけないのでしょうか?

 

 

 


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

今のところ変更は考えていません。
推奨サイズはありますが、バリデーターで指摘されないことから、絶対指定すべきものではないからです。
https://publishers.smartnews.com/hc/ja/articles/360010977813
また、AMPロゴとて完全にSmartNewsの推奨サイズではないからです。
また、AMPロゴは指定しない人も多いという理由もあります。
それだったら、実際にサイトのロゴを表すものを指定しておいた方が良いと思います。
AMPロゴにしたとて、AMPロゴが制約を受けてしまうことには変わりがないので。

完全に、対応させるのであれば、設定項目を追加するしかありません。
ただ今のところ、そこまでの需要はないように感じるので、とりあえずは以下のようにフックを追加することで対応できるようにしておきました。

//SmartNewsのロゴ変更
//「横700px以下 x 縦およそ100px」の透過PNGファイル推奨
add_filter('smartnews_logo_url', function ($logo_url){
  return 'ここにSmartNewsロゴ用のURLを入力';
}); 

今後、要望があるようであれば、またその都度検討したいと思います。


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

試す場合は、GitHubのものでアップデートする必要があります。

------------------------------------------------
https://github.com/yhira/cocoon
最新ファイルをダウンロードする場合は、上記ページのダウンロードボタンからzipファイルをダウンロードしてください。

FTPでのアップデート方法はこちら。
https://wp-cocoon.com/ftp-update/
------------------------------------------------


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

こんな感じでフィルターフックを追加しました。
https://github.com/yhira/cocoon/commit/8e1d292b90a64a9ed9350bc295d9d3249e69dfee


   
kodaira
(@kodaira)
Eminent Member Registered
結合: 5年前
投稿: 29
Topic starter  

わいひら様

お世話になっています。

上の方法で、子テーマのfunction.phpに追記をし、

Smartnews用のロゴを表示させることができました。

ありがとうございます。


   
わいひら reacted
共有:

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

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

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

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

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

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

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

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