管理画面のバナーが下まで見えない問題の修正

[kurudrive]

チケットへのリンク / 変更の理由（元のissueがあればリンクを貼り付ければOK）

#1392

どういう変更をしたか？

• 管理画面の js 制御による固定スクロールを廃止
• 左側メニュー部分のみCSS制御に変更

実装者の確認事項

実装者はレビュワーに回す前に以下の事を確認してチェックをつけてください。

• 複数の意図の変更 （ 機能の不具合修正 + 別の機能追加など ） を含んでいないか？
• Files changed (変更ファイル)の内容は目視で確認したか？
• readme.txt に変更内容は書いたか？ -> 設定画面の内部的な変更なので省略
• 本当にちゃんと確認をしたか？

プログラムの変更の場合

テストを書かないのは普通ではありません。書けるテストは極力書くようにしてください。

• 書けそうなテストは書いたか？ → PHPでもJSでもないので省略...

変更内容について何を確認したか、どういう方法で確認をしたかなど

実装者が確認した手順を箇条書きで記載してください。

• X-T9など沢山バナーの表示される構成で管理画面を表示
• 管理画面右側下部バナーが一番下まで表示される事を目視
• 右カラムの方が低い場合に、スクロールしたら固定される事を目視
• 画面サイズを変更しても表示要素が不自然に被ったり崩れたりしないか目視

確認URL

https://x-t9.vektor-inc.co.jp/wp-admin/options-general.php?page=vk_blocks_options
https://x-t9.vektor-inc.co.jp/wp-admin/admin.php?page=vkExUnit_main_setting

レビュワーに回す前の確認事項

• 実装者はこのテンプレートのチェック項目をちゃんと確認してチェックしたか？

レビュワー確認方法・確認内容など

レビュワーがどういう手順で何を確認して欲しいかを記載してください。

• https://x-t9.vektor-inc.co.jp/wp-admin/options-general.php?page=vk_blocks_options を開く
• 右側のバナーが下まで表示されるか確認
• 右側の要素が本文欄より少ない場合、スクロースしたら右カラムは一番下が表示された時点で固定される事を確認
• 左側のページ内ナビゲーションはスクロールしても固定される事を確認

---

レビュワー向け

レビュワーが確認して変更が反映されていない場合の確認事項

レビューしてみて意図した動作をしない場合は再度ビルドするなど以下の項目を確認してください。

• プルしたか？
• ビルドしたか？
• ビルドしたディレクトリは正しいか（別の開発環境のディレクトリを見ていないか）？
• npm install したか？
• composer install したか？
• キャッシュをクリアして確認したか？

[sysbird]

確認しました、問題ないです

たぶん ExUnit は別途修正するのですよね
ExUnit が有効な状態だとひきづられて(?)解決しないので、ExUnit 無効にして確認しました

どなたか2人め確認お願いします〜

[shimotmk]

@kurudrive
一番下までスクロールした状態でリロードすると、またはページを表示してすぐに一番下までスクロールすると、なぜか少し上に行ってしまう様ですがこれは自分だけですかね？

@sysbird

あとExUnit が有効な状態だとひきづられて(?)解決しない

これは確認できなかったですが、どの様な状態で発生しますか？？

vk-blocks-admin.mp4

[kurudrive]

@sysbird @shimotmk

あとExUnit が有効な状態だとひきづられて(?)解決しない

ExUnit が有効化されていると 環境によっては ExUnit 内の古い vk admin が優先で読み込まれるため、ExUnit 内の vk admin が古いと VK Blocks Pro 側の VK Admin が新しくても効かないという状態だと思います。
最新のExUnitでは既に新しい VK Admin で配信済みなので、問題は発生しないはずです。

このあたりも本当は早く composer のライブラリ化して 新しい方のライブラリが適用されるようにしたい所です...。

[kurudrive]

@shimotmk

一番下までスクロールした状態でリロードすると、またはページを表示してすぐに一番下までスクロールすると、なぜか少し上に行ってしまう様ですがこれは自分だけですかね？

あー、なるほど。
ブラウザが右側要素の一番下を判断して配置した後で、RSSが vektor-inc の要素を読み込んで右カラムの高さが増えてしまうから発生する感じですね。
RSSの読み込みタイムラグによる現象なのでちょっと仕方ないかなと思ってます。

あー、最後に読み込んだ分をDBで保持しておいて、新しく読み込み終わったらJSで差し替えるとかできなくはないか...でも個人的には他に優先順位の高い事が沢山あるので、ここはこれくらいでもいいんじゃないかという印象...（汗

[shimotmk]

@kurudrive

ここはこれくらいでもいいんじゃないかという印象...（汗

それならいいと思います
これもVWSのレスポンスの悪さが原因の一部ですね

ちなみにTransients_APIというものを使えばoption値を扱うように一時的にDBに保存するみたいなことは出来るかもです
https://wpdocs.osdn.jp/Transients_API

[kurudrive]

@shimotmk

ちなみにTransients_APIというものを使えばoption値を扱うように一時的にDBに保存するみたいなことは出来るかもです

ありがとうございます。恥ずかしながら知りませんでした。
ちょっといろいろ適用してみたいなと思いました！

が、ここに関してはリアルタイム性の方が重要なので、そのままで行きたいと思います (￣人￣)

[shimotmk]

@kurudrive
良いと思います
「最後に読み込んだ分をDBで保持しておいて...」と書いてあったのでキャッシュ機能の一部?としても使えるかなと思った次第です
vektor-inc/vk-all-in-one-expansion-unit#729

一旦このプルリクとしては良いと思います！

[shimotmk]

@sysbird
これお手隙の時に返信していただけませんか？
#1408 (comment)

[sysbird]

@shimotmk @kurudrive
ExUnit を更新して問題ないことを確認しました
下村さん、もしよかったら 承認してマージお願いします！