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

GUI v2: Alert popup #888

Open
6 tasks
Tracked by #885
apyrgio opened this issue Aug 8, 2024 · 3 comments
Open
6 tasks
Tracked by #885

GUI v2: Alert popup #888

apyrgio opened this issue Aug 8, 2024 · 3 comments
Labels
enhancement New feature or request gui-v2 Issues that target the redesign of the Dangerzone GUI

Comments

@apyrgio
Copy link
Contributor

apyrgio commented Aug 8, 2024

Main issue: #885

Make our alert popups more sleek, by doing the following:

  • Use orange color for the selected action.
  • Make borders round
  • Make the title bold
  • Remove any window decorations
  • Make text centered
  • Make main window inactive and gray

This design will probably clash in subtle ways with our existing warning messages. We'll comment on what issues arise, and how we can circumvent them.

Design material

Single action

image

Two actions

image

@apyrgio apyrgio added enhancement New feature or request gui-v2 Issues that target the redesign of the Dangerzone GUI labels Aug 8, 2024
@apyrgio apyrgio changed the title GUI V2: Alert popup GUI v2: Alert popup Aug 8, 2024
@apyrgio
Copy link
Contributor Author

apyrgio commented Aug 8, 2024

Docker message

The Docker message in the design material misses the option we give to the user to redo the check. We can either add it, or check for the existence of Docker/Podman every 1s, and then automatically proceed. In any case, we also need to add the following:

  • A proper explanation of why Docker is needed
  • The error that occured (applicable to the case of missing/corrupted container images)

@apyrgio apyrgio mentioned this issue Aug 8, 2024
12 tasks
@DeltaEpsilon19498
Copy link

Will a warning message specify that docker is needed for windows or mac but not linux (or windows subsystem for linux)?

@apyrgio
Copy link
Contributor Author

apyrgio commented Aug 20, 2024

Definitely, I'll make sure to create an issue for the 0.8.0 release and tackle it there, as it's long overdue. This fix will be orthogonal to the designs we suggest here. Thanks for the reminder!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request gui-v2 Issues that target the redesign of the Dangerzone GUI
Projects
None yet
Development

No branches or pull requests

2 participants