AMPのカエレバ・ヨメレバ

PR 2016/10/19

このトピックには7件の返信、2人の参加者があり、最後にわいひらにより8年、 4ヶ月前に更新されました。

7件の返信スレッドを表示中

投稿者

投稿
- 2016年10月16日 8:04 AM #46232
  
  やーまん
  ゲスト
  
  先日はご対応ありがとうございました。
  またAMPネタで恐縮です。
  
  １：カエレバ・ヨメレバ側の不具合と思いますが、
  当方の環境ですと7netへのリンクを作成した際、style属性が取り除かれない現象になりました。
  7netへのリンクのhtmlタグ内のみ空白が空白では無く、文字コード「C2A0」が入っているようです。
  style属性を取り除くコードを実行する前に下記コードを入れて解決いたしました。
  $the_content = str_replace( "\xc2\xa0", " ", $the_content );
  
  ２：imgタグに一括でlayout=”responsive”を適応されていると思います。
  1×1サイズのアフィリエイト側へのアクセス解析等に使われる
  表示に意味の無い透明画像にも適応され
  空白が沢山の間延びした形になります。
  1×1サイズの画像のみlayout=”fixed”にするのはどうでしょうか？
  
  ３：カエレバの商品画像にwidthとhightを追加するコードがありますが、
  amazonのみとなっています。
  楽天とYahooショッピングの画像にもwidthとhightを追加して頂けないでしょうか？
  楽天は
  http://thumbnail.image.rakuten.co.jp/
  ヤフーは
  http://item.shopping.c.yimg.jp/
  が商品画像のURLとなっています。
  
  以上沢山ですがよろしくお願いします。
- 2016年10月16日 4:53 PM #46249
  わいひら
  キーマスター
  1.よかったら、どういった状態になるのかを見たいので、不具合の出るコードを貼り付けていただいてよろしいでしょうか（アフィリエイトIDなどはxxxxxなどにしていただいて）。
  
  2.これに関しては、lib/amp.phpの以下の部分を
```
  //画像タグをAMP用に置換
  $the_content = preg_replace('/<img/i', '<amp-img layout="responsive"', $the_content);
```
  以下のように変更するとどうでしょうか？
```
  //画像タグをAMP用に置換
  $the_content = preg_replace('/<img/i', '<amp-img', $the_content);
```
  僕の環境だと、layout=”responsive”がなくとも、問題なくレスポンシブ表示されているようでした。
  
  ３.ちょっとこれから対応させてみようと思います。
- 2016年10月16日 5:10 PM #46252
  
  わいひら
  キーマスター
  
  3.手元のファイルは、修正して楽天とYahoo!ショッピングに対応できたと思います。
  次のバージョンで改善されると思います。
  あと、楽天とYahoo!のURLがあったおかげで、すぐに作業を終えることができました。ありがとうございます！
- 2016年10月17日 9:25 PM #46312
  
  やーまん
  ゲスト
  
  早速の対応ありがとうございます。
  m(_ _)m
  
  1の問題箇所のソースを貼り付けます。
  もしもアフィリエイト経由のため1X1サイズの画像が挿入される様です。
  アマゾン等他のショップは問題ありませんが、7netに入る画像のみエラーとなります。
  普通の空白に見えますが、imgタグ内の空白は空白では無いようです。
  
  <a href="http://c.af.moshimo.com/af/c/click?a_id=xxxxx&p_id=xxxxx&pc_id=xxxxxx&pl_id=xxxxxxxxxx&url=http%3A%2F%2F7net.omni7.jp%2Fsearch%2F%3Fkeyword%3D%25E3%2583%259D%25E3%2583%2586%25E3%2583%2588%25E3%2583%2581%25E3%2583%2583%25E3%2583%2597%25E3%2582%25B9%26searchKeywordFlg%3D1" target="_blank" ><img src="http://i.af.moshimo.com/af/i/impression?a_id=xxxxx&p_id=xxxxx&pc_id=xxxxxxx&pl_id=xxxxxxx" width="1" height="1" style="border:none;">7net</a>
  
  http://www.saka-en.com/java/c2a0/
  上記サイトでこのスペースのことを書かれています。
  
  Choromeからコピペするとこの問題が発生するようです。
  試しにFireFoxからコピペすると普通の半角スペースとして認識されます。
- 2016年10月17日 9:32 PM #46315
  
  やーまん
  ゲスト
  
  ↑のコードは掲示板に貼り付けると普通に問題ありませんね・・・
  カエレバよりChrome経由で記事にコピペするとアウトです。
  
  2に関してはバッチリでした。
  layout属性は必須では無かったのですね。
  勉強になりました^^;
  グーグル先生のエラーが無くなりました。
  ちなみに変更前は↓こんな感じでした。
  
  3もご対応ありがとうございます。
  次回更新を楽しみにしています。
  ＼(^o^)／
- 2016年10月17日 10:11 PM #46320
  
  わいひら
  キーマスター
  
  http://www.saka-en.com/java/c2a0/
  上記サイトでこのスペースのことを書かれています
  
  ようやく、おっしゃっていることの意味がわかりました。参考資料ありがとうございます。
  手元のファイルは修正したので、次のバージョンで改善されると思います。
  
  2に関してはバッチリでした。
  layout属性は必須では無かったのですね。
  
  AMP化で参考にしたページで、layout=”responsive”を使えと書いてあったので、僕も使っていましたが、CSSでレスポンシブするように書いておけば、問題なかったようです^^;
  僕の環境でもそれでうまくいっているので、次のバージョンで適用したいと思います。
  ご協力ありがとうございます！
- 2016年10月18日 8:29 PM #46378
  
  やーまん
  ゲスト
  
  2.3.5適応した所、こちらもGoogleのエラーが無くなりました。
  ご対応ありがとうございました。
  またよろしくお願いします。
  ＼(^o^)／
- 2016年10月19日 1:08 PM #46392
  
  わいひら
  キーマスター
  
  ご報告ありがとうございます！
  うまくいったようでよかったです。
  ご報告いただけたおかげで、修正部分に問題なかったら事がわかり安心しました^^
投稿者

投稿