-
-
Notifications
You must be signed in to change notification settings - Fork 151
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
show difficulty subボタンの追加 #1470
base: master
Are you sure you want to change the base?
Conversation
概ねいい感じに見えます。difficultyを表示するボタンを👉で表現するのは分かりやすくていいと思いました。 👉ボタンについてbuttonではなくspanで実装されているのが気になります。普通の文字と同じ扱いなので、カーソルをかざした時これがボタンなのかどうか分かりづらいと感じました。また、Tabキーでのカーソル移動に引っかからずアクセシビリティに問題があるので、buttonでの実装にした方が良いと思います。 2値 or 3値これは完全に個人の感想なのですが、👉の絵文字がたくさん並んでいると従来と比べて情報量が多くて、自分的にはちょっとつらいです。full/sub/noneの3値だとうれしいかもです。 tooltipの説明は確かにいらないかも...内容が全部同じなのでどこか一か所に置けば事足りるような気もします。 |
ありがとうございます!マークアップ分かってなさすぎてよくなかったです、、修正してみますw
同じ気持ちです、ありがとうございます。3値に変えるってなると、大項目の
確かに、3値にするなら使わない人は使わないですし、一旦省いてみます。 |
差分大きくなっちゃってるのでPR自体は必要であれば分けます。まずは完成版の機能の議論をできると嬉しいです。 |
思い付きなんですが、Color ByでContest Resultを選択したときのShow Penaltiesみたいな感じで、Show DifficultyがオフなときにPartial Show Difficulty(もっといい命名がありそう)的なボタンを表示してみるのはどうでしょうか。Difficultyを表示する(All相当)のときの見た目は以前と変わらなくなります。 2024-04-21.050156.mp4ただごちゃつき具合はうーんという感じですね。むずかしい... (動画が再生できなかったので差し替えました) |
設定について考えていて思ったのですが、あくまで自分の理想として、部分表示は常に「有効」になっていて、表示ボタンは👉ほど目立たない...というのが良いよなあと感じました(さっき3値の方がいいかもと言ったのに。申し訳ない)。デザインとしてどうなるかは謎ですが...。 |
返事遅れました。諸々ご提案ありがとうございます!
なるほど、確かにです。
この案で、スマホの方は最悪UIをトルツメとかでもいいですかね。一旦PCだけでも導入できたらいいのかなと。 |
ちょっとローカルで実装してて思ったのですが、
これを導入するとどうしても誤表示が起きてしまうかも、と思ったのでやるなら表示/非表示自体は選べた方がいいのかも。と思いました。。
3値よりこちらの方が良さそうというのは僕も思ったので、partial showのボタンを追加する仕様にしつつ、デザインは上記で挙げてもらったような控えめな感じのやつってイメージです。どうでしょう...? |
ありがとうございます!部分表示の常時有効はデメリットが大きそうなので、選択制にしましょう。選択方法も含めて、自分が追加で議論したいことは無いです。 控えめな表示ボタンは大きく分けると
の二種類があると思ったのですが、どちらが良いでしょうか(もちろん、3、4種類目があるかもしれない)。 自分の感想、放言ですが、前者は画面に残り続けるのではっきり言って邪魔だと思いました。後者はボタンをクリックしたら消えてしまうので、余計な情報が無くて良いのではないかと思っています。 |
(おふとぴ)ところで、開発時のホットリロードは効いていますか?コードの変更がすぐブラウザに反映されて大変便利なのですが、今のmasterだと効かないんですよね。#1466 で修正をしているので、もし困っていたらお試しください。 |
#1468
show difficulty = off かつ on/off切り替えのためのボタン = off
show difficulty = off かつ on/off切り替えのためのボタン = on
show difficulty = on (as-isと同じ)
各問題ごとにon/off切り替えのためのボタンを設置したことで、見た目の影響が大きい...?
大元のshow difficultyボタンそのものを、on/offの2値ではなく、full/sub/noneみたいな3値にした方が、ユーザビリティ高かったりする?要議論。
reactは得意ではないので実装方針変だったらすみません。