Skip to content

feat(saves): show offline indicators in play section and saves tab (#221)#223

Merged
danielcopper merged 2 commits intomainfrom
feature/221-offline-slots
Apr 9, 2026
Merged

feat(saves): show offline indicators in play section and saves tab (#221)#223
danielcopper merged 2 commits intomainfrom
feature/221-offline-slots

Conversation

@danielcopper
Copy link
Copy Markdown
Owner

@danielcopper danielcopper commented Apr 9, 2026

Summary

  • Adds a conditional offline info item in the RomM play section — a warning triangle with "RomM offline" label, shown as the first info item whenever the connection state is offline.
  • Adds a red top banner in the saves tab that explains slot switching is disabled while RomM is unreachable. The banner is visible during the slotsLoading state too, so the offline state is communicated immediately on tab open.
  • Subscribes to the existing romm_connection_changed event (same pattern as CustomPlayButton), no new backend calls or API changes.

Closes #221.

Test plan

  • With RomM online: no offline indicator in play section, no red banner in saves tab
  • With RomM offline on tab open: offline indicator shown in play section, red banner shown at top of saves tab (also during "Loading slots...")
  • RomM goes from online to offline while saves tab is open: banner appears without reloading the tab
  • RomM recovers from offline to online: banner and indicator disappear
  • Slot switch attempt while offline still shows the existing inline error (unchanged behavior)
  • Playtime / BIOS / other play-section info items still render correctly alongside the offline indicator

)

Add a conditional offline info item to the RomM play section (warning
triangle + "RomM offline" label) and a red banner at the top of the
saves tab explaining that slot switching is disabled while the server
is unreachable. The saves tab subscribes to romm_connection_changed so
the banner appears during the slots-loading state too.
Resolves SonarCloud typescript:S7764. Matches the pattern already used
in RomMPlaySection for dispatching the same event.
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Apr 9, 2026

@danielcopper danielcopper merged commit 78331c0 into main Apr 9, 2026
9 checks passed
@danielcopper danielcopper deleted the feature/221-offline-slots branch April 9, 2026 11:02
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.

Show cached slots when RomM is offline, block switch with warning

1 participant