Skip to content

Conversation

@tiffany-slac
Copy link

Description

This PR refactors the Restore dialog in the Snapshot details page. Instead of displaying a full PV table inside the dialog, the dialog now shows a simple confirmation message. Users who want to review the PVs they’ve selected can use the newly added filter.

  • Added a "Show selected" checkbox above the PV table to filter displayed PVs
  • Removed the PV table from the Restore dialog and made it more compact
  • Adjusted dialog width and padding

Motivation

Closes #13

Previously, the dialog contained a PV table which was redundant since users can filter and select PVs on the main page. This change:

Screenshots

New 'Show selected ( )' Filter
Screenshot 2026-01-29 at 10 55 31 AM
Screenshot 2026-01-29 at 10 56 35 AM

Restore Dialog
Screenshot 2026-01-29 at 10 55 42 AM

Pre-merge checklist

  • Code works interactively
  • Code follows the style guide
  • Code contains descriptive docstrings
  • New/changed functions and methods are covered in the test suite where possible
  • Test suite passes locally
  • Test suite passes on GitHub Actions

- Added a "Show selected" checkbox above the PV table to filter displayed PVs
- Removed the PV table from the Restore dialog and made it more compact
- Adjusted dialog width and padding for proper alignment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants