Skip to content

feat(GameScreenshotsRelationManager): adjust copy for systems that support upscaling#4905

Merged
wescopeland merged 2 commits into
RetroAchievements:masterfrom
wescopeland:media-ss-upload-upscaling-nudge
May 21, 2026
Merged

feat(GameScreenshotsRelationManager): adjust copy for systems that support upscaling#4905
wescopeland merged 2 commits into
RetroAchievements:masterfrom
wescopeland:media-ss-upload-upscaling-nudge

Conversation

@wescopeland
Copy link
Copy Markdown
Member

This PR slightly adjusts the helper text for the screenshot upload when the game's system supports upscaling. Now, the helper text nudges the user towards uploading an upscaled screenshot. A similar change will be needed in the player-facing dialog.

Before
Screenshot 2026-05-19 at 6 45 25 PM
Screenshot 2026-05-19 at 7 02 34 PM


After
Screenshot 2026-05-19 at 7 00 38 PM

Screenshot 2026-05-19 at 7 00 23 PM

@wescopeland wescopeland enabled auto-merge (squash) May 21, 2026 21:31
@wescopeland wescopeland merged commit 67b7d51 into RetroAchievements:master May 21, 2026
21 checks passed
wescopeland added a commit that referenced this pull request May 21, 2026
…4906)

Accompanies #4905.

This PR adjusts the validation copy in the player-facing screenshot
upload dialog. The empty state is a bit calmer now, and there are
multiple 2x/3x nudges for systems that support upscaling. Supported
resolutions are no longer shown unless there's very few of them (most
systems) or there's a validation error.

These changes are entirely centered around the labels that users see.
Functionally, the dialog behaves the same.

**Empty States**
<img width="459" height="431" alt="Empty Simple"
src="https://github.com/user-attachments/assets/6f282625-d697-4e42-bf43-de21ddcac22a"
/>

<img width="476" height="464" alt="Empty Upscaling"
src="https://github.com/user-attachments/assets/ffaa5df8-7da4-4808-81e7-c23c726d1ec5"
/>

---

**Upscaling Warning**
<img width="467" height="559" alt="1x Warning"
src="https://github.com/user-attachments/assets/e39e372c-3094-42da-b3ab-64e0198e9e2f"
/>

---

**Validation Errors**
<img width="469" height="585" alt="Invalid Error without Upscaling"
src="https://github.com/user-attachments/assets/a5b13d0a-c82a-4f2c-8103-7f3167e69fe5"
/>

<img width="470" height="578" alt="Invalid Error with Upscaling"
src="https://github.com/user-attachments/assets/34133c39-5ae8-44d1-a7b3-4da1d34220b0"
/>

<img width="338" height="145" alt="Invalid Resolution Tooltip"
src="https://github.com/user-attachments/assets/3b3cb14f-601e-4e3f-82f6-517785bf2f55"
/>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants