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

古いブロックは使ってほしくないのでオンオフ機能など検討 #1047

Closed
5 tasks done
kurudrive opened this issue Mar 1, 2022 · 9 comments
Closed
5 tasks done
Assignees

Comments

@kurudrive
Copy link
Member

kurudrive commented Mar 1, 2022

  • 新規インストールしたユーザーには古い非推奨のブロックは選択できないようにしたい
  • WordPressデフォルトで存在する利用するブロックの管理機能があるので、新規インストールの場合にその保存値を自動的に書き換えれば良いのではないか

対象

  • PR Block
  • PR Content
  • スタッフ
  • アイコンカード
  • カード
@shimotmk
Copy link
Contributor

@kurudrive
お疲れさまです。
ブロックのオンオフ機能について相談があります。

まず非推奨ブロック コアの方法はblock.jsonのsupport "inserter": false,にしてインサーターからブロックを選べないようにしていました
インサーターから消えるだけなのでブロックが壊れることやコピーやペーストを禁止はしていない
参考:WordPress/gutenberg#40167

WordPressデフォルトで存在する利用するブロックの管理機能があるので、新規インストールの場合にその保存値を自動的に書き換えれば良いのではないか

これについてですが、設定の表示ブロックを変更することはこのように設定すれば可能でした

wp.data.dispatch( 'core/edit-post' ).hideBlockTypes( [ 'core/heading', 'core/list' ] );

WordPress/gutenberg#14139 (comment)
ただ表示ブロックを変更してもサイトエディタ、ウィジェットエディタではインサーターに表示されてしまいます(コアの仕様)

なのでウィジェットやサイトエディターで使えるブロックもカスタマイズしたいなら管理画面が必要ということになりそうです。

管理画面を作る方向でもよろしいでしょうか?

また"inserter": false,にするブロックはないですか?
(旧 FAQブロックは新 FAQがあるので不要かなと思ったりしています)

@kurudrive
Copy link
Member Author

@shimotmk

確認ありがとうございます。

ただ表示ブロックを変更してもサイトエディタ、ウィジェットエディタではインサーターに表示されてしまいます(コアの仕様)

これ多分コアのみなさんが気づいてない(意図的にサイトエディターで使えるようにしてるわけではない)だけで、そのうちウィジェットやサイトエディターでも設定が反映されるようになる系の問題なんじゃないですかね?
※ 誰かがissue投げないと永遠に修正されないとは思うけど

その場合、コアとVK Blocks での 二重UIになるからあまりよろしくないのではないかなという印象。

サイトエディタやウィジェットは触る頻度が多くないし、今の段階では何が何でも非推奨のブロックを使わせたくないというほどでもないので、通常の記事編集画面で初期状態では選べなくするという処理で良いのではないかなという感覚です。

逆に下村さんの方で、
「いや、管理画面あった方がどのみち今後こんな感じの展開も出来るから作っちゃって損はないですよー。」というのがあれば作っちゃっうのもありだと思いますがいかがでしょう?

あー、そうか、でも管理画面あった方が、「ベクトルとしてこのブロックは非推奨になった」という表記をしやすいか...。

よし!作りましょう!(汗

@shimotmk
Copy link
Contributor

@kurudrive

これ多分コアのみなさんが気づいてない(意図的にサイトエディターで使えるようにしてるわけではない)だけで、そのうちウィジェットやサイトエディターでも設定が反映されるようになる系の問題なんじゃないですかね?
※ 誰かがissue投げないと永遠に修正されないとは思うけど
その場合、コアとVK Blocks での 二重UIになるからあまりよろしくないのではないかなという印象。

そうなのですよね。。後に修正されると思います。
おそらくこの辺りですべてのエディターで値を取得出来るように準備を進めていると思います。
WordPress/gutenberg#31965
そして私も出来るのであれば二重管理はしたくないと思っています。

なので急ぎでなければ一旦少しコアの動向を待ったほうが良い気がしてきました
すみません

@kurudrive
Copy link
Member Author

@shimotmk

いえいえ、独自UIを実装しての制御については見送りましょう。

@shimotmk shimotmk removed their assignment May 17, 2022
@shimotmk
Copy link
Contributor

shimotmk commented Sep 7, 2022

やっぱりブロックマネージャーは独自にDBに保存して欲しいと思うので優先度を戻そうと思います

理由
・VK Blocksの非推奨のブロックは新規インストール時にはオフにしたい
・コアはカスタムストアでやっているのでブラウザのストアを削除すると引き継がれない(DBには保存しないのでシークレットウィンドウで見ると表示が異なる)
・今後も機能は同じでもブロックを作り直したいみたいなケースあると思う

いや実装はしない方がが良いなどあったら言っていただければと思います

@shimotmk
Copy link
Contributor

shimotmk commented Dec 7, 2022

仕様やデザインは調整がないと判断したのでそのままプルリクに回します
何かあれば誰でもコメントください
#1498
block-manager

@kurudrive
Copy link
Member Author

@kaorock72 こちらも 1.48.0 でリリース済みなので、お知らせよろしくお願いいたします🙏

@kaorock72
Copy link

@kurudrive
こちらもお知らせ公開しました!おかしな箇所や不足などありましたら修正します。後ほどシェアいたします。
https://www.vektor-inc.co.jp/product-update/vk-blocks-1-48-0-add-block-manager-function/

@kurudrive
Copy link
Member Author

@kaorock72 確認しました!ありがとうございます!

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

No branches or pull requests

5 participants