Commit Graph

25033 Commits

Author SHA1 Message Date
ecf70c1707 Remove unnecessary container 2021-05-17 18:55:18 +09:00
b80768b44a Hook up seeking flow 2021-05-17 18:52:56 +09:00
0c433cda86 Update HUDOverlay logic to add automatic layout for bottom-aligned components 2021-05-17 18:52:56 +09:00
da0913ca2d Make SongProgress a skinnable component 2021-05-17 18:52:56 +09:00
4151f18f74 Merge pull request #12804 from peppy/fix-autoplay-load-fail-nullref
Fix a potential nullref when starting `Player` with autoplay disabled and beatmap fails to load
2021-05-17 18:40:05 +09:00
1f3ae901ce Expose DrawableRuleset for consupmtion by HUD components 2021-05-17 18:23:01 +09:00
0d7a349500 Exclude interfaces from skinnable types 2021-05-17 18:16:09 +09:00
555e3e2db3 Delete NewsSideBar.cs 2021-05-17 12:02:33 +03:00
01090de1fd Fix filenames does not match contained type 2021-05-17 11:55:55 +03:00
586c5c7365 Emulate year changes in the test scene 2021-05-17 11:36:53 +03:00
ab16c32bb1 Merge remote-tracking branch 'refs/remotes/ppy/master' into news-sidebar-new 2021-05-17 11:36:41 +03:00
d75ef160e8 Merge branch 'master' into directory-select-screen 2021-05-17 17:22:03 +09:00
c0cfbd11dd Add tooltip and action for PostButton in MonthSection 2021-05-17 11:20:31 +03:00
5059bfaef9 Use FillFlowContainer in YearsPanel 2021-05-17 11:17:02 +03:00
e3b8d8ee18 Add support for overlay-coloured links 2021-05-17 16:58:54 +09:00
e754d2e590 Simplify YearButton 2021-05-17 10:54:45 +03:00
6db291fe84 Merge remote-tracking branch 'refs/remotes/ppy/master' into news-sidebar-new 2021-05-17 10:51:56 +03:00
baa4089364 Expose method to adjust header text, not whole drawable 2021-05-17 16:41:46 +09:00
abeeda5d04 Rewrite YearsPanel to not be insanely long 2021-05-17 16:24:56 +09:00
032f60819d Rename content container 2021-05-17 16:11:46 +09:00
22561cda19 MonthDropdown -> MonthSection 2021-05-17 16:02:21 +09:00
50e2b5a327 SideBar -> Sidebar 2021-05-17 16:00:36 +09:00
ce7c503e6e Merge branch 'master' into playlist-unicode 2021-05-17 11:47:10 +09:00
a32f1ef35a Merge pull request #12811 from PercyDan54/hidden-code-cleanup
Remove obsolete methods in `ModHidden` and apply few touches
2021-05-16 22:07:45 +09:00
3e1b1c6c3e Improve statement readability 2021-05-15 19:14:58 +03:00
e479db9186 Clear transforms in PostsContainer for all children 2021-05-15 19:14:02 +03:00
881d82ccb6 Merge remote-tracking branch 'refs/remotes/ppy/master' into news-sidebar-new 2021-05-15 19:08:48 +03:00
34d1490754 Remove null conditional
Shouldn't guard against that here.
2021-05-15 10:00:05 +03:00
dc56250a3c Merge branch 'master' into player-loader-star-rating 2021-05-15 15:39:57 +09:00
94b7e89ac0 Merge pull request #12747 from frenzibyte/current-star-rating
Allow changing current star difficulty of a `StarRatingDisplay`
2021-05-15 15:39:39 +09:00
bdcb1a624e Merge pull request #12759 from Naxesss/beatmap-verifier-context
Encapsulate check arguments in context object
2021-05-15 15:39:12 +09:00
166974506e Duplicate implementions 2021-05-15 11:55:50 +08:00
6e5c4ed7c6 Revert "Remove empty override"
This reverts commit a86a4bab91.
2021-05-15 11:45:14 +08:00
7632b8621e Merge pull request #12805 from peppy/add-user-id-beatmap-metadata
Add database tracking of beatmap creator `user_id`s
2021-05-15 09:03:52 +09:00
7665bdfbbb Merge branch 'master' into null-assignment-warning-enable 2021-05-15 01:08:34 +02:00
69fc072429 Ignore skin component json data if deserialisation fails instead
Crashing was not really the best thing to do there given the preceding
code that already allowed a few continues in case of a missing file.
2021-05-15 01:08:11 +02:00
b51d038088 Ignore possible path-related nulls
They're all in test code anyway, so any issue there will cause a test to
fail.
2021-05-15 00:09:34 +02:00
fa6b5515b7 Ignore possible null from JsonConvert.DeserializeObject()
Nothing better can be done if a `null` is indeed returned.
2021-05-15 00:09:34 +02:00
5b2b701915 Ignore possible null in GetResponseString()
A null there indicates a deserialisation error and therefore due to the
catch block immediately succeeding the changed line everything will
continue to work as intended.
2021-05-15 00:09:34 +02:00
628e7a71ed Ignore possible nulls in Type.GetType() calls
They're mostly used in extensibility scenarios, so everything happens in
runtime. There is no better resolution than to crash with a null
reference exception.
2021-05-15 00:09:34 +02:00
43c73f9583 Mark access to exception if task faulted as safe
There are seemingly no C#-side compile-time guarantees that it is safe,
but if the task's state is `Faulted` (as is checked right before), the
exception cannot be null as per the documentation.
2021-05-15 00:09:34 +02:00
c9facf70f9 Use conditional nullability attribute
As it turns out, C# 8 provides an attribute that allows annotating that
an `out` parameter's nullability depends on the method's return value,
which is exactly what is desired here.
2021-05-15 00:09:34 +02:00
aaa7c7eb05 Handle null case explicitly in SpectatorState.Equals()
Uses the usual pattern of two `ReferenceEquals` checks against `this`
and `null` before proceeding to inspect field values. Doing this causes
the compiler to infer that at the point that field values are checked,
`other` can no longer viably be `null`.
2021-05-14 23:58:07 +02:00
044770f1a2 Locally suppress warning in SerializationReader
`SerializationReader` is not written in a form that would support
turning nullability checking on for the entire class. The biggest
problem there is the inner `DynamicDeserializer` static class, whose
members are initialised via an `initialize()` method, which the compiler
knows nothing about.

For this reason, just opt to suppress the single inspection about
returning a `null` from a method with a return type of `string` (rider
expects `string?`). It would have been also viable to enable nullability
checking for this one method, but that's pretty much the same thing and
adds no safety anyways, so just disable the warning to minimise
surprise.
2021-05-14 23:29:34 +02:00
94d80f091f Merge branch 'master' into fix-deleting-skin-elements 2021-05-14 22:33:41 +02:00
3d3c5028e6 Trim unnecessary array copy 2021-05-14 22:33:26 +02:00
743b4fbff1 Pass correct member name to ArgumentExceptions 2021-05-14 22:16:37 +02:00
876f53bf3b Fix copy-paste oversights in xmldoc & exception messages 2021-05-14 22:15:43 +02:00
8b4e6d2911 Remove no longer necessary FinishTransforms(true) call
As the component no longer has any transition transforms applied.
2021-05-14 22:03:35 +02:00
a86a4bab91 Remove empty override 2021-05-14 21:55:32 +08:00