yhira/simplicity2

header.phpのviewportの部分について

Closed this issue · 1 comments

もしかしたら僕が以前のバージョンをカスタマイズして勘違いしていたのかも知れませんけれども、header.phpのviewportなんですが、

<?php //ビューポート
//モバイルもしくはページキャシュモードの時
if ( is_mobile() || is_page_cache_enable() ): ?>
  <meta name="viewport" content="width=device-width,initial-scale=1.0">
<?php else: ?>

このifの中に、is_responsive_enable()が含まれている必要があるのではなかろうかと思ったりなんちゃったり。
完全レスポンシブの時は、viewportがdevice-widthだった気がするんですけれども、違いましたでしょうか?
現在のheader.phpのviewportだと、完全レスポンシブがonの時にmobile.cssが読み込まれないと言うような状態で、viewportが1280とかになると、ややこしいことになるのではなかろうかとふと思ったりしたので報告します。

実験的に現在Simplicity2の最新版をサイトに適用して、最新scssから出力したcssも適用した状態にしています。ここで完全レスポンシブをonにして上記ソースのif文にis_responsive_enable()を追加してviewportをdevice-widthにしていますが、特別問題は起きないようです。

yhira commented

そのようにしたほうがよさそうですね。
修正しておきました。ありがとうございます!
完全レスポンシブモードは、あまり使わないので、今回見直してみたら他にも結構具合のありました;