BFilePanel empty after pressing ESC

[humdinger]

This is hrev50808.

If a filepanel is used in B_DIRECTORY_NODE mode and the user presses ESC, all the contents in the panel disappears. I first saw it with BurnItNow, confirmed it in Ubertuber and can replicate with Expander's destination file panel. Are there other apps using B_DIRECTORY_NODE?

Weirdly, it shows also in Expander's source panel, which uses B_FILE_NODE mode. But it doesn't show with any other open file dialog I checked, not in StyledEdit, Pe, ShowImage...

Could this be related to the even more annoying "Missing shortcuts in file panels", #11979 ?

[owenca]

I don't think it's related to #11979. See the attached patch below.

[humdinger]

I locally applied this patch and the one from #11979, build Tracker, put it and the libtracker.so into non-packaged and blacklisted the system Tracker and libtracker.so in the boot options.

For some reason, upon booting I'm greeted with an empty blue desktop and an empty title-less window...

[owenca]

Please see the updated patch. I'm not sure if we still want to make Atl-W close the file panel because Esc will do the job.

Also, the Esc key still clears the file list view before closing the panel. It seems to be a bug in Tracker. I don't know if we want to not send Esc to Tracker before closing the panel so as to avoid the bug.

[humdinger]

We should keep ALT+W, it's a system wide shortcut.
I think the ESC handling should be: if something's filtered with type-ahead-filtering, ESC clears that filter. Only if nothing is filtered, ESC closes the panel.

[pulkomandy]

I would prefer if ESC did not close windows, especially if it depends on the context.

As a vim user, I tend to press ESC a lot as this is how you switch mdoes in vim. I find it very annoying when it closes a window without a warning. So we should restrict that to very few cases, and I'm not sure file panels are one of these, because closing them may lose some state (maybe you navigated to a specific directory, carefully searched for the right file in there, etc).

[humdinger]

I'm pretty sure file panel always used to be closable with ESC. It just doesn't always work for all apps for some reason.

Personally, not being a vim user (shocker!), ESC is synonymous to "Cancel". I save about a hundred mouse miles with my finger hovering near ESC instead of aim and click a "Cancel" button or even tinyer close widget.

[diver]

FWIW, I'm with humdinger here.

[pulkomandy]

Depending on the context:

• If you are renaming a file, ESC aborts that operation
• If you are using filtering, ESC aborts that operation
• If you are not doing either, ESC closes the whole window

I find it annoying that the same key can be used for the third case. I would allow this only in windows where ESC is not used for anything else. There's already Alt+W if you need to close the window from the keyboard, and this one always work no matter what you are doing (renaming a file, filtering, ...).

[humdinger]

My opinion differs. I see your 3rd item as also "aborting that operation". The operation being open/save a file via file dialog. I agree that using ESC to close the main window of an app would be bad (in most cases). But the open/save dialog is only a sub-process, an operation, in your work flow within that app.

[humdinger]

Please also see my comment at ticket:11979#comment:22

[humdinger]

Maybe related: pressing ESC in a query result window also clears away all items.

[diver]

Was this patch migrated to gerrit? Couldn't find it there.

[jscipione]

PR to fix: ​https://review.haiku-os.org/c/haiku/+/8689

The reason you see this bug in Expander and not in StyledEdit, Pe, ShowImage is because Expander has a ref filter applied while those other apps don't. The patch set does a better job of differentiating between ref filtering and type-ahead filtering.

[waddlesplash]

Fix merged in hrev58432.