AtCoderProblems
AtCoderProblems copied to clipboard
ユーザー名が未入力のときは自分のデータを表示するようにする。
現状ユーザー名が未入力のときは入力欄のプレースホルダーに自分のユーザー名が表示されますが、自分のデータが読み込まれるわけではありません。これではややこしいので、ユーザー名が未入力のときは自分のデータを表示するようにしましょう。
Show Acceptedの動作を修正しました。
![]()
Show Acceptedの動作を修正しました。
具体的に何をどう修正したのか書いてもらって良いですか?
元の挙動:
Show Accepted Onの時、
Show Acceted Offの時、
このように、ShowAcceptedの値がそのコンテストの問題を全部解いたかの表示にしか適応されていなかったため↑のコメントのように全体に適応されるように修正しました。(説明不足ですみません…)
全体と言いますと?
コンテスト名だけでなく、問題名の背景にもShow Acceptedが効くようにしたということです。
すみません、よく分かりませんでした… (これは僕の理解力が低いためなので気にしないでください)
え、一応確認しますけどKenkooooさんの想定するShow Acceptedトグルスイッチの機能ってなんですか…? (もしかしたら僕がなにか勘違いをしているかもしれません)
「show accepted がオフのとき、全完したコンテストが表示されなくなる」です。
Show Acceptedってそういうスイッチだったんですか…自分はこのスイッチをみたときに「ACした問題や全完したコンテストの背景を緑色にするかしないかを切り替える」ものだと思っていました。そのため、いままでのような説明だったのですが…すみません、僕の勘違いでした。ただ、Show Acceptedという名称は分かりづらくないですか?
ACした問題の背景を塗りたくなかったらユーザー名を空にすればいいので……
名前が分かりにくいのはそのとおりだと思います。元々こるとんさんのABC全埋めチャレンジのためにできたスイッチで、使ってる人は分かっているので問題にならなかったんでしょう。適当なものに変更したいですね。
名前が分かりにくいのはそのとおりだと思います。元々こるとんさんのABC全埋めチャレンジのためにできたスイッチで、使ってる人は分かっているので問題にならなかったんでしょう。適当なものに変更したいですね。
デフォルトでオフにして、Hide Completed Contestとかにするのはどうでしょう? となりのShow Difficultiesが色をつけるかつけないかというボタンなので同じようにShowという表現を使うのは僕がしたみたいな誤解を生みやすいと思います。
ACした問題の背景を塗りたくなかったらユーザー名を空にすればいいので……
このIssueはユーザー名を空にしたときに自分のデータを表示しようというものなので、背景を塗らないオプションが新たに必要になると思います。 そこでShow Acceptedをいじっていました。
ユーザー名を空にしたときに自分のデータを表示する意義については現状入力欄のプレースホルダーが自分のユーザー名なので、ユーザー名が空なときは誰のデータも表示されないというのはProblemsを初めて利用する方には伝わりにくいと思いました。 というか自分も時々混乱します。 そのためプレースホルダーの中身を変えるかデフォルトで自分のデータを表示するか考えてみて、デフォルトで自分のデータが表示されたほうが便利だと思ったのでこのようなIssueを建てています。
このIssueの問題の本質は、 AtcoderProblemsのトップページで、空ユーザのTableコンテンツを表示している仕様にあると思います。
適当な改善方法としては、
- ログイン時のトップページをTableカラム選択状態にする(つまり、
/table/<user>へリダイレクト) - 適当なトップページを表示する
現時点のサイトのフローや挙動(Tableタブを押すことでログイン情報が反映される)は分かりにくいと思うので、何かしらの改善は欲しいなと思いますが
- はとりあえずの対応はできますが、現状の仕様に最適化した対応なので、本質的な改善にはならないと思います。
- は要はサイトフローの見直しになるので、トップページの内容の議論含めコストが高いです。
すみません、そもそも解いた問題を塗らないオプションの必要性を全く理解していないのですが、どういう時に必要になるんですか?
すみません、そもそも解いた問題を塗らないオプションの必要性を全く理解していないのですが、どういう時に必要になるんですか?
自分も使う予定はありませんが… ただ、長らくユーザー欄を空にすることで問題の背景を塗らなくすることができたので活用している人が存在している可能性はあります。 例えば、単純に問題のDiffだけをスクショでシェアしたいときとか… うーん、そういうの方への対応は声があがってからでいいかもしれませんね。 とりあえず解いた問題をぬらないオプションの実装は見送ります…
このIssueの問題の本質は、 AtcoderProblemsのトップページで、空ユーザのTableコンテンツを表示している仕様にあると思います。
適当な改善方法としては、
- ログイン時のトップページをTableカラム選択状態にする(つまり、
/table/<user>へリダイレクト)- 適当なトップページを表示する
現時点のサイトのフローや挙動(Tableタブを押すことでログイン情報が反映される)は分かりにくいと思うので、何かしらの改善は欲しいなと思いますが
- はとりあえずの対応はできますが、現状の仕様に最適化した対応なので、本質的な改善にはならないと思います。
- は要はサイトフローの見直しになるので、トップページの内容の議論含めコストが高いです。
トップページをつくるのはともかく、リダイレクトまで必要ですか? 単純にログインしているかつユーザー欄が空なら自分のデータを表示するだけで良いと思っていますが。
仕様が変わりそうなので一旦プルリクを取り下げました。
ReduxもしくはRecoilが取り入れられるか、いい方法が見つかるまで保留とします。