Commit Graph

2891 Commits

Author SHA1 Message Date
70d5b616f2 Add scaling path type recovery 2021-03-29 15:49:49 +02:00
51f0477df4 Move bounding box logic to framework 2021-03-26 04:42:46 +01:00
ce9130ca50 Remove determinant limit
This has since been added into the framework through https://github.com/ppy/osu-framework/pull/4302
2021-03-25 17:38:55 +01:00
9df059b01d Add bounding box limit 2021-03-24 17:25:28 +01:00
e0240ab9d9 Increase exterior threshold 2021-03-24 05:55:34 +01:00
4ae3eaaac6 Move path type correction
This is better because `PathControlPointVisualizer` is local to the editor, meaning there is no chance that this could affect gameplay.
2021-03-24 03:02:19 +01:00
0bcd38e661 Simplify path type maintenance when dragging 2021-03-24 02:57:47 +01:00
5ee280f941 Update PointsInSegment when adding/removing points
There was a bug where if you created a slider, moved the last point, and then added a point such that it became a PerfectCurve, it would fail to recover after becoming a Bezier.

This fixes that.
2021-03-24 02:56:32 +01:00
067178e537 Maintain path type when dragging/placing 2021-03-22 15:59:06 +01:00
e67c759eef Mark control point visualiser as possibly-null 2021-03-19 22:44:31 +01:00
9634560d4b Fix control point visualiser crashing after deselections
`SliderSelectionBlueprint.OnDeselected()` would expire the
`ControlPointVisualiser` on deselection, leading to its removal from the
blueprint and eventual disposal, but still kept a separate reference to
said visualiser in another field.

This could lead to that stale reference to a disposed child getting read
in `ReceivePositionalInputAt()`, crashing quite a ways down over at the
framework side on futilely trying to compute the bounding box of a
drawable with no parent.
2021-03-19 21:40:22 +01:00
27c38db14d Add tooltips for slider path nodes which aren't inheriting 2021-03-19 16:58:08 +09:00
5f31304d05 Give each type of slider path type a unique colour to help visually distinguish them 2021-03-19 14:00:26 +09:00
7fa5fd5647 Update usages of config with framework changes 2021-03-17 16:10:16 +09:00
e3813ab828 Merge branch 'master' into legacy-spinner-placements 2021-03-12 19:47:34 +09:00
ea9b48d17d Remove unused using 2021-03-11 21:21:48 +09:00
b5bdf235ca Slightly improve comments more 2021-03-11 21:21:44 +09:00
8bc494b224 Adjust explanatory comments 2021-03-11 20:57:00 +09:00
1591d593e2 Move spin start time to inside result and switch to standard state handling 2021-03-11 12:58:15 +09:00
3f34981664 Fix incorrect spinner top offset calculation with clarification 2021-03-11 05:40:20 +03:00
4e8bcc9265 Fix SPM counter decreasing after spinner has already been completed 2021-03-09 16:54:46 +09:00
5a6864eb78 Fix SPM counter immediately disappearing on completion of spinners 2021-03-09 16:54:46 +09:00
0549395869 Inline "legacy coordinates container" and add "spinner Y centre" const 2021-03-09 08:55:32 +03:00
bb79da1aac Correct playfield shift counteract comment
Co-authored-by: Bartłomiej Dach <dach.bartlomiej@gmail.com>
2021-03-09 00:34:00 +03:00
0ad3073c1a Use MathF utility class instead
Co-authored-by: Berkan Diler <b.diler@gmx.de>
2021-03-07 21:21:44 +03:00
fbfaa378fc Move spinner top offset constant outside 2021-03-07 20:47:16 +03:00
413cbb30a0 Reword playfield shift counteract comment
Co-authored-by: Bartłomiej Dach <dach.bartlomiej@gmail.com>
2021-03-07 13:39:46 +03:00
1525480e73 Demonstrate value of SPINNER_TOP_OFFSET to being more sensible 2021-03-06 19:18:40 +03:00
5b6018295d Merge branch 'master' into diffcalc/skill-mods 2021-03-04 13:06:26 +09:00
d528ef426f Reposition legacy spinner components in-line with osu!stable 2021-02-28 22:59:22 +03:00
41b43dd39a Add nested legacy-simulating coordinates container 2021-02-28 21:32:56 +03:00
8004652d27 Merge branch 'master' into optimised-has-flag-checks 2021-02-25 21:24:39 +09:00
90e8308716 Merge branch 'master' into localisation-refactor-framework 2021-02-25 17:28:55 +09:00
dff1d80f39 Update HasFlag usages to HasFlagFast 2021-02-25 15:38:56 +09:00
0c202e0777 Merge branch 'master' into fix-selecting-incompatible-freemods 2021-02-24 19:24:04 +09:00
14160b897e Fix references to ModSuddenDeath 2021-02-24 14:44:53 +09:00
1fd76ea3fb Apply changes to UI components overriding functions with changing signatures 2021-02-22 19:01:42 +09:00
ca92ad715a Add test 2021-02-22 13:32:54 +09:00
66643a97b0 Add a list of mods to Skill class
Although this isn't necessary for existing official rulesets and calculators, custom calculators can have use cases for accessing mods in difficulty calculation.
For example, accounting for the effects of visual mods.
2021-02-20 20:37:44 +11:00
d3f0c0730d Merge branch 'master' into non-concurrent-sample-playback 2021-02-12 17:22:15 +09:00
cb4d119f23 Merge pull request #11740 from peppy/editor-fix-slider-samples-new-placement
Fix newly placed sliders in the editor not playing hitsounds
2021-02-12 13:31:00 +09:00
f84ea30637 Expose Mods in DrawableRuleset to avoid using external DI 2021-02-11 17:47:29 +09:00
aaa0362b12 Merge branch 'master' into autoplay-rate-independence 2021-02-11 17:39:45 +09:00
db79080bc4 Fix GetNodeSamples potentially returning a live reference and overwriting existing samples 2021-02-11 17:14:49 +09:00
4710a12774 Merge branch 'master' into classic-mode-flags 2021-02-11 16:21:55 +09:00
5d1d6ec1cb Fix inverted calculation 2021-02-10 22:09:24 +09:00
20a6405fd2 Add explanatory comments + const 2021-02-10 22:06:19 +09:00
bd2486e5a0 Fix grammatical error in comment 2021-02-10 21:27:12 +09:00
1d425b8322 Simplify case 2021-02-10 21:25:31 +09:00
4a391ce03d Fix div-by-0 when 0 ticks are hit 2021-02-10 21:24:41 +09:00