【コア様子見待ち】蛍光マーカー カスタムカラーが使えないバグ修正

[shimotmk]

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

#1456

どういう変更をしたか？

カスタムカラーが使えるように修正
以前からカラーが変わった瞬間にsetIsAddingColorをfalseにしていましたがカラーが変わった瞬間にsetIsAddingColorをfalseにしなくても特に問題は無いと思ったのでクリアした時のみにしました。

実装者の確認事項

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

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

プログラムの変更の場合

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

• 書けそうなテストは書いたか？

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

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

• レビュー確認方法と同様です。

確認URL

（　どこかのデモサイトかテストサーバーにデプロイ済みなどで確認できる場合はそのURL　）

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

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

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

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

• 蛍光マーカーでカスタムカラーが使えるか
• 他気になる箇所があればご指摘お願いします

---

レビュワー向け

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

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

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

[kurudrive]

※ ユーザーからの不具合報告 & マージしたら VK Blocks Pro 1.46.0 をリリースするため優先度を上げました🖐

[osmdik]

蛍光マーカーでカスタムカラーが使えることを確認しました。
2人目確認をどなたかお願いいたします。

[drill-lancer]

@shimotmk
キーボードで値を入力しようとすると数秒毎に値が保存されてうまく値を入力できません。

[shimotmk]

@shimotmk キーボードで値を入力しようとすると数秒毎に値が保存されてうまく値を入力できません。

確認ありがとうございます
これはコアのハイライトと同じ状況ですか？

同じであればコアのバグということで様子見にさせていただければ。

[drill-lancer]

@shimotmk
コアも同じような状況なので今回はスルーで Approve します。

[shimotmk]

@drill-lancer
このブランチはカスタムカラーが使えないバグ修正する意味で出したのですが@drill-lancerさんはキーボード入力のバグが終わるまでマージしない方が良いという判断でしょうか？

[drill-lancer]

@shimotmk
https://vektor-vws.slack.com/archives/CJD95D5LZ/p1668399223451569
現状では上記に対する完全回答とはいえないので見送ったという意味が強いです。
（そもそも何が変わったのかがよくわかりませんし。）

[shimotmk]

@drill-lancer

https://vektor-vws.slack.com/archives/CJD95D5LZ/p1668399223451569
現状では上記に対する完全回答とはいえないので見送ったという意味が強いです。

「カラーコードを編集しようとバックスペースキーを押すとポップアップが消えてしまう」は解決していると思うのですが、、どうでしょうか？

（そもそも何が変わったのかがよくわかりませんし。）

このブランチの変更点がわからないという意味ですか？

[drill-lancer]

@shimotmk

「カラーコードを編集しようとバックスペースキーを押すとポップアップが消えてしまう」は解決していると思うのですが、、どうでしょうか？

確かにそれは治ってますが、結局キーボードを打とうとすると今度はキーボードバグを突っ込まれる展開になるのは明白。
まあ、そこは先手を取って説明すれば良いのですが・・・

そういう意味でマージは見送った感じですが、上記を加味してマージしても良いという判断ならマージしても良いと思います。

[kurudrive]

• 完全には修正されていないが、そこはコアの領域でこちらで対応する事ではない
• VK Blocks Pro 側の問題によって発生していた症状があり、そこは改善されている

という事ならマージで良いと思います。

[shimotmk]

@drill-lancer
コアでも同じissuesがあったので説明はつくと思います
WordPress/gutenberg#41088

このブランチの変更点に問題がないか確認してマージお願いします

[drill-lancer]

@shimotmk
マージしました。