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

[FileDialog] Rework option controls. #95140

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

bruvzg
Copy link
Member

@bruvzg bruvzg commented Aug 4, 2024

Fixes #95079

  • Replaces GridContainter with nested BoxContainers.
  • Replaces CheckBox with CheckButton to ensure check element is on the same side as OptionButton in multiple choice options.
Screenshot 2024-08-04 at 20 04 22

@bruvzg bruvzg added this to the 4.4 milestone Aug 4, 2024
@Giganzo
Copy link
Contributor

Giganzo commented Aug 4, 2024

Is the choice of CheckButton over a CheckBox the correct choice for export with debug?

Isn’t Toggle switches (CheckButton) usually used for settings that take immediate effect and should not require the user to click Save or Submit to apply the new state?

Also an example from Microsoft https://learn.microsoft.com/en-us/windows/apps/design/controls/toggles#choosing-between-toggle-switch-and-check-box

@Calinou
Copy link
Member

Calinou commented Aug 4, 2024

Is the choice of CheckButton over a CheckBox the correct choice for export with debug?

Yes, I agree a CheckBox makes more sense than a CheckButton here. I can't think of any use cases for a CheckButton in the FileDialog option controls, as none of them are expected to have an immediate effect.

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

Successfully merging this pull request may close these issues.

Export With Debug text can't be clicked
3 participants