Commit Graph

2936 Commits

Author SHA1 Message Date
9c3d15171c Reword xmldoc slightly 2021-04-02 18:00:28 +09:00
5ac36a2462 Switch AlwaysPlaySample to SamplePlaysOnlyOnHit in DrawableSliderTail for conformity 2021-04-02 03:56:23 -05:00
48e9985782 Make "AlwaysPlayTailSample" a mod setting rather than a hardcoded constant. 2021-04-02 03:10:28 -05:00
5063cd957f Force hit sample to play when Classic mod is enabled 2021-04-02 02:54:35 -05:00
b82247aabe Add inline comments and use Vector2.Zero 2021-03-30 14:13:16 +09:00
eda45e0e28 Merge branch 'master' into scale-breaking-bounds 2021-03-30 14:05:19 +09:00
17b16d4f89 Clarify purpose of getClampedScale() 2021-03-29 14:17:30 +02:00
1d99a63f17 Limit minimum size for single slider scaling 2021-03-29 14:16:27 +02:00
690debe507 Merge pull request #12030 from LeNitrous/editor-platform-clipboard-copy 2021-03-29 19:31:16 +09:00
9a02f3868c return a string instead 2021-03-29 17:29:05 +08:00
a50c4be8ab Add missing space 2021-03-26 17:41:36 +01:00
305c2e31cf Clarify todo comment 2021-03-26 16:53:17 +01:00
25ea60cb92 Remove return values from HandleScale submethods 2021-03-26 16:40:10 +01:00
5d272bef97 Remember ContolPoint positions instead of recalculating them 2021-03-26 16:31:04 +01:00
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
2bea69456e remove implementations 2021-03-26 15:24:33 +08:00
bada1e7189 Update legacy spinner bonus counter usage 2021-03-25 15:00:06 +03:00
6aa0c9d8fd Merge branch 'master' into legacy-sprite-text-fonts 2021-03-25 14:52:01 +03:00
b4bccc19ba Refactor LegacySpriteText initialisation 2021-03-25 18:37:13 +09:00
c715b9b89e Merge branch 'master' into legacy-spinner-bonus 2021-03-25 18:09:25 +09:00
e67ab3cca7 Change single slider scaling to a method that works 2021-03-24 12:24:11 +01:00
3d471d239f Clamp multi-object scale instead of cancelling it 2021-03-24 12:24:11 +01:00
def0e5c42e Fix off-by-one error in isQuadInBounds 2021-03-24 12:24:11 +01:00
ade542191a Merge pull request #12154 from peppy/fix-hitcircle-glow-fade-out
Fix default skin's glow resetting fade on miss
2021-03-23 20:14:51 +09:00
d10ff615fe Fix default skin's glow resetting fade on miss 2021-03-23 19:28:08 +09:00
64e85ba995 Always fade out approach circles at a HitObject's start time to better match stable 2021-03-23 19:19:07 +09:00
9e38d580ca Merge pull request #12139 from peppy/fix-editor-spinner-looping-too-long
Fix spinners playing looping sound too long in the editor
2021-03-23 17:44:07 +09:00
098005393e Remove unnecessary null checks and debug code 2021-03-23 10:38:37 +09:00
e60ff45b73 Add another test for colinear perfect curves 2021-03-22 16:57:40 +09:00
c4d08463ad Fix spinners playing looping sound too long in the editor
The `OnComplete` event was never being run due to the transform playing
out longer than the spinner's lifetime. I've matched the durations, but
also moved the `Stop()` call to what I deem a safer place to run it (I
did notice that without this it would still potentially never fire).

Note that this is more noticeable in the editor because of lifetime
extension. In gameplay, the returning of a spinner to the pool will
clean things up (but in the editor that can take longer, depending on
timeline zoom level).

Another thing worth mentioning is that the fade doesn't actually work.
This is due to https://github.com/ppy/osu-framework/pull/4212.

Closes #12119.
2021-03-22 16:04:51 +09: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
51e0304c54 properly format strings per ruleset 2021-03-17 18:31:09 +08:00
7fa5fd5647 Update usages of config with framework changes 2021-03-17 16:10:16 +09:00
848adddd92 Use double.ToString(InvariantInfo) instead 2021-03-15 11:05:29 +03:00
779c55d768 Fix potentially adding null legacy text to hierarchy 2021-03-14 03:27:47 +03:00
43c1e1d217 Update existing usages
Resolve post-conflict issues
2021-03-13 06:09:07 +03:00
115c186cb7 Move hit circle font from osu! ruleset 2021-03-13 06:05:36 +03:00
98f6e16113 Move default spinner bonus counter to new DefaultSpinner 2021-03-13 05:59:42 +03:00
774ebf50bc Move legacy spinner bonus counter to LegacySpinner 2021-03-13 05:59:42 +03:00
8fdab5a7de Revert legacy spinner presence changes and bonus counter component
No longer necessary, after inlining legacy coordinates logic to `LegacySpinner` and limiting precisely-positioned legacy components there
2021-03-13 05:57:47 +03:00
020a03e01e Use sensible "score per tick" constant 2021-03-13 05:56:28 +03:00
c4048c9831 Merge branch 'master' into legacy-spinner-bonus 2021-03-13 05:54:34 +03: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