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

[project-redisign-help-dialog] 色変数名をnew-color-*からcolor-v2-*に変更 #2132

Conversation

takusea
Copy link
Contributor

@takusea takusea commented Jun 22, 2024

内容

間が空いてすみません。色変数名・その格納ファイル名を変更します。
・格納ファイル名をcolors-v2.scssへ変更
・上記ファイル内でSCSS変数へ橋渡しするためのCSS変数の名前を--color-v2-*へ変更

@takusea takusea requested a review from a team as a code owner June 22, 2024 15:00
@takusea takusea requested review from Hiroshiba and removed request for a team June 22, 2024 15:00
@takusea takusea changed the title 色変数名をnew-color-*からcolor-v2-*に変更 [project-redisign-help-dialog] 色変数名をnew-color-*からcolor-v2-*に変更 Jun 22, 2024
Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ほぼLGTMです!!

プルリクエストありがとうございます!!お待ちしていました!!

結構時間が空いていて、コンフリクトがいっぱいあったりしてマージするのは難しい箇所や、どう解決すればいいかわからない箇所も結構あると思います。
その時は気軽にお声かけください!!

@@ -0,0 +1,111 @@
// 基本色
Copy link
Member

@Hiroshiba Hiroshiba Jun 25, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

このファイルは何なのかを1~2行で紹介するのとかどうでしょう?
(いわゆるドキュメントコメント的な)

例えばですが、
「新デザイン以降のためのカラー定義。
詳細は issue URL
とか・・・!

後から見た方が、このファイルはどうなる方向になっているか追えるようにしておきたいなと!

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

了解です!
issueの作成はこのリポジトリで行う感じで大丈夫でしょうか。

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

あ、リデザインプロジェクトのissue urlのつもりでした!
VOICEVOX/voicevox_project#40

各色の説明とか、設計方針もどこかに書いてあると良いかもですね!
実はdocsディレクトリに色についてのメモがあるので、こちらにメモを追記するとかもありかもです。あったほうが良さそうであれば・・・!!
https://github.com/VOICEVOX/voicevox/blob/2b059d462c1481003907acacd006cf53320ee230/docs/%E8%89%B2%E3%81%AB%E3%81%A4%E3%81%84%E3%81%A6.md

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

あっなるほどです!
色変数のメモに関してはカスタムテーマやソングUIとの兼ね合いでまだ変動がありそうなので、colors-v2.scssへの追記だけしておきます!

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

承知です!! ということでresolve・・・!

Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!!

@Hiroshiba Hiroshiba merged commit 1247fed into VOICEVOX:project-redisign-help-dialog Jun 27, 2024
7 checks passed
@Hiroshiba
Copy link
Member

再度になりますが、もしコンフリクト解消が難しそうorファイル移動とかフォーマッターとかがよくわからない感じになったら気軽にお声掛けください 🙏

@takusea
Copy link
Contributor Author

takusea commented Jun 28, 2024

ありがとうございます…!

@takusea takusea deleted the rename-color-variables-file branch July 26, 2024 10:49
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 this pull request may close these issues.

2 participants