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

ブロック一覧の最上部にベクトル製品のブロックが表示されるようにしてほしいです #2000

Closed
mtdkei opened this issue May 9, 2024 · 8 comments · Fixed by #2024
Assignees

Comments

@mtdkei
Copy link
Contributor

mtdkei commented May 9, 2024

ブロックを挿入する際に+ボタンを押したときに表示されるブロック一覧の最上部に、ベクトル製品のブロックが表示されるようにしてほしいです。

参考例(SWELL) :
image

@drill-lancer
Copy link
Member

@kurudrive
順番は下記で OK でしょうか?
VK Blocks ⇒ VK ブロックレイアウト => ( VK EXUNIT Blocks ) ⇒ コア

@kurudrive
Copy link
Member

@drill-lancer ほぃ。それで。ってこれまだ @mtdkei さん着手してなかったっけ?

@mtdkei
Copy link
Contributor Author

mtdkei commented Jun 3, 2024

@kurudrive
こちらは着手してないです。

@drill-lancer
Copy link
Member

@kurudrive
Deprecated Blocks は VK ブロックレイアウトと ( VK EXUNIT Blocks ) の間ですか?
それともコアの後でしょうか?

@kurudrive
Copy link
Member

@drill-lancer え?そんなカテゴリー ( Deprecated Blocks )存在しなくない?
今回新規に作って分けるって事(・w・?

@drill-lancer
Copy link
Member

drill-lancer commented Jun 3, 2024

@kurudrive
存在自体はしているみたいです。使ってないだけかもですが・・・

if ( ! in_array( 'vk-blocks-cat-deprecated', $keys, true ) ) {
$categories = array_merge(
$categories,
array(
array(
'slug' => 'vk-blocks-cat-deprecated',
'title' => $vk_blocks_prefix . __( 'Deprecated Blocks', 'vk-blocks-pro' ),
'icon' => '',
),
)
);
}

@kurudrive
Copy link
Member

@drill-lancer あー、作ってあるけど使われてはいないって感じか。

  • カテゴリーで非推奨にわけてもユーザーが気づかない(だから今ラベル自体に非推奨表記してる)
  • もしかして非推奨から通常に復帰するブロックが出てくるかもしれない

なので、カテゴリー自体削除でいいかな。

@drill-lancer drill-lancer linked a pull request Jun 3, 2024 that will close this issue
6 tasks
@mtdkei
Copy link
Contributor Author

mtdkei commented Jun 5, 2024

マージされましたのでCloseします

@mtdkei mtdkei closed this as completed Jun 5, 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 a pull request may close this issue.

5 participants