【2人目確認中】「Twitter」の名前やアイコンを「X」に変更

drill-lancer commented 1 year ago

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

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

どういう変更をしたか？

「Twitter」の名前やアイコンを「X」に変更

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

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

プログラムの変更の場合

[ ] 書けそうなテストは書いたか？ ⇒名称変更を行っただけなので不要かと

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

「Twitter」の名前やアイコンを「X」に変更されているのを確認

確認URL

ローカル環境

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

「Twitter」の名前やアイコンを「X」に変更されているか確認

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

[x] このテンプレートのチェック項目をちゃんと確認してチェックしたか？

レビュワー向け

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

プルしたか？
ビルドしたか？
ビルドしたディレクトリは正しいか（別の開発環境のディレクトリを見ていないか）？
npm install したか？
composer install したか？

kurudrive commented 1 year ago

@drill-lancer ありがとうございます。 class 名は変更するとCSSでカスタマイズしてる人に影響が出るので、 vk_icon_w_r_sns_x_twitter にするのではなく vk_icon_w_r_sns_x vk_icon_w_r_sns_twitter みたいに２つ出力する方向で調整よろしくお願いいたします。

drill-lancer commented 1 year ago

@kurudrive vk_icon_w_r_sns_x_twitter も vk_icon_w_r_sns_twitter も :before でアイコンを出力しているのでおかしなことになりそうです。

kurudrive commented 1 year ago

@drill-lancer あー、なるほど、そういう事ね。

まぁよくよく考えたらシェアボタン共通でカスタマイズはあると思うけど twitter 固有でカスタマイズはそうそうないから変更でもいいか。

vk_icon_w_r_sns_x_twitter じゃなくて vk_icon_w_r_sns_x でいいんじゃない？

drill-lancer commented 1 year ago

@kurudrive Font Awesome が square-x-twitter みたいに x-twitter ってつけてるのにちなんだ形です。

kurudrive commented 1 year ago

@drill-lancer ぐほ...FontAwesome そんな半端な.... なるほど、それなら継承した方がいいですね（・ｗ・；

sysbird commented 1 year ago

@drill-lancer アイコンが変更されているのを確認しました

別件かもしれませんが、私のローカルのとある環境(WP6.2)で、ソーシャルボタンがすべて緑色になっています。どのテーマでもおなじ色になっており、有効なプラグインは本件のみですボタンの色はどこかで設定できますか？

sns

sysbird commented 1 year ago

@drill-lancer 連投すみません〜他のローカル環境(WP6.3)で表示されているこちらが正しいでしょうか？ xのボタンのシャドウが青いままな気がします (たぶんグレーっぽくする?) ↓↓↓

drill-lancer commented 1 year ago

@sysbird

私のローカルのとある環境(WP6.2)で、ソーシャルボタンがすべて緑色になっています。

多分 Evergreen を適用しているとかではないですか？

xのボタンのシャドウが青いままな気がします (たぶんグレーっぽくする?)

シャドウは確かに青いままですね。いい感じのコードを教えていただけませんか？

    .sb_twitter a {
        background: #00abec;
        box-shadow: 0 2px 0 rgba(0, 126, 170, 1);
    }
    .sb_x_twitter a {
        background: #0f1419;
        box-shadow: 0 2px 0 rgba(0, 126, 170, 1);
    }

sysbird commented 1 year ago

@drill-lancer そうですね、提案もせず失礼しました

@goutetsuguma @kurudrive ねんのため下記はいかがでしょうか？

(1) 他のボタンに合わせると、シャドウは濃いめになるので、 x-1

.sb_x_twitter a {
    background: #0f1419;
    box-shadow: 0 2px 0 rgba(0, 0, 0, 1);
}

(2) 境い目が見えないからグレーにするとか？ x-2

.sb_x_twitter a {
    background: #0f1419;
    box-shadow: 0 2px 0 rgba(204, 204, 204, 1);
}

sysbird commented 1 year ago

多分 Evergreen を適用しているとかではないですか？

たしかに Evergreen の色ですね Evergreen は無効で、テーマが x-t9やTT3でもこの色になりますキャッシュは削除しましたこの issue には関連ないですね、どこか別にメモしておきます

goutetsuguma commented 1 year ago

多分 Evergreen を適用しているとかではないですか？

たしかに Evergreen の色ですね Evergreen は無効で、テーマが x-t9やTT3でもこの色になりますキャッシュは削除しましたこの issue には関連ないですね、どこか別にメモしておきます

カスタマイズ→ExUnit設定　の「SNS設定」の「ソーシャルボタンのスタイル設定」のボタンの色のところに色が設定してあるのかもしれません！

goutetsuguma commented 1 year ago

@sysbird @kurudrive @drill-lancer

ボタンの影の色の提案ありがとうございます！悩みますね、、！ (1)と(2)の中間くらいの影の色はどうでしょうか？

box-shadow:0 2px 0 rgb(124 124 124); とか、、、？

sysbird commented 1 year ago

@goutetsuguma ありがとうございます！テーマの設定のほうですね、ありました！ Evergreen の色が入っているのはなにかの拍子にはいったままになってしまったのかな？ここでは忘れます〜

sysbird commented 1 year ago

@drill-lancer 下記でお願いします〜

.sb_x_twitter a {
    background: #0f1419;
    box-shadow: 0 2px 0 rgba(0, 0, 0, 1);
}

drill-lancer commented 1 year ago

@sysbird X の Shadow の修正しておきました。

sysbird commented 1 year ago

@drill-lancer ありがとうございます、マージしました！

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