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

ソングページ:トークページにあるOSSとかの案内を追加 #204

Closed
Hiroshiba opened this issue Apr 25, 2024 · 2 comments · Fixed by #213
Closed

ソングページ:トークページにあるOSSとかの案内を追加 #204

Hiroshiba opened this issue Apr 25, 2024 · 2 comments · Fixed by #213

Comments

@Hiroshiba
Copy link
Member

内容

ソングページ内でもせっかくなのでオープンソースへのリンクなどを案内したいです。

実現方法

トーク側には全く同じものがあるので、それをコンポーネントに切り出して流用するのが良さそうです。

ただトーク側は白基調、ソング側は黒基調なので見た目が合わないはずです。
他のコンポーネントでもたまにあるisDark引数を追加して両対応するのが良いかなと。

あと多分横幅もずれてるのですが、これはちょっとbulmaの知識が必要なのでめんどうかもです。
is-max-desktop的なクラスがついてると思うので、それを入れ替えるたらできたりします。
けどまあ置いといてくださったら僕がちゃちゃっとやっちゃいます!

その他

@jdkfx
Copy link
Contributor

jdkfx commented Jul 1, 2024

@Hiroshiba
こちらのIssueについて、提案があります。
以下の画像のように、トークとソング画面で説明を記述している部分についての幅を統一したいと考えております。

トーク
ソング

現状のハミングについて説明している箇所の幅を設定しているクラスはexplain-hummingで、調べてみた限りでは、このハミングの説明をしている箇所でしか使用されていないため、

          {/* explain-hummingをis-max-desktopに変更 */}
          <div className="container is-max-desktop"> 
            <h2 className="title">ハミングとは?</h2>
            <p>
              喋り声のデータを用いて音声ライブラリを作成し、
              歌えるキャラクターに歌い方を倣うことで、
              キャラクターの喋り声に近い声で歌える機能です。
            </p>
            <p>キャラクターによっていろんなスタイルで歌うことができます。</p>
          </div>

とすることで、コンポーネント化したオープンソースへの案内と同じ幅に設定することができます。

PRも作成することが可能で、もしよろしければPRを出してもよろしいでしょうか?

@Hiroshiba
Copy link
Member Author

Hiroshiba commented Jul 1, 2024

@jdkfx コメントありがとうございます!!

ソングの横幅に関しては気になっていたので、調整してくださるならありがたいです!!
もしかしたらプルリクエストにコメントするかもですが、一旦提案の形でPR作成していただけると非常に助かります!!

ご提案はいくつかあるかもですが、1つにまとめてしまっていただいても大丈夫です!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants