search

antonmedv / walk

Terminal file manager
MIT License
3.24k stars 88 forks source link

Feature add - Context Menu #11

Closed habitualdev closed 1 year ago

habitualdev commented 2 years ago

Adds a spacebar powered context menu, with the following options:
select - Focuses on a file for move and copy operations copy - copies the selected file to the currently browsed directory move - moves the selected file to the currently browsed directory delete - runs "rm -rf" on the currently focused file/directory settings - directly opens ~/.llamarc

This is facilitated by the addition of a bool to the model, "contextMenu". Based on the value of this new bool, it selects for different actions for select keys (Up, down, enter, spacebar).

At this time, the options are scrolled through, but only one is displayed at a time.

Context Menu screenshot attached. ContextMenu

antonmedv commented 2 years ago

This is not really looking good. I think we can work on better design.

habitualdev commented 2 years ago

For sure. I'll rework the context menu to be more visually appealing. Are you ok in general with the " different page" design? I tried messing around with forcing bubbletea to do a drop-down style menu on the list view, but it ended up breaking the ui more often than not.

Additionally, thoughts on how llamarc is implemented? Are you ok with colon separated values, or do you want to do a specific notation format?

habitualdev commented 2 years ago

Updated the context menu to reflect the base look.

updatedUI

habitualdev commented 2 years ago

Remediated conflicts from the recent PR's

habitualdev commented 2 years ago

Should I change to .llama from .llamarc, or wait on the bubbletea update?

antonmedv commented 2 years ago

Let’s wait bubble tea first. Lots of bugs we will encounter otherwise.

antonmedv commented 1 year ago

Let’s implement something more terminal style.