Dean Herbert
15d286e57d
Merge pull request #27898 from bdach/preview-tracks-playing-after-hide
...
Fix preview tracks playing after their owning overlay has hidden
2024-04-16 23:54:54 +08:00
Bartłomiej Dach
6c943681b0
Fix preview tracks playing after their owning overlay has hidden
...
RFC. Closes https://github.com/ppy/osu/issues/27883 .
The idea here is that `PopOut()` is called _when the hide is requested_,
so once an overlay trigger would hide, the overlay would
`StopAnyPlaying()`, but because of async load things, the actual track
would start playing after that but before the overlay has fully hidden.
(That last part is significant because after the overlay has fully
hidden, schedules save the day.)
Due to the loose coupling between `PreviewTrackManager` and
`IPreviewTrackOwner` there's really no easy way to handle this locally
to the usages of the preview tracks. Heck, `PreviewTrackManager` doesn't
really know which preview track owner is to be considered _present_ at
any time, it just kinda works on vibes based on DI until the owner tells
all of its preview tracks to stop.
This solution causes the preview tracks to stop a little bit later but
maybe that's fine? Just trying to not overthink the issue is all.
No tests because this is going to suck to test automatically while it is
pretty easy to test manually (got it in a few tries on master).
The issue also mentions that the track can sometimes resume playing
after the overlay is pulled up again, but I don't see that as a problem
necessarily, and even if it was, it's not going to be that easy to
address due to the aforementioned loose coupling - to fix that, play
buttons would have to know who is the current preview track owner and
cancel schedules upon determining that their preview track owner has
gone away.
2024-04-16 16:19:26 +02:00
Dan Balasescu
3bd2bb0ed6
Merge pull request #27896 from bdach/fix-pause-overlay-missing-in-osu
...
Fix resume overlay not appearing after pausing when mouse cursor is inside window but outside of actual playfield area
2024-04-16 22:11:17 +09:00
Bartłomiej Dach
e5e345712e
Fix resume overlay not appearing after pausing inside window but outside of actual playfield area
...
Related to https://github.com/ppy/osu/discussions/27871 (although does
not actually fix the issue with the pause button, _if_ it is to be
considered an issue - the problem there is that the gameplay cursor
gets hidden, so the other condition in the modified check takes over).
Regressed in
bce3bd55e5
.
Reasoning for breakage is silent change in `this` when moving the
`Contains()` check (`DrawableRuleset` will encompass screen bounds,
while `OsuResumeOverlay` is only as big as the actual playfield).
2024-04-16 13:45:09 +02:00
Bartłomiej Dach
f9873968a5
Apply NRT in OsuResumeOverlay
2024-04-16 13:39:06 +02:00
Bartłomiej Dach
c4bf03e640
Add failing test
2024-04-16 13:39:05 +02:00
Bartłomiej Dach
d2a67d2d54
Merge pull request #27881 from smoogipoo/fix-mod-refilter
...
Don't re-filter unless mods may change the filter
2024-04-16 11:21:35 +02:00
Bartłomiej Dach
2c90ba9614
Merge pull request #27891 from peppy/stop-background-process-high-performance
...
Fix high memory usage during spectator modes when background processing is running
2024-04-16 10:57:33 +02:00
Dean Herbert
926424d8ea
Update HighPerformanceSessionManager.cs
...
Co-authored-by: Bartłomiej Dach <dach.bartlomiej@gmail.com>
2024-04-16 16:37:11 +08:00
Dean Herbert
a651cb8d50
Stop background processing from running when inside a high performance session
2024-04-16 10:04:38 +08:00
Dean Herbert
d89edd2b4f
Expose high performance session state
2024-04-16 10:04:38 +08:00
Dean Herbert
7e4782d4b1
Allow nested high performance sessions
...
Mostly just for safety, since I noticed this would pretty much fall over
in this scenario until now.
2024-04-16 10:04:38 +08:00
Dan Balasescu
0e218ee271
Merge pull request #27148 from Givikap120/traceable_pp
...
Add Traceable performance calculation support
2024-04-16 04:54:57 +09:00
Dan Balasescu
343b3ba0e6
Don't re-filter unless mods may change the filter
2024-04-15 21:07:36 +09:00
Dan Balasescu
fe7df808b6
Add tests
2024-04-15 21:07:08 +09:00
Dean Herbert
1c02cbfa5d
Merge pull request #27877 from bdach/fix-faded-pp-on-stable-scores
...
Fix stable scores showing with faded out pp display due to classic mod presence
2024-04-15 18:21:21 +08:00
Bartłomiej Dach
7c4c8ee75c
Fix stable scores showing with faded out pp display due to classic mod presence
2024-04-15 11:53:05 +02:00
Bartłomiej Dach
8506da725d
Add failing test
2024-04-15 11:49:47 +02:00
Dan Balasescu
d34834af05
Merge pull request #27853 from Drison64/mask-volume-bar
...
Fix toolbar volume bar masking
2024-04-15 03:36:31 +09:00
Ondřej Vajďák
ed6680a61d
Fixed type inconsistency and rounding
2024-04-14 15:10:05 +02:00
Ondřej Vajďák
9833dd955f
Fix toolbar volume bar masking
2024-04-14 01:30:59 +02:00
Dan Balasescu
e14e2a4629
Merge pull request #27845 from Loreos7/beatmap_hash
...
Fix missing underscore in `invalid beatmap_hash` notification message
2024-04-13 21:16:55 +09:00
Loreos7
5a8b8908dd
fix missing underscore
2024-04-13 14:53:51 +03:00
Givikap120
4a21ff9726
removed duplication
2024-04-13 13:59:09 +03:00
Givikap120
feb9b5bdb8
Make traceable pp match HD
2024-04-13 13:42:57 +03:00
Dean Herbert
b7b3401b34
Merge pull request #27839 from smoogipoo/fix-macos-crash
...
Fix crash when entering multiplayer on macOS
2024-04-13 12:54:00 +08:00
Dan Balasescu
c7f3a599c9
Fix crash when entering multiplayer on macOS
2024-04-13 13:17:06 +09:00
Dan Balasescu
3793a552b8
Merge pull request #27832 from peppy/skin-editor-new-items-apply
...
Fix newly placed items in skin editor not getting correct anchor placement
2024-04-12 21:00:52 +09:00
Dean Herbert
c0dce94f15
Fix newly placed items in skin editor not getting correct anchor placement
2024-04-12 17:08:49 +08:00
Dean Herbert
da1a07fb45
Merge pull request #27827 from smoogipoo/fix-ios-build
...
Update Sentry to fix iOS build
2024-04-12 08:48:12 +08:00
Dean Herbert
3ec93745a4
Fix test failures due to sentry oversight
2024-04-12 01:10:42 +08:00
Dan Balasescu
8b2017be45
Update Sentry to fix iOS build
2024-04-12 01:02:40 +09:00
Dean Herbert
3e8ddbd2a9
Add new entries to dotsettings (Rider 2024.1)
2024-04-11 11:33:16 +08:00
Bartłomiej Dach
b559bec49f
Merge pull request #27734 from smoogipoo/fix-catmull-bulbs
...
Fix bulbs on Catmull sliders
2024-04-10 11:49:29 +02:00
Bartłomiej Dach
518addf323
Merge branch 'master' into fix-catmull-bulbs
2024-04-10 11:25:02 +02:00
Dean Herbert
6cb5bffdfc
Update resources
2024-04-10 13:03:37 +08:00
Dean Herbert
6c5fad8638
Merge pull request #27810 from smoogipoo/fix-discord-rpc-crash
...
Fix possible crash due to race in DiscordRichPresence
2024-04-08 23:37:44 +08:00
Dan Balasescu
7d8fe51178
Fix possible crash due to race in DiscordRichPresence
2024-04-08 23:33:42 +09:00
Bartłomiej Dach
b226a05637
Merge pull request #27787 from peppy/update-hit-error-icons
...
Update hit error metre to use new icons
2024-04-05 09:58:12 +02:00
Bartłomiej Dach
d6b1e68ee7
Merge branch 'master' into update-hit-error-icons
2024-04-05 09:36:18 +02:00
Dean Herbert
6093b50641
Merge pull request #27797 from smoogipoo/revert-storyboard-optimisation
...
Revert recent storyboard optimisations
2024-04-05 14:56:44 +08:00
Dan Balasescu
7b92c725b1
Revert "Merge pull request #27454 from EVAST9919/sb-lifetime-improvements"
...
This reverts commit 0881e7c8c1
, reversing
changes made to 29a37e3585
.
2024-04-05 14:45:49 +09:00
Dean Herbert
673882f64b
Merge pull request #27789 from smoogipoo/disable-discord-uri-macos
...
Disable Discord URI registration on macOS for now
2024-04-04 15:58:19 +08:00
Dan Balasescu
fb8fb4f34e
Disable Discord URI registration on macOS for now
2024-04-04 16:43:26 +09:00
Dean Herbert
6ec5c1d7e2
Merge pull request #27788 from smoogipoo/update-framework
...
Update osu.Framework package
2024-04-04 15:18:23 +08:00
Dean Herbert
10d1308a0a
Update resources
2024-04-04 15:05:59 +08:00
Dan Balasescu
cd474de61f
Update osu.Framework package
2024-04-04 15:55:05 +09:00
Dean Herbert
9521c1e3e4
Update hit error metre to use new icons
...
- [ ] Depends on https://github.com/ppy/osu-resources/pull/317 .
2024-04-04 14:31:40 +08:00
Dean Herbert
2e1d63a1c2
Merge pull request #27783 from 64ArthurAraujo/add-invalid-beatmap-hash-custom-message
...
Add custom message in the case of a `invalid beatmap_hash`
2024-04-04 10:47:22 +08:00
Arthur Araujo
9e92ebaa43
Make message more general
...
Co-authored-by: Walavouchey <36758269+Walavouchey@users.noreply.github.com>
2024-04-03 19:15:22 -03:00