Commit Graph

100 Commits

Author SHA1 Message Date
bd512c4937 Remove equality comparison implementation from BeatmapCard
This turned out to be a hurdle instead as it disallows adding two
beatmap cards of equal beatmap, which, while being a good behaviour in
client, makes tests more complicated to work.
2022-11-04 19:27:20 +03:00
07bfac40fa Adjust padding to avoid overlap with card border when expanded 2022-11-03 14:03:28 +09:00
66a6084d3f Scale in the background fill alongside the icon 2022-11-03 14:03:19 +09:00
df9f49eef2 Move hover layer behind icon
Looked bad on the "already downloaded" state where the icon becomes
black.
2022-11-03 13:56:07 +09:00
ef72b66dad Remove beatmap card background workaround to fix broken corners 2022-10-13 16:09:54 +03:00
6c316bcc9e Make beatmap card icon buttons fill up to the area 2022-10-13 16:09:54 +03:00
aa8040d696 Add hover box to beatmap card icon button 2022-10-13 16:09:54 +03:00
07f577a0c6 Fix beatmap listing potentially showing duplicate beatmap cards 2022-09-13 01:39:21 +03:00
e9ce968ae3 Merge branch 'master' into fix-play-button-touch 2022-09-09 18:08:18 +09:00
ed81297611 Fix playlist items showing download button briefly during initial local presence checks 2022-09-08 17:10:45 +09:00
94ec653420 Add same load-cancel safeties to ensure tracks are disposed in card PlayButton 2022-08-05 14:26:01 +09:00
26c5b59f6d Replace usages of string.To{Lower,Upper}() 2022-06-24 11:57:45 +02:00
f8830c6850 Automated #nullable processing 2022-06-17 16:37:17 +09:00
1107e267e3 Fix beatmap card play button not working with touch inputs when not hovered 2022-06-10 22:03:51 -07:00
30cf6bffad Add tooltips to beatmap card icon pills 2022-06-03 21:41:52 -07:00
365819865e Remove 'submit' sample usages 2022-06-03 22:31:35 +09:00
88ba84ac9c Replace Pill with Badge everywhere 2022-05-14 20:52:38 +03:00
9bac33ec9f Integrate spotlight beatmap badge in listing cards 2022-05-14 19:47:31 +03:00
6f0e32826c Standardise ordering/grouping of IRulesetInfo/RulesetInfos 2022-02-11 04:27:11 +03:00
247c557eaf Fix expanded content scrollbar inadvertently hiding expanded content 2022-01-19 22:30:49 +01:00
77748a5f93 Show scrollbar on expanded card content where applicable 2022-01-19 22:21:08 +01:00
374dac57f2 Change expanded card content height to 200 2022-01-04 13:22:00 +09:00
0ad555e9f7 Remove surplus blank line 2022-01-03 18:33:32 +01:00
b9851b278d Add padding to the bottom of the beatmap listing overlay to avoid hovered panels exceeding visible bounds
Closes https://github.com/ppy/osu/issues/16120.
2022-01-03 13:18:28 +09:00
3ca238825b Unify width of normal and extra card sizes 2021-12-24 09:17:07 +01:00
a49a4329ee Add capability to switch between card sizes 2021-12-23 15:55:37 +01:00
cd4c1bc678 Add factory method for various card sizes 2021-12-23 15:55:36 +01:00
c7b9b176c6 Rename BeatmapCard{Base => } 2021-12-21 08:26:25 +01:00
88d4e95ad8 Rename BeatmapCard{ => Normal} 2021-12-21 08:26:21 +01:00
33e930f477 Move scale-on-expand logic to BeatmapCardContent 2021-12-17 13:29:20 +01:00
d6f6039934 Extract base class for beatmap cards 2021-12-17 13:27:11 +01:00
f052b47d87 Extract collapsible button container for card usage 2021-12-17 12:58:05 +01:00
7aab12d4b0 Share extra row dropdown show/cancel show logic 2021-12-17 11:45:05 +01:00
3fa45479b0 Share hype/nomination statistic show logic 2021-12-17 11:45:05 +01:00
c6d0b5d200 Merge branch 'master' into beatmap-card/extra-wip 2021-12-17 11:11:36 +01:00
ad430a6277 Centralise hover state handling (and fix back-to-front conditionals) 2021-12-15 16:44:58 +09:00
6a1f535257 Refactor cancellation of expand to be more explicit 2021-12-15 16:38:19 +09:00
94d1a2aaca Remove unnecessary collapse call from BeatmapCard
This is already handled at the `BeatmapCardContent` level. This call
actually causes the buggy behaviour reported in
https://github.com/ppy/osu/discussions/16085.
2021-12-15 16:37:47 +09:00
7a9db22c52 Tidy up method naming and structure for expanded state changes 2021-12-15 16:29:12 +09:00
ef4ab74565 Also only expose Expanded state of BeatmapCard as read-only 2021-12-15 16:26:19 +09:00
41e6c24dad Expose Expanded state of BeatmapCardContent as read-only bindable
This is just to reduce complexity of these interactions by ensuring that
the expanded state can only be changed by the class itself.
2021-12-15 16:26:19 +09:00
083ee92dee Adjust button vertical padding 2021-12-08 22:56:28 +01:00
2d739c95ea Lay out extra card statistics in grid as per design 2021-12-08 22:56:28 +01:00
3ecfaa532c Add source field to extra beatmap card 2021-12-08 22:56:28 +01:00
419fee1380 Move mapper link to bottom content 2021-12-08 22:54:11 +01:00
61e04f75cc Resize extra card to design size 2021-12-08 22:54:10 +01:00
a32492cdd5 Duplicate BeatmapCard{-> Extra} as blueprint for extra card size 2021-12-08 22:53:51 +01:00
25a0505c97 Scale card when expanding to better distinguish hovered card from other cards in listing 2021-12-07 23:48:14 +09:00
82ed8eae6b Ensure hover handling container always calls base on hover events 2021-12-06 21:02:40 +01:00
ccfc361626 Apply naming suggestions 2021-12-06 21:02:38 +01:00