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

Pie Chartsの表示をRatedとAllで切り替え可能にする #1369

Merged
merged 1 commit into from
Oct 15, 2023

Conversation

4tsakabe
Copy link
Contributor

@4tsakabe 4tsakabe commented Feb 25, 2023

該当するIssue

#1312
Can we have an option to limit the different pie chart pages to just the English problems? Right now if you don't speak japanese it is impossible to get to 100% on the pie charts :(

初期のAtCoderの問題文は英訳されていないため、日本語が読めない場合にsolvedを100%にできない。
英語の問題文がある問題だけの達成度を切り替えて表示することはできないか。

実装方針

UserPageにある3つのPie Chartのタブについて,それぞれ変更を行う.
全てのコンテストとRatedコンテストの表示を切り替えられるボタンを追加し,isRatedContest()を用いて数えるコンテストを変更する.

スクリーンショット

AtCoder Pie Chartsタブ (Only Rated Contests)

AtCoder_onlyRated

AtCoder Pie Chartsタブ (All Contests)

AtCoder_all

Diffculty Piesタブ (Only Rated Contests)

Diff_onlyRated

Diffculty Piesタブ (All Contests)

Diff_all

Category Piesタブ (Only Rated Contests)

Category_onlyRated

Category Piesタブ (All Contests)

Category_all

相談したいこと

  • Category PiesタブでonlyRatedContestsにするとアルゴ以外が0になってしまうので見栄えが悪い

    • isRatedContestがRated Point Sum向けに作られたもので,rated contestとされず問題数が0になってしまう
    • 0の場合は非表示にするなどの処理を入れた方が良いか?
  • AtCoder Pie Chartsタブでコンテストごとにボタンを置くのが見栄え悪い

    • 他のタブと統一感を出すためにh1より上にボタンを置くのはやめてここにしました
    • 他に良いデザインがあれば教えてください

@4tsakabe 4tsakabe changed the title Only Rated ContestsとAll ContestsdePie Chartsの表示をの切り替え可能にする Pie Chartsの表示をRatedとAllで切り替え可能にする Feb 25, 2023
@4tsakabe
Copy link
Contributor Author

お忙しいところ恐縮ですが,お時間あるときにコードレビューよろしくお願いします!
また,デザイン含めて少し悩んで実装した部分があるので,良い実装があればそのように実装するのでコメントお願いします!

@4tsakabe
Copy link
Contributor Author

@kenkoooo お忙しいとは思いますが、確認よろしくお願いします

@kenkoooo
Copy link
Owner

ありがとうございます!時間が取れ次第必ずレビューするので、しばしお待ちを…

@your-diary
Copy link

@kenkoooo
こちらrevertの検討って難しいでしょうか?

理由

  • バグが多い(しかもエッジケースではなく全ユーザーが影響を受けるもの)
  • そのバグを直すためのissueやPRが上がっているが、後でやります #1421 にあるようにリポジトリ自体がinactiveになりがちなので、それらが解決するまでに時間を要しそう

@kenkoooo
Copy link
Owner

リバートします!

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.

3 participants