AtCoderProblems icon indicating copy to clipboard operation
AtCoderProblems copied to clipboard

ユーザー名が未入力のときは自分のデータを表示するようにする。

Open makutamoto opened this issue 5 years ago • 18 comments

image 現状ユーザー名が未入力のときは入力欄のプレースホルダーに自分のユーザー名が表示されますが、自分のデータが読み込まれるわけではありません。これではややこしいので、ユーザー名が未入力のときは自分のデータを表示するようにしましょう。

makutamoto avatar Aug 11 '20 18:08 makutamoto

image image Show Acceptedの動作を修正しました。

makutamoto avatar Aug 12 '20 14:08 makutamoto

image image Show Acceptedの動作を修正しました。

具体的に何をどう修正したのか書いてもらって良いですか?

kenkoooo avatar Aug 13 '20 02:08 kenkoooo

元の挙動: Show Accepted Onの時、 image Show Acceted Offの時、 image このように、ShowAcceptedの値がそのコンテストの問題を全部解いたかの表示にしか適応されていなかったため↑のコメントのように全体に適応されるように修正しました。(説明不足ですみません…)

makutamoto avatar Aug 13 '20 06:08 makutamoto

全体と言いますと?

kenkoooo avatar Aug 13 '20 07:08 kenkoooo

コンテスト名だけでなく、問題名の背景にもShow Acceptedが効くようにしたということです。

makutamoto avatar Aug 13 '20 07:08 makutamoto

すみません、よく分かりませんでした… (これは僕の理解力が低いためなので気にしないでください)

kenkoooo avatar Aug 13 '20 15:08 kenkoooo

え、一応確認しますけどKenkooooさんの想定するShow Acceptedトグルスイッチの機能ってなんですか…? (もしかしたら僕がなにか勘違いをしているかもしれません)

makutamoto avatar Aug 13 '20 16:08 makutamoto

「show accepted がオフのとき、全完したコンテストが表示されなくなる」です。

kenkoooo avatar Aug 13 '20 16:08 kenkoooo

Show Acceptedってそういうスイッチだったんですか…自分はこのスイッチをみたときに「ACした問題や全完したコンテストの背景を緑色にするかしないかを切り替える」ものだと思っていました。そのため、いままでのような説明だったのですが…すみません、僕の勘違いでした。ただ、Show Acceptedという名称は分かりづらくないですか?

makutamoto avatar Aug 13 '20 16:08 makutamoto

ACした問題の背景を塗りたくなかったらユーザー名を空にすればいいので……

名前が分かりにくいのはそのとおりだと思います。元々こるとんさんのABC全埋めチャレンジのためにできたスイッチで、使ってる人は分かっているので問題にならなかったんでしょう。適当なものに変更したいですね。

kenkoooo avatar Aug 13 '20 16:08 kenkoooo

名前が分かりにくいのはそのとおりだと思います。元々こるとんさんのABC全埋めチャレンジのためにできたスイッチで、使ってる人は分かっているので問題にならなかったんでしょう。適当なものに変更したいですね。

デフォルトでオフにして、Hide Completed Contestとかにするのはどうでしょう? となりのShow Difficultiesが色をつけるかつけないかというボタンなので同じようにShowという表現を使うのは僕がしたみたいな誤解を生みやすいと思います。

makutamoto avatar Aug 13 '20 16:08 makutamoto

ACした問題の背景を塗りたくなかったらユーザー名を空にすればいいので……

このIssueはユーザー名を空にしたときに自分のデータを表示しようというものなので、背景を塗らないオプションが新たに必要になると思います。 そこでShow Acceptedをいじっていました。

ユーザー名を空にしたときに自分のデータを表示する意義については現状入力欄のプレースホルダーが自分のユーザー名なので、ユーザー名が空なときは誰のデータも表示されないというのはProblemsを初めて利用する方には伝わりにくいと思いました。 というか自分も時々混乱します。 そのためプレースホルダーの中身を変えるかデフォルトで自分のデータを表示するか考えてみて、デフォルトで自分のデータが表示されたほうが便利だと思ったのでこのようなIssueを建てています。

makutamoto avatar Aug 13 '20 16:08 makutamoto

このIssueの問題の本質は、 AtcoderProblemsのトップページで、空ユーザのTableコンテンツを表示している仕様にあると思います。

適当な改善方法としては、

  1. ログイン時のトップページをTableカラム選択状態にする(つまり、/table/<user>へリダイレクト)
  2. 適当なトップページを表示する

現時点のサイトのフローや挙動(Tableタブを押すことでログイン情報が反映される)は分かりにくいと思うので、何かしらの改善は欲しいなと思いますが

  1. はとりあえずの対応はできますが、現状の仕様に最適化した対応なので、本質的な改善にはならないと思います。
  2. は要はサイトフローの見直しになるので、トップページの内容の議論含めコストが高いです。

MatsuTaku avatar Aug 13 '20 23:08 MatsuTaku

すみません、そもそも解いた問題を塗らないオプションの必要性を全く理解していないのですが、どういう時に必要になるんですか?

kenkoooo avatar Aug 14 '20 01:08 kenkoooo

すみません、そもそも解いた問題を塗らないオプションの必要性を全く理解していないのですが、どういう時に必要になるんですか?

自分も使う予定はありませんが… ただ、長らくユーザー欄を空にすることで問題の背景を塗らなくすることができたので活用している人が存在している可能性はあります。 例えば、単純に問題のDiffだけをスクショでシェアしたいときとか… うーん、そういうの方への対応は声があがってからでいいかもしれませんね。 とりあえず解いた問題をぬらないオプションの実装は見送ります…

makutamoto avatar Aug 14 '20 05:08 makutamoto

このIssueの問題の本質は、 AtcoderProblemsのトップページで、空ユーザのTableコンテンツを表示している仕様にあると思います。

適当な改善方法としては、

  1. ログイン時のトップページをTableカラム選択状態にする(つまり、/table/<user>へリダイレクト)
  2. 適当なトップページを表示する

現時点のサイトのフローや挙動(Tableタブを押すことでログイン情報が反映される)は分かりにくいと思うので、何かしらの改善は欲しいなと思いますが

  1. はとりあえずの対応はできますが、現状の仕様に最適化した対応なので、本質的な改善にはならないと思います。
  2. は要はサイトフローの見直しになるので、トップページの内容の議論含めコストが高いです。

トップページをつくるのはともかく、リダイレクトまで必要ですか? 単純にログインしているかつユーザー欄が空なら自分のデータを表示するだけで良いと思っていますが。

makutamoto avatar Aug 14 '20 05:08 makutamoto

仕様が変わりそうなので一旦プルリクを取り下げました。

makutamoto avatar Aug 14 '20 05:08 makutamoto

ReduxもしくはRecoilが取り入れられるか、いい方法が見つかるまで保留とします。

makutamoto avatar Aug 23 '20 18:08 makutamoto