Commit Graph

57310 Commits

Author SHA1 Message Date
18c79dfda3 Move all the logic into BarsInfo class 2022-11-20 23:00:13 +03:00
f5fbb7c8de Changed comments 2022-11-20 19:27:40 +01:00
d20a357c0e Fixed ordering bug 2022-11-20 19:24:51 +01:00
9040dfbd4e Match leaderboard filter behaviour to web 2022-11-20 17:39:46 +01:00
1269afd0c5 Merge pull request #21307 from frenzibyte/disable-scroll-speed-on-osu
Hide scroll speed control from effects section on rulesets which don't support it
2022-11-20 16:17:55 +01:00
0f382590e6 Remove unnecessary #nullable disable 2022-11-20 15:31:51 +01:00
793d5b117e Fix timing screen test scene not creating editor beatmap properly 2022-11-20 15:20:35 +03:00
33b2fe46d9 Add xmldoc to HistoryTextBox 2022-11-20 12:29:47 +01:00
7b274083d3 Fix phantom beatmap sets appearing on carousel after delete/update
Regressed in c40c70509e.

As it turns out, `item.Item.Filtered.Value` is not the only condition
that should be checked to determine if a carousel item should be hidden
or not - `item.Item.State.Value != CarouselItemState.Collapsed` should
also be true. This was even available as the `item.Item.Visible`
convenience property, which is used in this commit.

Failing to check `item.Item.State.Value` led to setting non-zero alpha
on collapsed carousel items, leading to phantom beatmap sets appearing,
as the alpha was set in the entire carousel's `Update()` method, thus
firing every frame.
2022-11-20 10:08:34 +01:00
b3667821eb Add failing test case 2022-11-20 10:07:32 +01:00
36141cb2a4 Make BarsInfo a class 2022-11-20 05:14:07 +03:00
4b0ba86bd0 Identify scrolling rulesets using temporary DrawableRuleset instances 2022-11-20 04:12:52 +03:00
58296bd4f0 Introduce IDrawableScrollingRuleset for editor consumption 2022-11-20 04:11:10 +03:00
fcb52ee237 Make BarsInfo a readonly struct 2022-11-20 02:28:07 +03:00
fbfcf49ea6 Remove readonly modifier from struct fields 2022-11-20 02:13:54 +03:00
2cb966b47c Rework BarsInfo struct 2022-11-20 01:48:54 +03:00
6c62cfb830 Store barBreadth as a separate float 2022-11-19 23:40:02 +03:00
67ee9f3915 Naming adjustments 2022-11-19 23:34:55 +03:00
f1201454b7 Use value tuples 2022-11-19 23:29:50 +03:00
6f449a583e Handle empty values as a separate case 2022-11-19 23:27:48 +03:00
eff6c7be64 Removed unnecessary methods, changed tests, and moved LimitedCapacityQueue.cs to a more generic namespace. 2022-11-19 16:53:35 +01:00
9724c6bb53 Merge pull request #21205 from Samaoo/fix-tablet-aspect-ratio-values
Fix incorrect rounding of tablet aspect ratio values
2022-11-19 15:08:53 +01:00
0c671a2a82 Merge branch 'master' into fix-tablet-aspect-ratio-values 2022-11-19 14:32:24 +01:00
8bde669de1 Merge pull request #21266 from peppy/editor-test-play-dont-retain-test-time
When returning to the editor after test play, use the original editor time rather than the point of exit
2022-11-19 12:41:25 +01:00
c05c1625f2 Merge branch 'master' into editor-test-play-dont-retain-test-time 2022-11-19 12:09:04 +01:00
60c424837b Merge branch 'master' into disable-scroll-speed-on-osu 2022-11-19 12:13:45 +03:00
2e3af97892 Remove unused using 2022-11-19 12:13:36 +03:00
0239103b6b Fix BeatmapOverlay crashing test scene 2022-11-19 11:33:14 +03:00
05992d3aa8 CI fix 2022-11-19 10:23:37 +03:00
9b8f98735c Use struct for bars description 2022-11-19 10:16:58 +03:00
7d02cbafed Merge branch 'master' into beatmap-bar 2022-11-19 09:58:10 +03:00
11f5fddc1f Remove redundant returns 2022-11-19 09:57:52 +03:00
130f776427 Merge pull request #21267 from peppy/catch-file-enumeration-errors
Catch and gracefully handle file/directory enumeration failures during stable import
2022-11-19 15:38:40 +09:00
866b8b640a Merge pull request #21310 from frenzibyte/fix-present-beatmap-test
Fix intermittent present beatmap test failures
2022-11-19 15:38:07 +09:00
5fda702267 Merge pull request #21306 from frenzibyte/fix-editor-seeking-state
Fix editor not always playing hitsounds with clock offsets applied
2022-11-19 15:08:53 +09:00
ca67689a36 Fix intermittent present beatmap test failures 2022-11-19 07:44:43 +03:00
290369db47 Hide scroll speed slider on rulesets which don't support it 2022-11-19 06:42:27 +03:00
016de2f5a0 Fix editor not always playing hitsounds with clock offsets applied 2022-11-19 05:15:22 +03:00
4648e7bee0 Merge pull request #21299 from frenzibyte/perform-from-runner-load-state
Fix "perform from screen" invoking action on non-loaded screens
2022-11-19 09:55:21 +09:00
e8ca9f5dc5 Rework BarGraph to use Quads 2022-11-19 00:22:57 +03:00
d587f29351 Update tests to reflect desired behaviour 2022-11-18 21:03:30 +01:00
8943819ee7 Add test coverage 2022-11-18 17:49:03 +03:00
1be8c6bd9c Fix "perform from screen" not considering screen load state 2022-11-18 17:23:10 +03:00
b86b45b6d2 Fix excess string interpolation 2022-11-18 21:13:21 +09:00
da0de7c39e Merge pull request #21191 from Piggey/gameplay-leaderboard-fix
Display - symbol when tracked in-game leaderboard score is below 50
2022-11-18 20:23:21 +09:00
9ad484a0f4 Merge branch 'ppy:master' into gameplay-leaderboard-fix 2022-11-18 12:11:22 +01:00
06ce8130c2 Apply changes in line with framework NRT updates 2022-11-18 18:13:15 +09:00
95908ec023 Update framework 2022-11-18 18:13:00 +09:00
e4fc14faee Merge pull request #21270 from peppy/auto-advance-bindings
Auto-advance binding for ruleset key bindings
2022-11-18 16:19:54 +09:00
9d33a6568e Merge pull request #21292 from peppy/fix-binding-two-keys-single-binding
Fix being able to bind two non-modifier keys to the same binding
2022-11-18 16:19:38 +09:00