Commit Graph

69 Commits

Author SHA1 Message Date
75b8f2535f Move updatePathTypes to PathControlPointPiece
Here we produce a local bound copy of the path version, and bind it to update the path type.

This way, if the path version updates (i.e. any control point changes type or position), we check that all control points have a well-defined path.

Additionally, if the control point piece is disposed of, the GB should also swoop up the subscription because of the local bound copy.
2021-03-31 20:09:56 +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
b547abafb2 Fix slider right click context menus not being shown outside the valid playfield area
Closes #10816.
2020-11-20 13:46:23 +09:00
2d66423fbd Simplify inequality 2020-11-17 23:04:38 +01:00
c8fb49d540 Apply suggestions and remove redundant updateConnectingPath call 2020-11-17 22:23:46 +01:00
c6618f08aa Fix slider control point connections not being updated 2020-11-16 21:26:08 +01:00
3f24fabb57 Add change handler support for contorl point deletion 2020-11-05 15:05:43 +09:00
242ec1ca82 Don't override PathControlPointVisualiser's positional handling
It turns out it was relying on this to deselect control points on
clicking away from them.
2020-11-05 14:58:30 +09:00
9f333ac58a Add the ability to delete slider control points using shift+right click
Closes https://github.com/ppy/osu/issues/10672.

In two minds about how this should be implemented but went in this
direction initially. The other way would be to add local handling of
Shift-Right Click inside PathControlPointPiece (which is already doing
mouse handling itself).
2020-11-03 20:46:33 +09:00
9a65aa18d7 Fix connections hidden due to overlapping controlpoints 2020-04-13 16:13:14 +09:00
7b2f58eb30 Apply OnRelease method signature refactorings 2020-01-22 13:22:34 +09:00
42a01a10dd Merge remote-tracking branch 'upstream/master' into separate-path-connection 2019-12-11 20:14:29 +09:00
47b6b01739 Rename class to signify it is a drawable 2019-12-11 20:11:48 +09:00
d027c982e4 Merge branch 'master' into separate-path-connection 2019-12-11 19:03:14 +09:00
23959f3a3c Move control point removal to SliderSelectionBlueprint 2019-12-11 18:58:35 +09:00
d82ba3e7f7 Curve -> Curve type 2019-12-11 18:20:28 +09:00
03040d1750 Don't show inherit menu item when first control point is selected 2019-12-11 18:18:16 +09:00
97ca2e2753 Add missing bezier option to menu 2019-12-11 17:58:59 +09:00
cda6757f52 Merge branch 'master' into path-type-menuitem 2019-12-11 16:43:55 +09:00
f515bdc599 Merge branch 'master' into separate-path-connection 2019-12-11 16:43:04 +09:00
42a979b495 Merge branch 'master' into path-visualiser-refactor 2019-12-10 16:33:45 +09:00
5f3d163772 Merge branch 'master' into segmented-slider-paths-3 2019-12-10 14:28:20 +09:00
e9ec6591a9 Separate path connections from control points 2019-12-10 11:20:08 +09:00
1db218f908 Don't show count when deleting only one control point
Reads better.
2019-12-10 03:29:25 +09:00
9b318d2869 Add right-click menu item to change path type 2019-12-10 00:07:07 +09:00
bd2b0af269 Consider having only 1 control point as being deleted 2019-12-09 22:36:19 +09:00
af90b45c40 Refactor path visualisers to use bindables 2019-12-09 20:49:59 +09:00
883d5bc11d Remove automatic slider path offsetting 2019-12-09 17:54:22 +09:00
53f7c753fb General cleanups 2019-12-09 16:44:19 +09:00
4905709ea4 Remove unused usings 2019-12-09 13:19:21 +09:00
52dd7bf716 Fix deleting the first control point not working 2019-12-06 18:49:14 +09:00
16f8341a02 Handle control point positional updates within SliderPath 2019-12-06 18:49:01 +09:00
2b5f9515de Fix multiple control point deletions 2019-12-06 17:03:54 +09:00
247609388f Clean up unused/unnecessary properties 2019-12-06 12:31:22 +09:00
3ebbf62b2a Initial game-wide update with the new SliderPath 2019-12-05 19:53:36 +09:00
818553027b Merge branch 'master' into sharpen 2019-11-21 23:42:46 +08:00
031b686ee9 Only accept input while hovered 2019-11-13 17:38:34 +09:00
d835def4ab Add point count to the menu item text 2019-11-13 17:36:46 +09:00
c56503ee88 Select single control point on right click 2019-11-13 17:29:00 +09:00
9c52d239b4 Return null to allow passthrough 2019-11-13 16:56:48 +09:00
4c0bca5c07 Add ability to delete control points via right-click option 2019-11-13 16:38:15 +09:00
7d7b9e36b2 Use compound assignment. 2019-11-12 19:53:50 +08:00
7c20a589f2 Remove unused usings 2019-11-05 13:56:21 +09:00
c8beb5296f Use PlatformAction.Delete instead of Delete key 2019-11-05 13:26:44 +09:00
08bd811326 Merge remote-tracking branch 'origin/master' into slider-controlpoint-deletion
# Conflicts:
#	osu.Game.Rulesets.Osu/Edit/Blueprints/Sliders/Components/PathControlPointVisualiser.cs
2019-11-05 13:07:06 +09:00
bcf8a6d514 Fix slider creation regressing with path selection changes 2019-11-03 19:59:37 +09:00