search

bright-org / bright

3 stars 0 forks source link

「ナイスブライト」に山も表示する #1654

Closed ymn-t-yamanashi closed 3 months ago

ymn-t-yamanashi commented 3 months ago

対応内容 ・三角グラフ（山）を表示 ・レベルアップ前後でどのレベルかアイコンで表示

あらかじめ、PCの画面はhaigoさんにOKもらってます

PC image

SP image

今回も修正しないこと 各パーセンテージについて beginnerはmaxPercent - skilledPercent - normalPercent の為割り切れないと、LevelUPの前後の各パーセンテージと誤差は発生します beginner + normal + skilled　= 100%になることを重視しました

tato-wk commented 3 months ago

@ymn-t-yamanashi みました 🙇

下記2点については、対応もコメントも頂いていない認識です。ご確認をよろしくお願いします。 https://github.com/bright-org/bright/pull/1654#discussion_r1733680139 https://github.com/bright-org/bright/pull/1654#discussion_r1733720168

ymn-t-yamanashi commented 3 months ago

@tato-wk コメントを返していましたが、見えづらいようなのでスクリーンショット貼りました (githubの仕様？)

image

image

tato-wk commented 3 months ago

@ymn-t-yamanashi コメントがPendingなので私には見えてないですね（GitHubあるある）

ymn-t-yamanashi commented 3 months ago

@tato-wk 失礼しました Pending解除しました

ymn-t-yamanashi commented 3 months ago

@tato-wk レビューありがとうございました

もし、負担が目立ったら今後対策を検討します