Editor timestamp jump does not handle full mods

[kadambishreyas]

Type

Game behaviour

Bug description

@bdach this pertains to the timestamp jump feature added in #28511.

context:
the most common use case for this feature by mappers is to view a selection of objects from outside web.
typically this means navigating the editor to & selecting a specific pattern while modding over discord.

the typical format of a mod looks like <timestamp> (<object list>) - <any text content>.

problems:

• the current implementation merged into lazer does not handle pasting a full mod (as many do for convenience when using discord or other plaintext).
• it also does not handle the trailing - after the object list of a timestamp, which is traditionally treated as a part of the timestamp.
  • notably, this means that lazer cannot parse the timestamp format that it produces itself when copying objects (as demonstrated in the video, where parsing only succeeds after trimming to <timestamp> (<object list>)).

relevant regex:

osu/osu.Game/Rulesets/Edit/EditorTimestampParser.cs

Line 35 in 5651072

private static readonly Regex time_regex_lenient = new Regex(@"^(((?<minutes>\d{1,3}):(?<seconds>([0-5]?\d))([:.](?<milliseconds>\d{0,3}))?)(?<selection>\s\([^)]+\))?)$", RegexOptions.Compiled);

Screenshots or videos

Screen.Recording.2024-07-27.at.2.45.55.PM.mp4

Version

2024.727.0-lazer

Logs

compressed-logs.zip