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

一覧に著者名,更新日を追加する #314

Closed
horimasumi opened this issue Mar 17, 2021 · 6 comments
Closed

一覧に著者名,更新日を追加する #314

horimasumi opened this issue Mar 17, 2021 · 6 comments
Labels
design デザインや UI/UX に関するもの enhancement 新機能実装またはリクエスト

Comments

@horimasumi
Copy link

どのような問題を解決する機能追加でしょうか?

ブックやトピックを探し出しやすくする.

あなたが望む解決方法について書いてください

追加するのは,
マイブック
マイトピック
ブックの再利用
トピックの再利用

  • [✓ ] コードに実装された。
@horimasumi horimasumi added the enhancement 新機能実装またはリクエスト label Mar 17, 2021
@kou029w kou029w self-assigned this Mar 26, 2021
@kou029w
Copy link

kou029w commented Mar 26, 2021

@knokmki612 ブック一覧、ブックの再利用のあたりですがどうしましょうかね。
おそらくブック周りで変更していくのはこのあたりのコンポーネントという認識です。

  • BookAccordion
  • BookTree

個人的に思っていることとしては、次の2点です。

  • BookAccordionはBookPreviewに見た目を寄せたい気持ちがあります。
    • クリックした際の選択の有無の振る舞いの違い以外で、もともとどういう経緯で見た目が異なるんでしたっけ?
  • BookTreeはブック名の横に加えていく感じですかねぇ。ごちゃっとした感じになりそうかも。

もしなにかアドバイスあるならもらえると嬉しいです。

@knokmki612
Copy link

クリックした際の選択の有無の振る舞いの違い以外で、もともとどういう経緯で見た目が異なるんでしたっけ?

  • BookAccordion: 1ブックあたりの面積を折り畳むことで小さくして、複数のブックとしての一覧性を重視している
  • BookPreview: 折りたたまないことでブックのアウトラインを見回す&プレーヤーも配置して単一のブックとしての一覧性を重視している

という違いがあるでしょうか

BookTreeはブック名の横に加えていく感じですかねぇ。ごちゃっとした感じになりそうかも。

基本的に横に広がっていって親のwidthを使い切ったら折り返されるような状態だったと思います

レイアウトとして破綻はしないですが、たしかにごちゃごちゃするかもしれませんね。

更新日の情報はブック名より文字を小さくするとかコントラストを弱めるとか(アクセシビリティの観点との相談にもなりますが)見せ方を変えたり工夫する必要がありそうです(別途検討でもよいかと思います)

@knokmki612
Copy link

更新日の情報はブック名より文字を小さくするとかコントラストを弱めるとか

つまりItemコンポーネントを使うということになりますかね

@knokmki612
Copy link

knokmki612 commented Mar 26, 2021

kou029wさんと方針を相談しました

BookAccordionはBookPreviewに見た目を寄せたい

背景として、ブックを閲覧/編集するためのUIが5個ぐらいあり、「同じ対象を扱うのに、見た目が異なるものがこれだけあるのは混乱するのでは」という懸念がある。

  • BookAccordion
  • BookTree
  • BookPreview
  • template/Book
  • BookForm

そのため、統一可能なものは統一したほうが、UIの実装としても、ユーザー体験としてもよいのではないかと考える。

BookAccordionは折りたたむかわりに個別のトピック・セクションにアクセス可能という特徴があるが、

  • 閲覧系: BookPreviewの「もっと詳しく」で閲覧できればよい
  • 編集系: ブック編集→トピック編集と遷移すればよい

という2点により、BookPreviewにUIを統一できるのではないかという話になった。

方針としては、BookPreviewに統一してハンズオンいただいてから、変更が支障ないか様子をみていくのがよいと考えた。

@knokmki612
Copy link

コンポーネントの整理が必要な点については別途issueを作成しました #406

@horimasumi
Copy link
Author

horimasumi commented Aug 21, 2021

マイブック
マイトピック
ブックの再利用
トピックの再利用

のうち,ブックの再利用だけが,まだ,著者名,変更日が追加されていない.というか,古いデザインのまま.
とはいえ,一旦,こちらはクローズして, #498 で議論する.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
design デザインや UI/UX に関するもの enhancement 新機能実装またはリクエスト
Projects
None yet
Development

No branches or pull requests

3 participants