Luxeritas 2.4.1.2 リリース(主に不具合の修正)
Luxeritas 2.4.1.2 をリリースしました。
主に不具合の修正と、軽い仕様変更のみ。
詳細は開発者ブログの記事か後述の改訂履歴をご覧ください。
ダウンロードは、こちらのページ から。
改訂履歴
仕様変更
- 同じウィジェットエリア内に2つ以上のアドセンスウィジェットが存在した場合、それぞれの data-ad-client、data-ad-slot を踏襲した状態で amp-ad に置換するようにした
- 「関連記事」「NEXR/PREV ナビ」「新着記事ウィジェット」のサムネイルで 100x100px の画像が見つからなかったら 150x150px の画像を表示するようにした
- 外部リンクにアイコン付ける設定にしてあっても AMP ページではアイコン付けない仕様に変更(AMP だと画像にも勝手にアイコン付けられちゃうから)
内部最適化
- 誰も気づかない程度の最適化w
バグ修正
- 以下の組み合わせで AMP エラーが出ちゃう不具合修正
- サイトアイコンを設定している
- AMP を設定している
- フッターもしくはサイドバーをモバイル非表示に設定している
- 以下の条件だとモバイルで投稿本文内のグリッドが表示されない不具合修正
- bootstrap モード使用
- 投稿本文内で bootstrap のグリッドを使用
- フッターをモバイル非表示に設定している
- モバイルでサイドバーかフッターを非表示に設定してる時に、AMP の CSS 許容量の計算で24~48バイト程度の誤差が出るので修正
ディスカッション
コメント一覧
LuxeritasでHPを立ち上げようとしているWP初心者です。
それらしきものは出来上がりつつあるのですが、Next/Prevナビ(?)に関して調べても設定が分からず困っています。
具体的には、下記2点です。
1)ページ下部に表示されるNext/Prevナビ(この名称で正しいかも分かりませんが)の大きさ(高さ)を変更したいのですが、設定が分かりません。
現在、Next部分で約3行分程度、Prev部分で同じく3行 計6行程度の表示となっています。
PC等の大きな画面であれば、あまり目立ちませんが、スマホ画面だと目立ちます(特にスマホを横にするとほとんどNext/Prevナビになってしまいます)
この為、何とか大きさを小さくしたいのですが、どの様にすればよいか分かりません。
2)同じく、ヘッダーナビでモバイルメニューの種類を「豪華バージョン」にすると、上部ナビにNext/Prevのメニューが出てきますが選択することができません。 どの項目で設定すればよいか、調べてみても分かりません。
以上、初心者の質問かもしれませんが、初心者用の問い合わせ先も分からないのでこのコメントとして記述します。
平島 泰裕 さん。
> 1)大きさ(高さ)を変更したい
まぁ、手っ取り早いのは、高さを固定してしまう。
ただ、Next/Prev で 3行ってのはタイトル文字数がマルチバイトで40文字前後ってことなので、SEO 的にはよろしくないですねぇ。
> 2)選択することができません
あー、そーいえばモバイルメニューの Next/Prev は投稿ページだけの対応になっちゃってますねー。
固定ページのこと忘れてた。
近いうちに正式対応しますが、とりあえず、
luxeritas/navi.php の 60行目を
↓
に変更すればよいです。
るな様
平島です。早速の回答有難う御座います。
1)のNext/Prevの高さは変更は、「追加CSS」から張り付けてみましたが、多少変わった様な・・・
2)モバイルメニューの Next/Prevは、luxeritas/navi.php をどうすれば変更できるか分かりません。
多分FTPでサーバを見て、ファイルを直接書き換えるのでしょうが、自信がないので・・・
と言うことで、るな様のVersion-Upを待ちます。
ワープロ程度しかできない初心者には、この辺りが敷居が高いですが、Next/Prevを無視しても十二分のHPが出来ることに感謝しています。
有難う御座いました。
平島 泰裕さん。
> 1)のNext/Prevの高さ
「追加 CSS」ってのは WordPress の標準機能で Luxeritas の機能じゃないです。
Luxeritas を使うなら「追加 CSS」よりも子テーマの style.css に書いた方が「断然」良いです。
むしろ Luxeritas を使う場合は「追加 CSS」の中身は「完全に空っぽに」しちゃった方が良いくらいです。
るな様
>同じウィジェットエリア内に2つ以上のアドセンスウィジェットが存在した場合、
>それぞれの data-ad-client、data-ad-slot を踏襲した状態で amp-ad に置換するようにした
→こちらありがとうございます。が、正常に動作しないことを確認してしまいました。
amp-func.php内 function thk_replace_amp_adsense( $ad )で、
$ret = array_combine( $ad_client, $ad_slot );の箇所がありますが、
$ad-clientは一人のAdSenseユーザーで全て同じ文字列(ca-pub-〜数値〜)なので、
$ad_client, $ad_slotに2つ以上の要素が入っていてもここで要素1つになってしまいます。
例:同じウィジェットエリア内に2つ以上のアドセンスウィジェットが入っている
–記事下ウィジェット(AMP用)—
-Adsenseユニット-
ad-client : ca-pub-12345
ad-slot : 98765
-Adsenseユニット-
ad-client : ca-pub-12345
ad-slot : 54321
このときAMP表示させると、$retには最後の$ret[ca-pub-12345]=54321しか残らないので、
結果、最後のAdSenceユニットしか表示されません。
auto-ts さん。
あー。slot の方をキーにしないとダメか。
じゃぁ、単純に slot と client を逆にすれば良いだけなので、
amp-func.php の
132行目を
452行目を
456行目を
これで。
ありがとうございます!次のリリースで修正頂ければ幸甚です。
ちなみに、、、子テーマでstyle.cssとstyle-amp.cssの内容を同じにする効率良い方法ってありますでしょうか?
style-amp.cssに「@import url(“/wp-content/themes/luxech/style.css”); 」と記載したのですが動作せず。
style.cssにAMPの内容も書き込んで、管理を効率化させたいのです。
auto-ts さん。
AMP は外部 CSS 自体を許可してないので @import で動作するわけない。
やるならこんな感じかな?
1. ブログにアクセスがあった場合に style.css のタイムスタンプとサイズをチェック。
2. 双方が異なってたら style.css を style-amp.css にコピー。
3. style-amp.css のタグを AMP 用に置換。
アドバイスありがとうございます!これをもとに以下のコードでやってみることにしました。
毎回サイズをチェックしなくてもよいので、randで確率的にチェックするようにしてみました。(wp-cronに組み込もうかと思いましたがより単純にしました)
ありがとうございます!
auto-ts さん。
例えば、img とかは amp-img に置換しないといけない場合もあるので、ファイルサイズよりタイムスタンプのチェックの方がええな。
なので、
1. タイムスタンプをチェック
2. タイムスタンプが違ってたらコピーして、必要な箇所を置換
3. タイムスタンプが同じになるように双方のファイルを touch
この方がええかも。
あと「確率」でやるよりも、
を使って「自分がブログにアクセスした時だけ」っていう条件の方が良いと思う。