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

Allow Esc key to always close dialog (no file selected) - p3 #14

Closed
ghost opened this issue Apr 8, 2021 · 1 comment
Closed

Allow Esc key to always close dialog (no file selected) - p3 #14

ghost opened this issue Apr 8, 2021 · 1 comment
Assignees

Comments

@ghost
Copy link

ghost commented Apr 8, 2021

Minor issue.

  1. Open File chooser
  2. Single-click Recent items
  3. Press Esc key

Expected: File chooser dialog closes.

Actual: File chooser dialog remains open.

Regardless of what File chooser dialog widget has focus, the Esc key should always close the dialog window, returning an empty list. This is another reason why having a separate dialog to navigate to a new directory is probably not used by other OS-level file choosers: pressing Esc would have to close both dialogs in succession. Easier not to deal with two dialogs.

@ghost ghost changed the title Allow Esc key to always close dialog (no file selected) Allow Esc key to always close dialog (no file selected) - p3 Apr 9, 2021
@io7m
Copy link
Owner

io7m commented Apr 9, 2021

Yep, seems fine.

@io7m io7m self-assigned this Apr 10, 2021
@io7m io7m closed this as completed in 5587639 Apr 10, 2021
io7m added a commit that referenced this issue May 18, 2021
Release: com.io7m.jwheatsheaf 3.0.0

A big thanks to @DaveJarvis for suggestions, bug reports, and some
code contributions!

Change: Add the ability to specify custom strings (Ticket: #7)
Change: Add the ability to confirm file selections (Ticket: #17)
Change: Allow for including ".." in directory listings (Ticket: #23)
Change: Improve "select directly" dialog behaviour (Ticket: #29)
Change: Fix the size formatter (Ticket: #30)
Change: Use the default filesystem by default (Ticket: #21)
Change: Add support for glob-based filters (Ticket: #19)
Change: Improve filename field behaviour (Ticket: #28)
Change: Add an optional home directory button (Ticket: #12)
Change: Enable sorting of directory items (Ticket: #22)
Change: Allow for setting a default file filter (Ticket: #9)
Change: Allow the escape key to close file choosers (Ticket: #14)
Change: Allow for specifying an initial filename in choosers (Ticket: #15)
Change: Allow for specifying custom dialog titles (Ticket: #8)
Change: Make the default filters part of the public API (Ticket: #10)
io7m added a commit that referenced this issue May 18, 2021
Release: com.io7m.jwheatsheaf 3.0.0

A big thanks to @DaveJarvis for suggestions, bug reports, and some
code contributions!

Change: Add the ability to specify custom strings (Ticket: #7)
Change: Add the ability to confirm file selections (Ticket: #17)
Change: Allow for including ".." in directory listings (Ticket: #23)
Change: Improve "select directly" dialog behaviour (Ticket: #29)
Change: Fix the size formatter (Ticket: #30)
Change: Use the default filesystem by default (Ticket: #21)
Change: Add support for glob-based filters (Ticket: #19)
Change: Improve filename field behaviour (Ticket: #28)
Change: Add an optional home directory button (Ticket: #12)
Change: Enable sorting of directory items (Ticket: #22)
Change: Allow for setting a default file filter (Ticket: #9)
Change: Allow the escape key to close file choosers (Ticket: #14)
Change: Allow for specifying an initial filename in choosers (Ticket: #15)
Change: Allow for specifying custom dialog titles (Ticket: #8)
Change: Make the default filters part of the public API (Ticket: #10)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant