1
0
mirror of https://github.com/ppy/osu.git synced 2024-11-16 06:27:27 +08:00
Commit Graph

70267 Commits

Author SHA1 Message Date
Bartłomiej Dach
232301f27d
Terminate existing anchor create operation when object is deselected 2024-10-15 12:54:11 +02:00
Bartłomiej Dach
1a7acbac33
Terminate existing anchor drag operations when object is deselected 2024-10-15 12:53:40 +02:00
Bartłomiej Dach
2d0e98c951
Add failing test case for second tool switching failure case
Not exact (doesn't reproduce the crash), but will do.
2024-10-15 12:53:38 +02:00
Bartłomiej Dach
0cd352cdcb
Add failing test case for first tool switch failure scenario 2024-10-15 12:39:08 +02:00
Salman Alshamrani
0fa1d22210 Add tests covering expected behaviour when going on/off compose area 2024-10-14 15:12:09 -04:00
Salman Alshamrani
14c6cfc365 Extend test cases to cover direct interaction (i.e. no touch-compose-area-first) 2024-10-14 15:12:09 -04:00
Salman Alshamrani
e151c0fab1 Fix coding mistake 2024-10-14 15:11:44 -04:00
Jonas Brandstötter
a007a81fe8
Only keep track of the names of hit results to display in a ScoreTable 2024-10-14 16:55:07 +02:00
Jonas Brandstötter
285756802c
Sum up totals for hit results with the same name 2024-10-14 16:33:30 +02:00
Jonas Brandstötter
511f0e99b3
Correct typo 2024-10-14 16:16:00 +02:00
Jonas Brandstötter
25c0ff4168
Correct reference to hit result and link to them 2024-10-14 16:14:29 +02:00
schiavoanto
d7021f989b Revert 9cd7f2b and fc1ebfd 2024-10-14 16:14:23 +02:00
Jonas Brandstötter
750e0b29ca
Use ChildrenOfType<> to get ScoreTable to test 2024-10-14 16:11:58 +02:00
Dean Herbert
790f863e06
Merge pull request #30277 from bdach/fix-shuffle-more
Fix shuffle not actually changing the track sometimes
2024-10-14 23:09:13 +09:00
Leander Furumo
035e5a9613
migrate clearance of conflicting ToggleFPSDisplay keybind 2024-10-14 16:03:29 +02:00
Salman Alshamrani
40b98d4863 Move conditional 2024-10-14 09:55:24 -04:00
Bartłomiej Dach
8cec318c1f
Loop track even if shuffling if there is only one available
Co-authored-by: Dean Herbert <pe@ppy.sh>
2024-10-14 15:28:32 +02:00
Bartłomiej Dach
59655a1830
Fix flip operations sometimes not preserving selection
Closes https://github.com/ppy/osu/issues/30250.
2024-10-14 14:50:01 +02:00
Bartłomiej Dach
98141430b0
Add failing tests 2024-10-14 14:48:07 +02:00
Bartłomiej Dach
1f1a174c50
Remove no longer required nested object reposition hacks
As touched on in
https://github.com/ppy/osu/issues/30237#issuecomment-2408557766, these
types of maneouvers are no longer required after
https://github.com/ppy/osu/pull/30021 - although as it turns out on
closer inspection, these things being there still *did not actually
break anything*, because the `slider.Path` mutation at the end of
`modifySlider()` causes `updateNestedPositions()` to be called
eventually anyway. So this is at mostly a code quality upgrade.
2024-10-14 14:26:27 +02:00
Bartłomiej Dach
275b86cd3c
Fix strict tracking mod not populating path progress for ticks/repeats
Closes https://github.com/ppy/osu/issues/30237.

This is the root failure in the issue, and one that *only* presents when
another conversion mod that repositions the objects is also active. That
makes the `PathProgress` of the nesteds to be zero, therefore making
them occupy the position of the slider head after any mutation to the
path.
2024-10-14 14:26:27 +02:00
Bartłomiej Dach
945d907a3d
Add test covering correct operation of mirror mod 2024-10-14 14:21:00 +02:00
Bartłomiej Dach
17aed26f85
Fix shuffle not actually changing the track sometimes
See https://github.com/ppy/osu/pull/30215#issuecomment-2407775408 for
context.

Turns out the test failures were more correct than I'd thought. The
long-and-short of it is that both in "pure random" mode and in
"permutation" mode, when running out of track history to fall
back on, it was possible for the random algorithm to pick the same song
twice in a row - which is probably not desired, and which this explicit
exclude should make impossible.
2024-10-14 13:18:45 +02:00
Bartłomiej Dach
639eb7ae4a
Merge pull request #30274 from shinmorisawa/master
Respect "Prefer metadata in original language" toggle in beatmap deletion dialog
2024-10-14 12:02:03 +02:00
Shin Morisawa
8ff86d560f
Merge branch 'master' into master 2024-10-14 17:48:04 +09:00
Bartłomiej Dach
120eb9adea
Merge pull request #30261 from peppy/faster-quick-retry
Decrease the time it takes to quick retry by ~650 ms
2024-10-14 10:24:35 +02:00
Shin Morisawa
0664b185c3
Merge branch 'master' into master 2024-10-14 16:46:44 +09:00
Bartłomiej Dach
cd59b93cc9
Merge branch 'master' into feat/optimize-polygon-tool 2024-10-14 08:28:30 +02:00
Bartłomiej Dach
74675c8c78
Merge pull request #30080 from OliBomby/clamp-scale2
Clamp scale with lower and upper bounds
2024-10-14 08:17:22 +02:00
Shin Morisawa
90a08b8a68 Fix Beatmap Delete Dialog 2024-10-14 09:55:28 +09:00
Leander Furumo
379794c462
replace empty keybinding array with input key type of None 2024-10-14 00:00:45 +02:00
Leander Furumo
0e768cc517
remove default keybind for toggling fps counter 2024-10-13 22:50:58 +02:00
Dean Herbert
f8bce706fa
Merge pull request #30239 from rozukke/reset-offsets
Add functionality to mass reset offsets
2024-10-14 01:56:14 +09:00
Leander Furumo
8f1fbb44c4
change fps toggle keybind 2024-10-13 17:36:06 +02:00
Dean Herbert
53671ad11e
Only update beatmaps which actually had offsets
Without this every beatmap gets a write and it reloads the whole of song
select, basically.
2024-10-14 00:35:18 +09:00
Salman Alshamrani
f8c8184c5c Fix placement blueprints not receiving latest mouse position with touch input 2024-10-13 11:01:18 -04:00
Salman Alshamrani
86b240d131 Add failing test cases 2024-10-13 11:01:17 -04:00
Dean Herbert
868a7db9e9
Start preparing player earlier when quick retrying
Should help with https://github.com/ppy/osu/issues/9039.
2024-10-13 22:29:00 +09:00
Dan Balasescu
9681e3ac46
Overwrite downloaded data packages
In https://github.com/ppy/osu/actions/runs/11311858931/job/31458581002, I cancelled the run during the download from `data.ppy.sh`.

In https://github.com/ppy/osu/actions/runs/11313128285/job/31461534857, `wget` skipped downloading the file due to the `-nc` option (no-clobber), i.e.: if the file exists, don't re-download.

The only way I'm aware of to resolve this with wget is to either use `-c` (continue), which may lead to broken files, or to explicitly specify the output file via `-O`. Thought I'd clean up a few pieces in the process.

Why not curl? Mostly historical - some distros don't come with curl. It may be okay now but there's probably no point changing this at the moment...
2024-10-13 18:43:02 +09:00
Dan Balasescu
9abca53829
Merge pull request #30246 from Joehuu/recommend-csdevkit-instead
Recommend C# Dev Kit extension on VSCode
2024-10-13 15:02:03 +09:00
finadoggie
6bcfed8963
Revert "remove score debugging code I accidentally left in"
This reverts commit 5192599543.
2024-10-12 16:53:32 -07:00
finadoggie
5192599543
remove score debugging code I accidentally left in 2024-10-12 16:45:34 -07:00
finadoggie
88af57818c
only assign countLargeTickMiss for slider accuracy scores
helps indicate it should only be used for slider acc scores
2024-10-12 15:36:42 -07:00
finadoggie
29b1697a70
consolidated if statements for getting effectiveMissCount and countSliderEndsDropped 2024-10-12 15:34:04 -07:00
finadoggie
3ac6a9f0ae
Join variable assignments with declarations 2024-10-12 15:30:02 -07:00
Jonas Brandstötter
dcd3e5194e
Group HitResults with the same name into one column in beatmap ranking
Closes #29911
2024-10-12 22:53:08 +02:00
Joseph Madamba
e1e8e39a89
Recommend C# Dev Kit extension on VSCode 2024-10-12 12:12:40 -07:00
Artemis Rosman
e4f9c861ba Add functionality to mass reset offsets 2024-10-13 00:50:45 +11:00
Dan Balasescu
2de1955892
Merge pull request #30218 from bdach/daily-challenge-conclusion-offline
Do not show daily challenge conclusion notification on disconnection
2024-10-12 18:14:22 +09:00
finadoggie
3b517e03aa
Convert estimateSliderEndsDropped assignment into '?:' expression
I would just like to say that I don't know why anyone would ever want this but github told me to do it
2024-10-12 01:08:39 -07:00