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

ダウンロードページを追加 #73

Closed
wants to merge 2 commits into from

Conversation

sopisoft
Copy link
Contributor

内容

ダウンロードページを追加
ダウンロードページ、トップページに「特徴」枠を追加

関連 Issue

#30
#56
#62

@Hiroshiba
Copy link
Member

PRありがとうございます!!
特徴欄の説明が情動的でいいなと感じました!

すみません、ちょっとホームページ自体が大きく変わる内容なので、もうちょっと考える時間を頂ければと思います。
主にこのあたりを考えています。

  • ダウンロードページができるとキャラクター一覧への誘導性が下がってしまうのをどう補填するか
  • 「特徴」はどれくらい推すか(その他の内容と表示順序をどうするか)
  • デザインをどうするか

@Hiroshiba
Copy link
Member

Hiroshiba commented Jun 12, 2022

おまたせしました!

「特徴」と「ダウンロードページ」に関して色々考えてみました。


特徴に関して、内容がとても素晴らしいのでぜひマージさせていただきたいです!
が、ちょっとデザイン的にまだ課題があるかなと感じました。
デザインは例えば、ちょっと想像できてないのですが、アイコンを追加したりしてもう少しポップな感じを演出できると嬉しそうです。
アイコンも少し探してみて、こちらのサービスがとても合いそうだなと感じました。
http://flat-icon-design.com/?page_id=41

デザインが用意できるまでマージをお待たせするのは申し訳ないので、実験モードでのみ表示するように変更していただくことは可能でしょうか。
こちらで実験モードを追加しました。

.env.developmentのフラグをtrueにすればcontexts.experimentsがtrueになるので、そのときだけfeatureコンポーネントを表示するようにする感じの実装になるかなと思っています。


ダウンロードページに関して、やはりキャラクターへの認知が下がってしまうのが課題になるだろうと考えました。
こちらもよければ実験モードのみの表示にして頂けるととても嬉しいのですが、こちらはページ追加なのでexperimentsのみ表示というのは難しそうに感じています。
デザイン的解決策が出てくるまでずっとopenとさせて頂くか、別途branchを作ってそちらにマージさせて頂ければと思っています。
どちらのほうが都合が良いでしょうか。


せっかく素敵なデザイン・実装のプルリクエストを頂いたのに、こちらが予めドキュメント化していなかった内容ですぐにリリースまで案内できず、本当に申し訳ないです・・・。

@sopisoft
Copy link
Contributor Author

デザインが用意できるまでマージをお待たせするのは申し訳ないので、実験モードでのみ表示するように変更していただくことは可能でしょうか。

デザイン的解決策が出てくるまでずっとopenとさせて頂くか、別途branchを作ってそちらにマージさせて頂ければと思っています。

すみません。ずっと放置してしまっていました。
近頃忙しいためしばらく手を付けられそうにないです。
「特徴」と「ダウンロードページ」両方の課題が解決するまでOpenとさせてください

@Hiroshiba
Copy link
Member

先にお時間頂いたのはこちら側なので・・・こちらこそ申し訳ないです 🙇
承知しました!もしこちら側でデザインができた際もお伝えしたいと思います。

また、webページはまだまだ課題が多いので、お忙しいタイミングが明けたらぜひお力お借りしたいです。
こちらとか。。。

@sopisoft sopisoft marked this pull request as draft January 12, 2023 14:40
@sopisoft sopisoft closed this May 18, 2023
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