Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FEATURE] Switch from YES/NO buttons to labelling the actions that would be taken #813

Open
jedahan opened this issue Feb 7, 2024 · 3 comments
Labels
enhancement New feature or request

Comments

@jedahan
Copy link

jedahan commented Feb 7, 2024

Is your feature request related to a problem? Please describe.
The dialogue popups have caused confusion in the past (see #757 and #624 and #700) - it would be nice to have the buttons say what action they will take instead. As per #757, I am opening a pull request describing the solution before trying to implement it.

Describe the solution you'd like

On the write confirmation dialogue

  • Change 'Yes' to 'Erase all contents of the drive and continue'
  • Change 'No' to 'Keep the contents of the drive and cancel the write'

On the OS Customization Dialog

  • Change 'Yes' to 'Apply OS Customisations and write'
  • Change 'No' to 'Write without applying OS Customisations'
  • Change 'Edit Settings' to 'Edit OS Customisations'
  • Remove 'No, Clear Settings'

In OS Customization Settings

  • Add 'Reset all settings to defaults' or 'Clear All OS Customization Settings' button to the left of the Save button

Describe alternatives you've considered
Considered showing what customisations would be applied in the confirmation box

Additional context

@jedahan jedahan added the enhancement New feature or request label Feb 7, 2024
@tdewey-rpi
Copy link
Collaborator

tdewey-rpi commented Feb 8, 2024

This proposal would align closer with that I think should happen - however I've not previously explored it because it can only vary the vertical height of the dialog - there's no scope to make a wider window, due to constraints running on low-resolution displays.

Stacking the actions vertically, while it has precident in Raspberry Pi OS, has no equivalent reference inside Raspberry Pi Imager - where by convention actions have been kept as left-to-right, rather than top-to-bottom.

It is that convention that has prevented me considering such changes until we can re evaluate the UI. Otherwise we could wind up with a disjointed user experience that will go through potentially two paradigm shifts before arriving at a new stable point - rather than just transitioning directly to the new stable point.

@jedahan
Copy link
Author

jedahan commented Jun 3, 2024

What are the lowest and highest (1x scale) resolutions we would like to support? I can make some mockups/screenshots to help move this forward.

@tdewey-rpi
Copy link
Collaborator

tdewey-rpi commented Jun 3, 2024

What are the lowest and highest (1x scale) resolutions we would like to support? I can make some mockups/screenshots to help move this forward.

Highest: Underdetermined. I see 8K monitors are shipping now, though I very much doubt anyone is driving those at 1x
Lowest: Specified in our main QML layout file, https://github.com/raspberrypi/rpi-imager/blob/4b6ee9bb4b1f29587db6145d82f4784d8863d036/src/main.qml#L17C1-L20C59

Note that English is also one of the most compact representations of text supported by rpi-imager. We have a range of translations, including those from non-ASCII character sets that may occupy significantly greater widths.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants