Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ 固定表示 ] JavaScript のファイルが二重に読み込まれていたため対応 #2099

Merged
merged 3 commits into from
Jul 10, 2024

Conversation

mtdkei
Copy link
Contributor

@mtdkei mtdkei commented Jul 9, 2024

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

#2092

どういう変更をしたか?

固定表示ブロックでJavaScript のファイルが二重に読み込まれていたため対応しました。
#1973 と同じ対応をしています。

スクリーンショットまたは動画

変更前 Before

image
image

変更後 After

スクリーンショット 2024-07-10 8 34 39

実装者の確認事項

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

  • 複数の意図の変更 ( 機能の不具合修正 + 別の機能追加など ) を含んでいないか?
  • Files changed (変更ファイル)の内容は目視で確認したか?
  • readme.txt に変更内容は書いたか?
  • 本当にちゃんと確認をしたか?

プログラムの変更の場合

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

  • 書けそうなテストは書いたか?
    → スキップ

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

  1. 以下を固定ページに貼り付け保存
<!-- wp:vk-blocks/fixed-display {"mode":"show-on-scroll","scrollTiming":50,"scrollTimingUnit":"vh","fixedTopPosition":35,"fixedTopPositionUnit":"vh","blockId":"5978aa19-973e-4442-a878-395dfcb40edd","vkbCustomCss":"/* シェアボタン用 */\nselector .veu_socialSet ul {\n    display: block;\n}\nselector .veu_socialSet li {\n    width: auto;\n}\nselector .veu_socialSet .sb_icon .sb_icon_inner {\n  padding: 5px;\n  box-shadow: none;\n  border-radius: 50em;\n}\nselector .veu_socialSet .sb_icon .sb_icon_inner .icon_sns {\n    width: 25px;\n    height: 25px;\n    line-height: 25px;\n    padding: 0;\n    text-align-last: center;\n}\nselector .veu_socialSet .sb_icon .sb_icon_inner .sns_txt,\nselector .veu_count_sns_fb, selector .veu_count_sns_hb, selector .veu_count_sns_pocket {\n    display: none;\n}\nselector .veu_socialSet .sb_copy .sb_icon_inner i {\n    font-size: 14px;\n    margin: 0;\n}","className":"vk_custom_css"} -->
<div class="wp-block-vk-blocks-fixed-display vk_fixed-display vk_fixed-display-mode-show-on-scroll vk_fixed-display-position-right vk_fixed-display-5978aa19-973e-4442-a878-395dfcb40edd vk_custom_css" style="top:35vh" data-scroll-timing="50" data-scroll-timing-unit="vh" data-persist-visible="false"><!-- wp:vk-blocks/share-button /--></div>
<!-- /wp:vk-blocks/fixed-display -->

<!-- wp:group {"metadata":{"name":"シェアボタン確認用ブロック(削除してください。)"},"align":"full","style":{"color":{"gradient":"linear-gradient(180deg,rgb(238,238,238) 0%,rgb(169,184,195) 100%)"}},"layout":{"type":"constrained"}} -->
<div class="wp-block-group alignfull has-background" style="background:linear-gradient(180deg,rgb(238,238,238) 0%,rgb(169,184,195) 100%)"><!-- wp:vk-blocks/spacer {"spaceType":"height","unit":"vh","pc":35,"tablet":35,"mobile":35,"spaceSize":"custom"} -->
<div class="wp-block-vk-blocks-spacer vk_spacer"><div class="vk_spacer-display-pc" style="height:35vh"></div><div class="vk_spacer-display-tablet" style="height:35vh"></div><div class="vk_spacer-display-mobile" style="height:35vh"></div></div>
<!-- /wp:vk-blocks/spacer -->

<!-- wp:paragraph {"align":"center","fontSize":"large"} -->
<p class="has-text-align-center has-large-font-size">スクロールしてください</p>
<!-- /wp:paragraph -->

<!-- wp:paragraph {"align":"center"} -->
<p class="has-text-align-center">(シェアボタンが追従します。)</p>
<!-- /wp:paragraph -->

<!-- wp:vk-blocks/animation {"effect":"up-down","speed":"slow","blockId":"2ea1709a-f5b0-45b2-ac52-cd20f1b292fa"} -->
<div class="wp-block-vk-blocks-animation vk_animation vk_animation-up-down vk_animation-speed-slow vk_animation-range-short vk_animation-2ea1709a-f5b0-45b2-ac52-cd20f1b292fa"><!-- wp:vk-blocks/icon {"faIcon":"\u003ci class=\u0022fa-solid fa-arrow-down\u0022\u003e\u003c/i\u003e","iconSize":3,"iconSizeUnit":"em","iconAlign":"center","iconType":"2","iconColor":"black"} -->
<div class="wp-block-vk-blocks-icon vk_icon"><div class="vk_icon_frame text-center is-style-noline"><div class="vk_icon_border has-text-color has-black-color" style="width:calc(3em + 44px);height:calc(3em + 44px)"><i style="font-size:3em" class="fa-solid vk_icon_font fa-arrow-down"></i></div></div></div>
<!-- /wp:vk-blocks/icon --></div>
<!-- /wp:vk-blocks/animation -->

<!-- wp:vk-blocks/spacer {"spaceType":"height","unit":"vh","pc":120,"tablet":120,"mobile":120,"spaceSize":"custom"} -->
<div class="wp-block-vk-blocks-spacer vk_spacer"><div class="vk_spacer-display-pc" style="height:120vh"></div><div class="vk_spacer-display-tablet" style="height:120vh"></div><div class="vk_spacer-display-mobile" style="height:120vh"></div></div>
<!-- /wp:vk-blocks/spacer --></div>
<!-- /wp:group -->
  1. フロントエンドで1のページに挙動に問題がないか確認。
  2. ソースでページ内検索をして「vk-fixed-display.min.js」の表示が一回になっていることを確認。
  3. ダッシュボードから 設定 > VK Blocks で分割読み込みを指定した時にも3のようになっていることを確認。
    このプルリク以外で固定表示ブロックを設置したページをソースでページ内検索をすると「vk-fixed-display.min.js」が二回出てきます。

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

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

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

実装者と同じ確認を行って下さい。
なお、開発の方はソースの確認もお願いいたします。


レビュワー向け

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

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

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

@mtdkei mtdkei changed the base branch from master to develop July 9, 2024 23:57
@mtdkei mtdkei changed the title [ 固定表示 ] JavaScript のファイルが二重に読み込まれていたため対応 【確認待ち】[ 固定表示 ] JavaScript のファイルが二重に読み込まれていたため対応 Jul 9, 2024
@mtdkei mtdkei marked this pull request as ready for review July 10, 2024 00:02
@drill-lancer
Copy link
Member

@mtdkei
問題ないように思います。ありがとうございます。承認します。


どなたか2人目確認お願いします。

@mtdkei mtdkei changed the title 【確認待ち】[ 固定表示 ] JavaScript のファイルが二重に読み込まれていたため対応 【二人目確認待ち】[ 固定表示 ] JavaScript のファイルが二重に読み込まれていたため対応 Jul 10, 2024
@sysbird sysbird changed the title 【二人目確認待ち】[ 固定表示 ] JavaScript のファイルが二重に読み込まれていたため対応 【二人目確認中】[ 固定表示 ] JavaScript のファイルが二重に読み込まれていたため対応 Jul 10, 2024
@sysbird
Copy link
Member

sysbird commented Jul 10, 2024

対応ありがとうございます
WP6.5、6.6
分割読み込みありなしで解決されているのを確認しました!
マージしますね

* develop:
  Change default heml
  Added '2rows' to the classname containing wrap
  Add max-width for .vk_tab_labels--scroll-tab and .vk_tab_labels--wrap-tab
  Restore the deleted comment
  Restore the deleted comment
  Minor correction
  Add test file
  Add deprecated for ver.1.78.0
  Minor correction
  Add tab setting item for changelog
  Add tab setting item for changelog
  Minor correction
  Add not set
  Add tad display options
  Add tad display options

# Conflicts:
#	readme.txt
@sysbird sysbird merged commit ba16668 into develop Jul 10, 2024
14 checks passed
@sysbird sysbird deleted the fix/fixed-display/double-loading branch July 10, 2024 07:52
@kurudrive kurudrive changed the title 【二人目確認中】[ 固定表示 ] JavaScript のファイルが二重に読み込まれていたため対応 [ 固定表示 ] JavaScript のファイルが二重に読み込まれていたため対応 Jul 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants