806 Commits

Author	SHA1	Message	Date
Dan Balasescu	7bc8908ca9	Partial everything	2022-11-27 00:00:27 +09:00
Dean Herbert	76e9382e0b	Fix triangles showing on ExpandableButtons when in contracted state	2022-11-25 20:18:35 +09:00
Derrick Timmermans	65e5c9a3ae	Dispose DrawableRulesetDependencies in HitObjectComposer	2022-11-25 08:27:22 +01:00
Derrick Timmermans	7b620392fe	Provide DrawableRulesetDependencies from HitObjectComposer	2022-11-24 13:13:45 +01:00
Dean Herbert	ada039151b	Add the ability to toggle off hit marker displays in the editor	2022-11-08 18:07:06 +09:00
Dean Herbert	e02c477008	Merge pull request #21017 from andy840119/remove-nullable-disable-in-the-checks ... Remove nullable disable in the checks.	2022-11-06 18:39:03 +09:00
Dean Herbert	29bc653d24	Remove incorrect double multiplication and add missing test coverage	2022-11-01 17:17:54 +09:00
Dean Herbert	2a88409dfe	Fix time snap of sliders not matching when SV is not 1.0x ... This regressed with https://github.com/ppy/osu/pull/20850 because the function was used in other places which expect it to factor slider velocity into the equation. Rather than reverting, I've added a new argument, as based on the method naming alone it was hard to discern whether SV should actually be considered. The reason for the change in #20850 was to avoid the SV coming in from a reference object which may not have a correct SV in the first place. In such cases, passing `false` to the function will give the expected behaviour.	2022-11-01 15:12:58 +09:00
andy840119	a1a9238bd1	Use empty string instead of null because issue template not accept null.	2022-10-30 16:31:07 +08:00
andy840119	505ec800da	File name should be nullable.	2022-10-30 16:29:54 +08:00
andy840119	db48a57fa7	Remove nullable disable annotation in some checks class.	2022-10-30 16:28:49 +08:00
andy840119	24c27e62f6	Remove nullable disable annotation in the issue and issue template-related class.	2022-10-30 16:25:15 +08:00
Dean Herbert	6b53ea3400	Enable distance snapping when DS value is changed via user interaction	2022-10-26 13:37:01 +09:00
Dean Herbert	7d59bc6e2f	Merge branch 'grid-momentary-shortcuts' into read-current-distance-snap	2022-10-26 13:28:04 +09:00
Dean Herbert	54ae16badc	Move distance snap toggle button implementation to DistancedHitObjectComposer	2022-10-26 13:27:23 +09:00
Dean Herbert	8d9a85e9e5	Fix typos in xmldoc ... Co-authored-by: Bartłomiej Dach <dach.bartlomiej@gmail.com>	2022-10-26 13:14:12 +09:00
Bartłomiej Dach	48057412f5	Merge branch 'master' into grid-momentary-shortcuts	2022-10-25 22:17:08 +02:00
Dean Herbert	2b850694fa	Merge branch 'allow-distance-grid-snap' into grid-momentary-shortcuts	2022-10-25 14:10:45 +09:00
Dean Herbert	d0e6bda9ef	Stop HitObjectComposer from handling Shift+Number keys	2022-10-24 15:19:36 +09:00
Dean Herbert	f516e32949	Improve UI a bit	2022-10-21 23:35:53 +09:00
Dean Herbert	bb921ff9a7	Fix incorrect current DS value when inside a slider	2022-10-21 23:35:47 +09:00
Dean Herbert	f6de366766	Combine display and button into one control	2022-10-21 23:21:07 +09:00
Dean Herbert	645a84b1d6	Display current distance snap and add button to update to use it	2022-10-21 22:58:36 +09:00
Dean Herbert	7dc03097ff	Change distance snap to never account for slider velocity ... This is a nuanced detail that was implemented incorrectly from the outset. When mapping, generally a mapper chooses the distance spacing with no regard to the SV. It has always been common to have a lower or higher distance spacing than SV, but with the way the lazer editor has worked, the SV was multiplied into the distance snap grid display, incorectly changing its spacing depending on the "reference object" (which is usually the previous hitobject chronologically).	2022-10-21 21:51:24 +09:00
Dean Herbert	6cdfddea62	Always enable distance spacing when adusting distance space multiplier	2022-10-21 16:10:55 +09:00
Dean Herbert	b9f41611a7	Fix bank potentially being overwritten during placement	2022-10-19 21:48:18 +09:00
Dean Herbert	50e24ddd87	Add icon and radio button logic	2022-10-19 21:35:08 +09:00
Dean Herbert	860214c22a	Adjust paddings to feel better now that backgrounds are visible of toolboxes	2022-10-18 14:38:45 +09:00
Dean Herbert	6a7f6c76ff	Fix osu! editor padding being applied to toolboxes ... The result of https://github.com/ppy/osu/pull/20739 and https://github.com/ppy/osu/pull/20738 having unintended side effects when applied together.	2022-10-18 14:32:00 +09:00
Dean Herbert	ccbac08985	Merge pull request #20593 from ekrctb/catch-editor-height ... Allow arbitrary height catch editor	2022-10-18 12:10:49 +09:00
Dean Herbert	32c3e35762	Change background colour to a lighter tint to avoid clash with slider bars	2022-10-14 19:38:27 +09:00
Dean Herbert	4d99c7002b	Add background behind editor toolbox groups	2022-10-13 16:36:19 +09:00
ekrctb	9247ff3e0a	Allow changing scrolling speed in catch editor ... Scroll speed is not saved and doesn't affect gameplay. It is purely a feature for a better visualization. It is currently bind to scroll speed increase/decrease. Default F3/F4 crashes with editor shortcuts so it has to be changed.	2022-10-06 18:06:16 +09:00
Dean Herbert	ac99c1ad69	Migrate the majority of existing file lookups to use new extension methods	2022-08-10 16:01:16 +09:00
Dean Herbert	31a447fda0	Update parameter discards	2022-06-24 21:26:19 +09:00
Bartłomiej Dach	26c5b59f6d	Replace usages of string.To{Lower,Upper}()	2022-06-24 11:57:45 +02:00
Dean Herbert	bd5037fcad	Fix new hitobject placements not taking on the existing sample settings ... Similar to velocity / difficulty points, a user expectation is that volume and sample settings will be transferred to newly placed objects from the most recent one. Closes https://github.com/ppy/osu/issues/18742.	2022-06-18 13:49:59 +09:00
Dan Balasescu	f8830c6850	Automated #nullable processing	2022-06-17 16:37:17 +09:00
Bartłomiej Dach	e511c1dfff	Merge branch 'master' into use-normalised-precise-scrolling	2022-05-27 16:55:13 +02:00
Dean Herbert	17206bbebf	Update screen padding and colouring to better match new designs	2022-05-24 18:43:22 +09:00
Susko3	88217e0c98	Adjust ScrollDelta usages to account for normalised IsPrecise values	2022-05-23 20:22:27 +02:00
Dan Balasescu	64a371638e	Merge pull request #18236 from peppy/snap-unification ... Add `enum` to snap method as alternative to multiple nested invocations	2022-05-20 11:35:05 +09:00
Dean Herbert	b3d6f76cfa	Add "None" snap type to fix flags not working correctly	2022-05-19 15:57:17 +09:00
Dean Herbert	c63e65c677	Don't hide SelectionBlueprints themselves, only their children	2022-05-12 19:26:13 +09:00
Dean Herbert	c0abce918f	Add enum to snap method as alternative to mutliple nested invocations	2022-05-12 16:19:07 +09:00
Salman Ahmed	144d33f0d2	Merge branch 'master' into distance-snapping-test	2022-05-06 17:29:27 +03:00
Dean Herbert	c6bc6be128	Fix toolbox expand being interrupted by gaps between groups	2022-05-06 19:06:40 +09:00
Dean Herbert	5029710de7	Merge branch 'master' into distance-snapping-test	2022-05-05 22:00:40 +09:00
Dan Balasescu	39d10487b9	Merge branch 'master' into snapping-tidy-distance	2022-05-05 19:55:55 +09:00
Dean Herbert	e83604a1fc	Merge branch 'master' into cache-at-interface	2022-05-05 18:54:24 +09:00
Dean Herbert	7b71fb860b	Expose DistanceSpacingMultiplier for test usage	2022-05-05 18:49:12 +09:00
Dean Herbert	947a68006a	Add note about IDistanceSnapProvider not multiplying DistanceSpacing itself	2022-05-05 17:50:17 +09:00
Dean Herbert	c3d2648f85	Reword weird xmldoc ... Co-authored-by: Bartłomiej Dach <dach.bartlomiej@gmail.com>	2022-05-05 17:07:05 +09:00
Dean Herbert	4226583afd	Merge branch 'cache-at-interface'	2022-05-05 16:52:03 +09:00
Dean Herbert	19cf9dc20b	Merge branch 'snapping-tidy-distance'	2022-05-05 16:51:32 +09:00
Dean Herbert	5a1ac71d90	Remove unnecessary type specification in HitObjectComposer's caching	2022-05-05 16:44:37 +09:00
Dean Herbert	1c6a233cc0	Move snap provider caching to interfaces	2022-05-05 16:44:37 +09:00
Dean Herbert	b411b59006	Move IPlacementHandler caching to interface	2022-05-05 16:43:02 +09:00
Dean Herbert	977e6d8a80	Add xmldoc for IDistanceSnapProvider and related properties	2022-05-05 16:25:45 +09:00
Dean Herbert	1fce0da331	Reword slightly, to allow better conformity with IDistanceSnapProvider	2022-05-05 16:04:34 +09:00
Dean Herbert	f6fc926f1a	Add xmldoc and rename methods in IPositionSnapProvider for legibility	2022-05-05 15:58:21 +09:00
Bartłomiej Dach	bf80dd864c	Merge branch 'master' into fix-editor-toolbox-click-through	2022-05-04 11:57:32 +02:00
Dean Herbert	4e0f899159	Rename value changed variable	2022-05-04 18:13:30 +09:00
Salman Ahmed	f5d4f02200	Use ToUpper for key binding text	2022-05-04 11:59:29 +03:00
Salman Ahmed	0b8fd2e39f	Improve distance spacing toast inline with key binding changes	2022-05-04 11:43:19 +03:00
Dean Herbert	b325f0ee0b	Combine editor toolbox container implementation and fix input blocking ... Until now, toolbox scroll areas would block input from arriving behind them, even when no visible element was clicked. In addition, clicking on a button inside a toolbox would still send a `MouseDown` event to things behind it. Specifically, the editor's `HitObjectComposer` would receive these events and also place objects when the user does not expect them to be placed. This fixes another regression that occurred due to `ScrollContainer`s no longer blocking input theirselves.	2022-05-04 17:41:30 +09:00
Salman Ahmed	3d2072498e	Merge branch 'master' into distance-spacing-osd	2022-05-04 11:41:12 +03:00
Dean Herbert	a66743266f	Remove unused ScrollingToolboxGroup class	2022-05-04 17:20:58 +09:00
Salman Ahmed	504ca5be31	Fix scrolling no longer adjusting distance spacing by amount	2022-05-04 09:52:33 +03:00
Salman Ahmed	813d6fed48	Split activation keybind to separate increase/decrease keybinds	2022-05-04 09:00:54 +03:00
Salman Ahmed	0dd2e1652c	Mark OnScreenDisplay dependency as nullable	2022-05-03 11:15:28 +03:00
Salman Ahmed	b8287f3687	Display toast notification on editor distance spacing change	2022-05-03 10:30:32 +03:00
Salman Ahmed	521ec1a225	Add keybind for distance grid spacing activation	2022-05-03 10:15:24 +03:00
Salman Ahmed	fef94d49f4	Revert "Convert data type of DistanceSpacing to float" ... This reverts commit 7aaa88cac2.	2022-04-29 08:02:07 +03:00
Salman Ahmed	abb88b0907	Move distance spacing application to DistanceSnapGrid	2022-04-28 10:54:38 +03:00
Salman Ahmed	7aaa88cac2	Convert data type of DistanceSpacing to float	2022-04-28 10:24:36 +03:00
Salman Ahmed	2e8372fe90	Minor cleanup ... Just from reading diff.	2022-04-28 07:14:43 +03:00
Salman Ahmed	bfc6bfc91b	Let CatchHitObjectComposer inherit from DistancedHitObjectComposer and hide control	2022-04-28 06:44:50 +03:00
Salman Ahmed	835898dd30	Introduce DistancedHitObjectComposer and supersede OsuToolboxComposite	2022-04-28 06:44:50 +03:00
Salman Ahmed	59cf3ff50f	Move distancing methods from IPositionSnapProvider to IDistanceSnapProvider	2022-04-28 05:48:45 +03:00
Salman Ahmed	8ed39009fd	Encapsulate distance spacing control handling to a "distance toolbox composite" ... Encapsulated in a way which can allow further extensibility for the right-side area of toolboxes.	2022-04-24 08:33:03 +03:00
Salman Ahmed	70a93c0e7e	Move DistanceSpacingMultiplier bindable to its own IDistanceSnapProvider interface	2022-04-24 08:33:00 +03:00
Salman Ahmed	4f8f27a58b	Merge branch 'master' into osu-distance-spacing	2022-04-24 05:23:30 +03:00
Dean Herbert	ea9495eb74	Update all existing calls to extension method with correct fallback handling	2022-03-29 16:51:30 +09:00
Dean Herbert	8d1ee28e67	Add settings modification UI to skin editor	2022-03-13 17:03:25 +09:00
Salman Ahmed	0992bec2c8	Change "distance spacing" multipler type to double ... Avoids losing precision on initial load, causing an unnecessary hash change in `EditorChangeHandler`. Resolves test failures in `TestSceneEditorChangeStates` (https://github.com/ppy/osu/runs/5192493482?check_suite_focus=true).	2022-02-16 03:28:12 +03:00
Salman Ahmed	868dcd20f5	Remove e.ShiftPressed handling for now ... Broken on macOS, will be handled differently later on as discussed.	2022-02-15 02:21:53 +03:00
Salman Ahmed	528dc03b8c	Expose distance spacing in IPositionSnapProvider for updating distance grid ... Alternate method is to expose a `SnapDistancesChanged` event in `IPositionSnapProvider` instead, but I chose this way as an analogue to `IBeatSnapProvider.BeatDivisor`, which might even make sense to be exposed as `BindableBeatDivisor` instead of caching that separately.	2022-02-15 02:21:53 +03:00
Salman Ahmed	19ee05c232	Add "distance spacing" multiplier for osu! ruleset ... While osu!catch also implements a distance snap grid, it doesn't rely on `GetBeatSnapDistanceAt` (unlike osu!), therefore it can't have the "distance spacing" multiplier yet.	2022-02-15 02:21:53 +03:00
Dean Herbert	b9d9fc56af	Move files to UI namespace	2022-02-14 17:51:39 +09:00
Dean Herbert	51251e3204	Fix CI reported warnings	2022-01-12 22:39:00 +09:00
Dean Herbert	eb70a1eeb7	Replace compatibility properties with direct references	2022-01-12 18:13:14 +09:00
Dean Herbert	3811bd8520	Fix some null inspections	2022-01-12 17:00:16 +09:00
Dean Herbert	cea9cab4dc	Use ExpandingButtonContainer in editor composer	2022-01-06 21:10:45 +09:00
Dean Herbert	ef2a4aed9a	Fix editor playfield not being centered correctly ... This has come up multiple times, with mappers citing that they have muscle memory for mapping based on the centre of the playfield being in the centre of the window. The original plan was to have a second toolbar on the right hand side of the screen to balance the padding, but we're not at that point yet. Easiest solution is to do what stable does and allow the left-hand toolbar items to overlap the playfield underneath it. In edge cases where the user is running at an aspect ratio that causes overlaps, they can choose to collapse the toolbars down. We can probably work on this UI/UX a bit more as we update designs to be more friendly to such cases.	2022-01-05 16:05:18 +09:00
Bartłomiej Dach	5cbaa028eb	Use extracted ruleset config cache implementation in DI	2021-12-23 19:02:10 +01:00
Dean Herbert	59e763467f	Move StoragePath implementation to an extension method	2021-11-19 16:08:38 +09:00
Naxesss	b888185799	Properly dispose of Stream in bg quality check	2021-11-10 05:06:11 +01:00
Dean Herbert	e43d91ad5d	Fix another case of incorrect null checking in editor verification processing	2021-11-04 15:18:10 +09:00
Dean Herbert	1e73b09e57	Fix another couple of cases of incorrect string null/empty checking	2021-11-04 14:22:28 +09:00
Dean Herbert	6944151486	Apply batch fixing of built-in types using var	2021-10-27 13:04:41 +09:00
smoogipoo	aa380a11c1	Merge branch 'master' into no-more-difficulty-control-points-info	2021-10-14 12:02:10 +09:00
Bartłomiej Dach	c7db286b94	Merge branch 'master' into file-sanity-checks	2021-10-14 00:17:29 +02:00
Dean Herbert	82f7f99f37	Refactor to use scoped using (and also correctly dispose TagLib portion)	2021-10-12 10:46:26 +09:00
Naxesss	f2f97602f2	Dispose of video resource later ... We need the Stream to stay open here because `StreamFileAbstraction` uses it later in the block.	2021-10-12 00:40:39 +02:00
Naxesss	47b4d03145	Dispose of Streams when used	2021-10-12 00:13:38 +02:00
Naxesss	ce69dd3588	Use Prescan flag for audio length check	2021-10-11 23:57:23 +02:00
Naxesss	8f093b9a11	Use TagLib instead of ManagedBass	2021-10-11 13:56:26 +02:00
Naxesss	f0cd18a721	Only check each video file once	2021-10-11 13:55:50 +02:00
Naxesss	6aa054b5fa	Change to non-BASS error template	2021-10-11 13:53:23 +02:00
smoogipoo	bc37cb6f43	Merge branch 'master' into no-more-difficulty-control-points-info	2021-10-08 18:41:17 +09:00
Dean Herbert	b339c149d8	Copy BaseDifficulty to Beatmap<T> and move all write operations across	2021-10-06 15:10:45 +09:00
Dean Herbert	a92d499d7a	Convert usages of BeatmapDifficulty to IBeatmapDifficultyInfo	2021-10-01 16:55:50 +09:00
Dean Herbert	693ecdf62a	Refactor some incorrect/outdated terminology	2021-09-06 21:32:43 +09:00
Dean Herbert	d4e5a612ea	Update IPositionalSnapProvider to take a HitObject as reference, rather than raw time ... This allows fetching the correct `DifficultyControlPoint` from the hitobject. Nothing more.	2021-09-03 17:11:12 +09:00
Dean Herbert	a3d9ab1e2e	Move approach rate to EffectControlPoint	2021-09-03 16:58:16 +09:00
Dean Herbert	b29209d13f	Ensure tool is always set back to select tool when beatmap becomes untimed	2021-07-19 17:08:40 +09:00
Dean Herbert	bde35d9f21	Rename radio button classes to be local to editor	2021-07-19 16:57:12 +09:00
Dean Herbert	eac9b1ec7e	Disable toolbox composition buttons when beatmap is not timed	2021-07-17 02:30:13 +09:00
Naxess	3a5912e35e	Add new checks to verifiers	2021-07-13 10:53:25 +02:00
Naxess	6da2a3d51f	Add zero-length objects check and tests	2021-07-13 10:50:11 +02:00
Naxess	dc4285582b	Fix misplaced "f" in issue message	2021-07-13 06:07:04 +02:00
Naxess	c8f58cbf6c	Add audio in video check and tests	2021-07-13 04:17:41 +02:00
Naxess	a4a1919842	Add too short audio files check and tests	2021-07-13 03:46:45 +02:00
Naxess	0a8fd01b99	Add zero byte check and tests	2021-07-13 03:45:21 +02:00
ekrctb	663ffae42f	Fix hit object selection blueprint potential null reference	2021-07-08 14:19:00 +09:00
Naxess	9f9e96ce9e	Add check for spanDuration <= 0 prior to division	2021-06-27 15:40:09 +02:00
Naxess	a4a5325b73	Improve acceptable difference for repeat edges ... Likelihood that `spanDuration` is greater than E+7 is quite low in any realistic case, so this should work fine.	2021-06-27 15:39:31 +02:00
Naxess	4cfa0ae5ec	Improve precision for repeat edges	2021-06-27 03:26:35 +02:00
Naxess	2cd7eda3c4	Add "or equal to" to volume threshold xmldocs	2021-06-27 02:30:12 +02:00
Naxess	0c0fd291d9	Order hitobjects by endtime	2021-06-27 01:25:03 +02:00
Naxess	4796b1b208	Use local variables for hasHitsound & couldHaveHitsound	2021-06-27 00:04:30 +02:00
Naxess	5bc08ebadb	Rename hasHitsounds -> mapHasHitsounds	2021-06-26 23:49:25 +02:00
Naxess	d29e6f4695	Add negligible template to PossibleTemplates	2021-06-26 23:49:06 +02:00
Naxess	1913084342	Use HitSampleInfo.AllAdditions instead of new list	2021-06-26 23:48:28 +02:00
Naxess	f78cc9397e	Factor out edge type logic	2021-06-26 20:45:31 +02:00
Naxess	51147405c5	Make || and && priority explicit	2021-06-26 20:44:39 +02:00
Naxess	4b436b774d	Add few hitsounds check	2021-06-26 19:20:46 +02:00
Naxess	d8117fa730	Add muted objects check	2021-06-26 19:20:34 +02:00
ekrctb	9dcd0bf311	Remove IPlayfieldProvider by caching Playfield	2021-06-17 10:15:24 +09:00
ekrctb	ef96ceb4ab	Introduce IPlayfieldProvider	2021-06-15 14:43:04 +09:00
Dean Herbert	8a76d97b63	Remove replay logic from DrawableRuleset (and implement in DrawableEditorRulesetWrapper)	2021-06-02 11:06:30 +09:00
Dean Herbert	bbfd7ea23f	Ensure RegenerateAutoplay is only run once per frame	2021-05-23 21:20:08 +09:00
Dean Herbert	6c4709e7b4	Fix PlacementBlueprint using the wrong beatmap when applying defaults ... Closes #12855.	2021-05-19 18:34:07 +09:00
Dean Herbert	a96603f025	Merge pull request #12782 from smoogipoo/rework-hitobject-blueprints ... Rename hitobject blueprints and tie them to HitObjects	2021-05-18 16:30:58 +09:00
smoogipoo	882d54a8f8	Remove now unnecessary Apply() method	2021-05-18 14:26:26 +09:00
smoogipoo	e621cfc4ea	Add Apply() method for applying new DHOs	2021-05-18 14:14:10 +09:00
smoogipoo	f1f3606fd0	Fix unresolved xmldocs	2021-05-18 13:11:58 +09:00
Dan Balasescu	9d423245d8	Fix up xmldocs ... Co-authored-by: Bartłomiej Dach <dach.bartlomiej@gmail.com>	2021-05-18 13:02:23 +09:00
Dean Herbert	fcb226bd20	Add local variable for regular access to HitObjects	2021-05-14 16:23:45 +09:00
Naxess	a447f20095	Fix formatting of #nullable enable	2021-05-14 03:38:35 +02:00
Naxess	90e0b3374e	Add #nullable enable ... Co-authored-by: Dean Herbert <pe@ppy.sh>	2021-05-14 03:34:50 +02:00
smoogipoo	ffb6135a1b	Rework hitobject blueprints to take in hitobject models	2021-05-13 19:53:32 +09:00
Naxess	19800f5f7f	Move IBeatmap arg into context	2021-05-13 11:24:22 +02:00
Naxess	b37cb3bdbe	Change interpreted difficulty from bindable to regular value ... There's no reason for why checks would need this to be bindable. A 1-directional binding is more appropriate.	2021-05-13 09:00:30 +02:00
Naxess	b7bc42e0d3	Rename "playableBeatmap" check arg to "beatmap" ... The working beatmap is now in the context, so it's easier to distinguish beatmap type, hence no need for this prefix.	2021-05-12 02:34:16 +02:00
Naxess	64d96b06a6	Add interpreted difficulty info to BeatmapVerifierContext ... Enables checks to make use of the difficulty level as shown in the settings UI.	2021-05-12 02:30:21 +02:00
Naxess	c13b93e6f1	Replace IWorkingBeatmap arg with BeatmapVerifierContext in checks ... This simplifies passing of contextual information by enabling addition without needing to refactor lots of classes. See next commit for example.	2021-05-12 02:29:18 +02:00
Naxess	75adec57eb	Remove negligible default hidden TODO	2021-05-12 01:31:16 +02:00
Salman Ahmed	b145f46a0a	Merge branch 'master' into skin-components-list	2021-04-30 21:10:55 +03:00
Dan Balasescu	e6eea73b8b	Merge branch 'master' into basic-compose-checks	2021-04-30 23:41:46 +09:00
Dean Herbert	a1e64f4e3c	Use the existing toolbox design	2021-04-30 14:41:06 +09:00
Dean Herbert	9c62c90cfc	Refactor SelectionBlueprint and MoveSelectionEvent to work in screen-space coordinates ... Until now, the implementation of the overrides in `SelectionBlueprint` have been confusing to the point where I would just implement by trial-and-error (or copying from an existing implementation). This was due to a combination of using "object" space coordinates (ie. the thing the `Blueprint` is operating on) and screen-space coordinates. This change switches all event related coordinates to screen-space, which is how we already handle rotation/scale operations. With the introduction of other editor types where the related objects are drawables, this also makes a lot more sense.	2021-04-29 16:10:42 +09:00
Dean Herbert	f3c7694eeb	Rename methods to match generally how these find-methods are named elsewhere	2021-04-28 16:57:52 +09:00
Dean Herbert	48d6c9ac4b	Move snap/divisor helper methods to inside ControlPointInfo	2021-04-28 16:47:30 +09:00
Naxess	200352b750	Rename unsnap check templates	2021-04-27 13:56:05 +02:00
Dean Herbert	7ec5ea1eb5	Remove hitobject terminology from base classes	2021-04-27 19:01:29 +09:00
Dean Herbert	dd3d8e5d03	Make SelectionHandler abstract to ensure things get implemented	2021-04-27 19:01:29 +09:00
Dean Herbert	f2e56bd306	Refactor editor selection/blueprint components to be generic	2021-04-27 19:01:29 +09:00
Dean Herbert	ec1c336b0a	Fix a couple of inspections	2021-04-27 13:23:14 +09:00
Naxess	7a6e9e5070	Change category of unsnap check to timing ... Makes more sense, as this is typically the result of timing changes.	2021-04-27 02:32:57 +02:00
Naxess	9e49ecb573	Remove unused virtual keywords ... Added these in a previous iteration, where I had the mania variant inherit this class. No longer necessary as `IHasColumn` was used to make this check more generic.	2021-04-27 02:23:06 +02:00
Naxess	a3c1b1fd52	Fix accessibility of areConcurrent	2021-04-27 01:24:38 +02:00
Naxess	ce258febf6	Rename CheckUnsnaps -> CheckUnsnappedObjects ... Will potentially have `CheckUnsnappedKiai` or similar later, so this is worth specifying. Also consistent with `CheckConcurrentObjects`, which will likely have a `CheckConcurrentLines` later.	2021-04-26 20:32:44 +02:00
Naxess	b9e4f73f78	Add concurrent objects check to BeatmapVerifier	2021-04-26 20:28:59 +02:00
Naxess	a3570e18dd	Add concurrent objects check ... Here we use `IHasColumn` to support rulesets with columns, and so I moved that interface out into `osu.Game` from `osu.Game.Rulesets.Mania`. We also use the same threshold as the unsnap check to ensure that no problems slip through. Specifically where an object is simultaneously not concurrent and not unsnapped but still on the same tick.	2021-04-26 20:17:18 +02:00
Naxess	9b9c473616	Remove redundant string formatting	2021-04-26 16:17:38 +02:00
Naxess	7b9ed924be	Rename snapping methods ... Further separates them from `IBeatSnapProvider`'s `SnapTime`, and groups them together more, to prevent confusion between the two interfaces. Also changes the xmldoc of the reference time to that of `IBeatSnapProvider` for consistency.	2021-04-26 16:07:30 +02:00
smoogipoo	fd5fbaf0db	Rename ruleset wrapper class	2021-04-26 15:37:42 +09:00
Naxess	049e42fa85	Move snapping responsibility to IBeatmap ... Seems `EditorBeatmap` already implements a different kind of `SnapTime` from `IBeatSnapProvider`, so method names here aren't great. This is very similar to what https://github.com/ppy/osu/pull/12558 is doing, so may need to do some duplicate resolution later, especially surrounding `ClosestBeatSnapDivisor`. Worth noting that this change makes 1/7, 1/5, etc unsupported for now, as we now rely on `BindableBeatDivisor.VALID_DIVISORS`.	2021-04-26 05:07:24 +02:00
Naxess	6fd77e536d	Add unsnap check	2021-04-25 05:34:54 +02:00
Dean Herbert	9a7bf8109f	Allow certain mouse input to pass through PlacementBlueprints to the selection logic	2021-04-22 15:44:12 +09:00
Naxess	4510e795e1	Fix category of audio quality check	2021-04-20 02:13:26 +02:00
Naxess	c633f15565	Add audio quality check	2021-04-20 01:36:03 +02:00
Naxess	40ae856dfc	Show 2 decimals for background filesize	2021-04-20 01:34:05 +02:00
Naxess	14c626ffcb	Use the playable beatmap for file presence checks	2021-04-20 01:33:19 +02:00
Naxess	8656176ab8	Add the playable beatmap as check argument ... This is different from the working beatmap's `.Beatmap` property in that it is mutated by the ruleset/editor. So hit objects, for example, are actually of type `Slider` and such instead of the legacy `ConvertSlider`. This should be preferred over `workingBeatmap.Beatmap`.	2021-04-20 01:31:51 +02:00
Naxess	9a69ca34a6	Add audio presence check	2021-04-18 02:07:57 +02:00
Naxess	010720de74	Factor out general file presence checking ... This allows us to use the same method of checking for other files that should exist, for example the audio file. By using the same method, they all share test cases too.	2021-04-18 02:07:33 +02:00
Naxess	56bf49c85c	Take IWorkingBeatmap instead of WorkingBeatmap ... This makes testing much easier, and allows for checking of any class deriving from that interface, including `WorkingBeatmap`.	2021-04-18 01:21:20 +02:00
Naxess	abf512532e	Clean up check logic ... Makes use of the new `BeatmapSet.GetPathForFile` method and removes dependency on `WorkingBeatmap` specifically, allowing us to switch to `IWorkingBeatmap` later.	2021-04-18 01:19:25 +02:00
Naxess	cb41c89935	Don't return low res and too low res at the same time	2021-04-17 20:10:07 +02:00
Naxess	62c54e00cb	Add check for background resolution and filesize	2021-04-17 18:01:04 +02:00
Naxess	448574e7e6	Use WorkingBeatmap instead of IBeatmap ... This lets us access things like the background, track, etc. which are necessary for quality and filesize checks. Also improves the structure of the `CheckBackgroundTest` class in the process.	2021-04-17 17:33:53 +02:00
Dean Herbert	119c9b4294	Fix placement blueprints not being correctly removed after a rolled back placement	2021-04-16 14:11:33 +09:00
Dean Herbert	cb4f64133e	Add xmldoc to interfaces	2021-04-13 23:30:20 +09:00
Naxess	69da804f81	Add missing period	2021-04-13 13:57:56 +02:00
Dean Herbert	0d6890243f	Fix typo in xmldoc	2021-04-13 20:18:18 +09:00
Dean Herbert	03ba04e8ce	Split out general checks into its own verifier class (and remove static usage)	2021-04-13 19:50:22 +09:00
Naxess	aa5fe2e9fc	Rename BeatmapVerifier -> IBeatmapVerifier	2021-04-13 11:02:01 +02:00
Naxess	60c2494b31	Make BeatmapVerifier an interface	2021-04-13 10:40:56 +02:00
Naxess	6d3f9fa9ce	Use is class instead of Equals with template index ... Ensures ordering of `PossibleTemplates` does not affect tests.	2021-04-13 02:29:25 +02:00
Naxess	d8088777ea	Add Equals method to IssueTemplate ... This will be useful in tests.	2021-04-13 01:21:34 +02:00
Naxess	19a154ddf1	Rename checkOrigin -> check ... More consistent with `Issue.ctor`'s "template".	2021-04-12 17:28:12 +02:00
Naxess	bb720c23a0	Remove check ctors and locals	2021-04-12 17:12:37 +02:00
Dean Herbert	62c1812282	Remove redundant parameter naming	2021-04-12 23:37:47 +09:00
Dean Herbert	36bd235021	Move nested classes to bottom of file	2021-04-12 23:36:10 +09:00
Naxess	6d50d01186	Make IssueTemplate.Check readonly	2021-04-12 16:23:05 +02:00
Naxess	caaaba5950	Rename Check -> ICheck	2021-04-12 16:20:53 +02:00
Naxess	008dbc7dd6	Reverse IssueType ordering ... Reversed both in the enum and where it's displayed, so ends up the same in the end.	2021-04-12 15:49:13 +02:00
Naxess	7c4f6d2b62	Rework template usage ... Includes moving the origin check back to templates, constructing nested template classes in each check, and making parameterized template usage.	2021-04-12 15:47:26 +02:00
Naxess	65ebdd8f7a	Move check origin from IssueTemplate to Issue ... As a result we can also make check an interface, and need to provide the check itself when constructing an issue.	2021-04-12 10:08:08 +02:00
Dean Herbert	8bf85d737c	Change Metadata into a get property	2021-04-12 15:52:29 +09:00
Dean Herbert	78bbc8f5c8	Tidy some remaining code	2021-04-12 15:52:18 +09:00
Dean Herbert	8c31e96cdf	Change some methods to get properties	2021-04-12 15:52:18 +09:00
Dean Herbert	f78239c7f2	Move enums out of nesting	2021-04-12 15:52:18 +09:00
Dean Herbert	3551322f1d	Fix formatting of colour getter	2021-04-12 15:52:18 +09:00
Dean Herbert	257acf9cd8	Colour constants to private	2021-04-12 15:52:18 +09:00
Dean Herbert	136627b9ac	Wrap xmldoc less and make a few fixes	2021-04-12 15:52:18 +09:00
Dean Herbert	1c553b5d48	Checker -> BeatmapVerifier	2021-04-12 15:29:05 +09:00
Dean Herbert	14bd5bdc77	Merge branch 'master' into verify-tab	2021-04-12 15:14:37 +09:00
Naxess	3289bb0379	Merge Check and BeatmapCheck ... We're probably not going to need GeneralChecks or BeatmapsetChecks. The verify tab is only available to a single difficulty at a time, and we already have access to the rest of the set through `IBeatmap`.	2021-04-10 14:56:30 +02:00
Naxess	bc4f3351f3	Replace checks with realistic ones	2021-04-10 13:03:16 +02:00
Naxess	b30e41b805	Fix comment; mode -> ruleset	2021-04-10 13:02:36 +02:00
Naxess	d1007ff26a	Move components to more appropriate spot	2021-04-10 13:02:22 +02:00
Naxess	b24ce66a0d	Add check/issue classes	2021-04-07 14:35:33 +02:00
Nathan Alo	9a02f3868c	return a string instead	2021-03-29 17:29:05 +08:00
Nathan Alo	b8b7eb4c4b	refactor logic to its own component and handle hit object to string conversion to its ruleset-specific composers	2021-03-26 15:25:20 +08:00
Dean Herbert	e9730d4782	Move default sample addition to inside PlacementBlueprint ... This isn't actually required to fix the behaviour but it does feel like a better place to put this logic.	2021-02-11 17:16:18 +09:00
Dean Herbert	04fa32bc34	Rename and add xmldoc for smooth seeking method	2021-01-15 16:14:21 +09:00
Dean Herbert	3ad2eeaff5	Fix outdated xmldoc	2020-11-30 18:35:30 +09:00
Dean Herbert	9811c46e35	Rename application method to better describe what it actually does	2020-11-26 19:16:18 +09:00
Dean Herbert	c9a41f9dae	Make all objects in selection candidates for spatial snapping ... Closes #10898.	2020-11-24 17:35:54 +09:00
smoogipoo	3957697c48	Add pooling support to the editor	2020-11-13 17:08:20 +09:00
smoogipoo	4ef2e9548c	Pass HitObjectComposer to BlueprintContainer via ctor	2020-11-13 16:59:37 +09:00
Dean Herbert	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
Bartłomiej Dach	202fe09306	Group selection actions back up in SelectionHandler	2020-10-28 22:06:48 +01:00
Dean Herbert	27c1a4c4d3	Move right-click deletion logic to be handled at a SelectionBlueprint level	2020-10-27 12:53:54 +09:00
Bartłomiej Dach	3f8c4c57d0	Fix code style issues & restructure	2020-10-26 22:16:28 +01:00
Bartłomiej Dach	7c388f1132	Move editor playfield border locally to osu! composer	2020-10-19 21:20:13 +02:00
Bartłomiej Dach	4267d23d59	Move border to more appropriate namespace	2020-10-19 20:56:34 +02:00
Bartłomiej Dach	053c7a69a6	Fix code style issues & compilation failures	2020-10-19 20:22:48 +02:00
Dean Herbert	22bde43106	Merge branch 'master' into visible-playfield-boundary	2020-10-19 18:05:28 +09:00
Dean Herbert	beec0e4193	Hide children of SelectionBlueprint when not selected	2020-10-09 14:03:13 +09:00
Dean Herbert	575046e5fd	Don't update reply on add/remove (will be automatically handled by change handler events)	2020-10-02 18:21:13 +09:00
Dean Herbert	b7c276093d	Add fallback case when EditorChangeHandler is not present (for tests)	2020-10-02 16:21:51 +09:00
Dean Herbert	501e02db09	Only regenerate autoplay on editor state change	2020-10-02 15:44:37 +09:00
Dean Herbert	524c2b678c	Forcefully regenerate autoplay on editor changes	2020-09-28 15:48:42 +09:00
Dean Herbert	ff7c904996	Add autoplay mod in editor specific ruleset construction	2020-09-28 15:48:42 +09:00
Dean Herbert	8e6bf516d9	Merge branch 'master' into editor-ternary-buttons	2020-09-25 18:30:59 +09:00