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

open editor in current line #642

Open
electriquo opened this issue Aug 25, 2024 · 0 comments
Open

open editor in current line #642

electriquo opened this issue Aug 25, 2024 · 0 comments
Labels
enhancement New feature or request

Comments

@electriquo
Copy link

electriquo commented Aug 25, 2024

Is your feature request related to a problem? Please describe.
following 597e56b, it is expected that:

  1. pressing the ? to list the hotkeys will include e, but it does not.
  2. pressing e will work when opening a single file and not the entire directory. e.g. when using glow README.md, pressing e does not open the editor but rather scrolls down the view.
  3. a configuration to support for specifying the editor and its argument, e.g. code --wait, in the configuration file, rather then always use the shell default editor.
  4. the commit title states feat: open editor in current line, but what is the current line when viewing a markdown in glow?

Describe the solution you'd like
A clear and concise description of what you want to happen.

  1. add e to ? hotkey listing.
  2. pressing e should open the editor and not scroll down the markdown view.
  3. add support for specifying editor in the configuration file.
  4. add ability to highlight or have any other indication what is the current line.

Describe alternatives you've considered

  1. invoking glow after setting the EDITOR, e.g. EDITOR="code --wait" glow
  2. invoke glow --line-numbers without specifying any markdown file to show line numbers when viewing a markdown file, then pressing e to see the correlation between the view with line numbers to the current line on the editor. though, there was no noticeable correlation.

Additional context
Add any other context or screenshots about the feature request here.

  1. e is not in hotkey listing

       enter    open      /  find    r  refresh
       j/k ↑/↓  choose               q  quit
       h/l ←/→  page                 ?  close help
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant