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-redesign-help-dialog]mainリポジトリへマージする #2169

Merged
merged 19 commits into from
Jul 25, 2024

Conversation

Hiroshiba
Copy link
Member

内容

project-redesign-help-dialogの内容をmainリポジトリへマージします。

関連 Issue

その他

takusea and others added 18 commits December 18, 2023 21:49
* 見出し用のスタイルをMixinに追加

* BaseDocumentViewコンポーネントを追加

* ヘルプダイアログのドキュメントの表示を行うセクションのリデザイン

* コンポーネント名の変更

* インポートのパスを更新

* テストが落ちているのを修正

---------

Co-authored-by: Hiroshiba Kazuyuki <[email protected]>
* 新しい色変数scssファイルを作成

* _index.scssのuseにnew-colorsを追加

* 色の指定を色変数を用いるように変更
* 音声ライブラリの利用規約セクションを新しいデザインへ変更

* Update src/components/template/HelpLibraryPolicySection.vue

* Update src/components/template/HelpLibraryPolicySection.vue

---------

Co-authored-by: Hiroshiba <[email protected]>
…nViewSectionコンポーネントに共通化 (#1883)

* Markdown表示のセクションをHelpMarkdownViewSectionコンポーネントに統一

* テストの内容をコンポーネントに合わせて変更

* PageItem型にisShowOpenLogDirectoryButtonを追加

* is → should

---------

Co-authored-by: Hiroshiba <[email protected]>
* a要素のfocus時のoutlineのデザインを合わせる

* 背景・枠線のスタイルを統一

* 見出し1のフォントサイズを少し増加

* 最大幅・中央揃えを設定
* 色変数名を変更

* 背景色の削除・左枠線の追加

* ナビゲーションバーを新しいデザインへ変更

* サイドバー領域をスクロールできるように変更
* ドロワー背景とその他背景の色変数を分離

* セクション間の背景色を統一

* BaseRowCardコンポーネントに影を付与
* HelpDialog関連のファイルパスをmainリポジトリに合わせる

* Baseディレクトリの名前をPascalCaseに変更
* 色変数名とファイル名を変更

* ドキュメントコメントを追加
## 内容

mainリポジトリの変更差分をproject-redisign-help-dialogリポジトリに反映します。
* DocumentViewのデザインの微調整

* 色変数の間違いを修正
* 横幅が狭いときの表示崩れを修正

* 改段落ができていない箇所に改段落を追加

* スクロールバーの色を濃くする

* table要素のスタイルを追加

* DocumentViewの余白を調節

* div要素を削除
* 拡張子を.spec.tsに変更

* summary要素のスタイルを追加

* display,display-on-colorにlightenを追加

* SCSS変数を用いるように変更

* HelpDialogコンポーネントのパスを修正

* 新デザインで用いるscssファイルのパスをstyles/v2下に変更

* 不要なuseを削除
Copy link
Member Author

@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.

マージしたいと思います!
問題なさそうであればapproveいただければ・・・! @takusea

これはより良いデザインを目指していくための大きな第一歩だと感じています!!
@takusea さん、ありがとうございました!!!もしよければこれからもぜひよろしくお願いします!!!

ちなみにデザインと直接関係ないのですが、mainブランチでもコンポーネントライブラリである radix-vue が使えるようになります!
見た目的な実装のない、UIとしての機能だけが実装されているライブラリです。
Quasarに依存しない方が作りやすい UI はいっぱいあると思うので、その場合は是非使っていければと思います!


@takusea さんに1つ相談が!

大きめの機能が実装されたときにSNSで言及しておりまして、プロジェクトにリンクしてツイートしたいと思っています。
x.com/search?q=%23VOICEVOX%E9%96%8B%E7%99%BA%E7%8A%B6%E6%B3%81
もしよかったらそこで @takusea さんのXアカウントをツイート文に含めて紹介させていただきたいのですが、構わないでしょうか・・・?
(もしよければX のアカウントを教えていただければ・・・!)

こんな感じを予定しています!

#VOICEVOX開発状況 
ヘルプダイアログのデザインがよりVOICEVOXらしく刷新されました🎉(今後のアップデートで実装されます。)
【開発者: @???】
https://github.com/VOICEVOX/voicevox_project/issues/40

@takusea
Copy link
Contributor

takusea commented Jul 25, 2024

LGTM!

これまでの過程で不慣れな部分を多数ご助力いただきありがとうございました。ぜひこれからもよろしくお願いします!

もしよかったらそこで @takusea さんのXアカウントをツイート文に含めて紹介させていただきたいのですが、構わないでしょうか・・・?

OKです!Xのアカウントはまだ新しめのアカウントですが@takuseaが自分のものです!

@Hiroshiba
Copy link
Member Author

確認ありがとうございます、マージします!
(Xへのポストは明日の正午ぐらいを予定しています 🙏 )

@Hiroshiba Hiroshiba merged commit 846d07c into main Jul 25, 2024
18 checks passed
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