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

Button style in close window dialog #806

Open
cavearr opened this issue Jan 21, 2025 · 4 comments
Open

Button style in close window dialog #806

cavearr opened this issue Jan 21, 2025 · 4 comments

Comments

@cavearr
Copy link
Member

cavearr commented Jan 21, 2025

@lmcapacho thanks for this feature, i change the literals to do more clear, if you think are few large, try other options but it is important define clearly the operative (save and close, etc).

If you could put a "red ribbon" to this develop, define a better style to the buttons, according to other ui buttons of iceStudio:

Image

Thanks!

@lmcapacho
Copy link
Contributor

Hi @cavearr

Thanks for your feedback!

Previously, the buttons were named "Close" and "Cancel". How do feel about changing them to "Save and Close", "Discard and Close", and "Cancel"?

As for the button styles, they currently match the design used in other windows and buttons in Icestudio, following the theme used in Alertify. Could you share more about your idea?

@cavearr
Copy link
Member Author

cavearr commented Jan 21, 2025

@lmcapacho as you see in white theme (i don't try yet in dark theme) there are no buttons, appears links. i think we need to define a thin border like the "Import" in verilog form.

@cavearr
Copy link
Member Author

cavearr commented Jan 21, 2025

@lmcapacho For me, my thoughts:

  1. 💾 Save & Quit
  2. NO save & Quit
  3. Back ⏎

How do you think? Cancel is ambiguous because what do you cancel, the dialog, the save , .....i think back is clearly, the button return to your previous state.

With "quit" we gain small word and clear action.

For save changes, i think we should use save for both cases, because "cancel" is again ambiguos , what do you want to cancel? (the dialog, the changes, the windows closing....).

We could use unicode icons like save or back to do emphasis in the action. the point 2 i don't find an standard icon that like me, but i think could be fine without icon.

@TimRudy
Copy link
Contributor

TimRudy commented Jan 21, 2025 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants