juftin / browsr

🗂️ a pleasant file explorer in your terminal supporting all filesystems
http://juftin.com/browsr/
MIT License
225 stars 13 forks source link

Show file on navigation #20

Closed jalanb closed 10 months ago

jalanb commented 1 year ago

When navigating in the file tree only / arrows are recognised.

I'd suggest / should also be recognised

Also - nodes are highlighted in directory tree when using /, but the file viewer is not updated. When I hit to (e.g.) move to the file fred.py in directory tree, then I should also see text of fred.py on the right hand side

juftin commented 1 year ago

I like the navigating through the directory tree ideas - I'd have to find a way to further customize the Textual Directory Tree Widget to handle something like that.

For the code block to render a new code file, the Textual app sends a message, FileSelected. However there isn't a message sent on hover or when selecting with the arrow keys, that's possible but might be difficult to ratelimit for fast movement of a mouse. Instead you can use the ENTER key to "select" the file when selecting with the arrow keys, would that work for you?

Ideally, there's a way to tell that you're navigating with your mouse and it would just update the screen when doing so. This would probably be a good experience on local file systems but may suffer for remote filesystems.

jalanb commented 1 year ago

ENTER key works fine for me, especially given fast navigation with arrow keys