Skip to content

Add beatmap panel to the ranked play results screen#37558

Merged
peppy merged 2 commits intoppy:masterfrom
smoogipoo:rp-results-beatmap-info
Apr 28, 2026
Merged

Add beatmap panel to the ranked play results screen#37558
peppy merged 2 commits intoppy:masterfrom
smoogipoo:rp-results-beatmap-info

Conversation

@smoogipoo
Copy link
Copy Markdown
Contributor

Different take to / supersedes / closes #37550

Just going for minimal lines of code compared to the PR above, by reusing the quick play panel which is in a pretty good state.

image

@smoogipoo smoogipoo force-pushed the rp-results-beatmap-info branch from 88612af to 1985140 Compare April 28, 2026 09:17
@smoogipoo smoogipoo force-pushed the rp-results-beatmap-info branch from 1985140 to 49e2ff8 Compare April 28, 2026 09:23
@peppy peppy self-requested a review April 28, 2026 10:53
@peppy peppy force-pushed the rp-results-beatmap-info branch from 49e2ff8 to 46b636f Compare April 28, 2026 11:12
@peppy peppy merged commit 71f841d into ppy:master Apr 28, 2026
8 of 10 checks passed
@peppy peppy deleted the rp-results-beatmap-info branch April 28, 2026 13:08
Copilot AI pushed a commit to winnerspiros/osu that referenced this pull request Apr 28, 2026
Different take to / supersedes / closes
ppy#37550

Just going for minimal lines of code compared to the PR above, by
reusing the quick play panel which is in a pretty good state.

<img width="1298" height="864" alt="image"
src="https://github.com/user-attachments/assets/409e163d-d7ea-4c07-874f-aaca7e145221"
/>

---------

Co-authored-by: Dean Herbert <pe@ppy.sh>

Co-authored-by: winnerspiros <1675249+winnerspiros@users.noreply.github.com>
Copilot AI added a commit to winnerspiros/osu that referenced this pull request Apr 28, 2026
…rom upstream (ppy#37558)

Agent-Logs-Url: https://github.com/winnerspiros/osu/sessions/149b1b11-32bf-4074-85ef-bc5b623d3252

Co-authored-by: winnerspiros <1675249+winnerspiros@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants