vektor-inc / vk-all-in-one-expansion-unit

wordpress plugin of powerful support
https://ja.wordpress.org/plugins/vk-all-in-one-expansion-unit/
GNU General Public License v2.0
7 stars 1 forks source link

有効化設定画面でex-iconが他のメニューにも表示されていたのを修正&微調整 #807

Closed goutetsuguma closed 2 years ago

goutetsuguma commented 2 years ago

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

https://github.com/vektor-inc/vk-all-in-one-expansion-unit/issues/805

どういう変更をしたか?

レビューに回す前に確認する事

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

cssのクラス名をid名に変更して、他のメニューにもアイコンが出ないようにした。 今後編集しやすいようにmixinでまとめれるところはまとめた。

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

VK ExUnitの有効化設定画面を開いた時にサイドのExUnitのロボットアイコンがちゃんとExUnitのメニュー横にのみ表示されているかどうか。

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


レビュワー向け

確認して変更が反映されていない場合の確認事項

kurudrive commented 2 years ago

@goutetsuguma あれ、これSVGに変更した時点で発生しなくなる問題だと思ったんだけどそうでもないの?

goutetsuguma commented 2 years ago

あ はい、、、クラス名の問題でした、、、