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

[4tsakabe]

該当するIssue

https://github.com/kenkoooo/AtCoderProblems/issues/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 Pie Chartsタブ (All Contests)


Diffculty Piesタブ (Only Rated Contests)


Diffculty Piesタブ (All Contests)


Category Piesタブ (Only Rated Contests)


Category Piesタブ (All Contests)


相談したいこと

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

  • isRatedContestがRated Point Sum向けに作られたもので，rated contestとされず問題数が0になってしまう
  • 0の場合は非表示にするなどの処理を入れた方が良いか？

• AtCoder Pie Chartsタブでコンテストごとにボタンを置くのが見栄え悪い

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

[4tsakabe]

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

[4tsakabe]

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

[kenkoooo]

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

[your-diary]

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

理由

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

[kenkoooo]

リバートします！