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

Favorites and Sortings #717

Merged
merged 9 commits into from
Apr 5, 2024
Merged

Favorites and Sortings #717

merged 9 commits into from
Apr 5, 2024

Conversation

anatawa12
Copy link
Member

@anatawa12 anatawa12 commented Apr 5, 2024

Fixes #641

@anatawa12 anatawa12 added the enhancement New feature or request label Apr 5, 2024
@anatawa12 anatawa12 marked this pull request as ready for review April 5, 2024 12:55
@anatawa12 anatawa12 changed the title Feat favorite Favorites and Sortings Apr 5, 2024
@Sayamame-beans
Copy link
Collaborator

これって、プロジェクト種やUnityバージョンでソートした時の、同値間の並び順ってどうやって決まってるんでしょうか? (VCCと同じ並びっぽかったですけど、個人的によく分かっていない)

@anatawa12
Copy link
Member Author

同値間の並び順ってどうやって決まってるんでしょうか?

DBが返してきた順番ですね。最終更新日順とかにしといたほうがいいかもしれない?

@Sayamame-beans
Copy link
Collaborator

Sayamame-beans commented Apr 5, 2024

もう一点…
どのルールでソートされているのかが若干認識しづらい気がしました(実際にソートされているルールのヘッダで↑または↓が表示されていて、それ以外は↕️となってはいますが)
かといって、VCCの挙動(ヘッダに現在のソートを示す矢印だけ表示)は他でソート可能なことが分かりづらく、微妙です。

現在ソートされていないものについては、カーソルを重ねた時だけ↕️を見せる(ソート可能なことを示す)って可能だったりしますか?
(或いは、Windowsに準拠するなら、矢印ではなくヘッダごとハイライトされるっぽいです(win挙動 参照:
image
ハイライト右側の矢印は、ソートのものではなく絞り込みのものです)

@Sayamame-beans
Copy link
Collaborator

同値間の並び順ってどうやって決まってるんでしょうか?

DBが返してきた順番ですね。最終更新日順とかにしといたほうがいいかもしれない?

そうかも

@anatawa12
Copy link
Member Author

とりあえずハイライトと "カーソルを重ねた時だけ↕️を見せる" を実装しました

@anatawa12
Copy link
Member Author

同値間の並び順ってどうやって決まってるんでしょうか?

DBが返してきた順番ですね。最終更新日順とかにしといたほうがいいかもしれない?

そうかも

も実装しました

@Sayamame-beans
Copy link
Collaborator

おっと…(winのハイライトってカーソル重ねた時の話でした)
でも、これ(ソート中のルールを常時ハイライト)悪くないですね
割とアリなのでこのままで良さそう

細かくて申し訳ないですが、見た目に対してソート選択可能な範囲が少し狭いのが気になりました
それ以外は特に気になるところないです
(CSS的事情で困難そうなら後回しに)

@anatawa12
Copy link
Member Author

(CSS的事情で困難そうなら後回しに)

これなので後回しにします

Copy link
Collaborator

@Sayamame-beans Sayamame-beans left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@anatawa12 anatawa12 merged commit e036d59 into master Apr 5, 2024
17 checks passed
@anatawa12 anatawa12 deleted the feat-favorite branch April 5, 2024 15:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

プロジェクト一覧の表示順の種類を増やす/プロジェクトfav機能への対応
2 participants