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

feature suggestion #7

Open
joihn opened this issue Aug 28, 2023 · 2 comments
Open

feature suggestion #7

joihn opened this issue Aug 28, 2023 · 2 comments

Comments

@joihn
Copy link

joihn commented Aug 28, 2023

thanks for this tool ! the only vimium-like tool for linux OS to my knowledge

A few feature suggestion:

  1. like in homerow, the textbox could be shown under the button, not to hide it
    https://www.homerow.app/
  2. yellow textbox could be better for visibility than gray
  3. bringing support for slack app
  4. Removing the blinking textbox "Building" which can be disturbing
  5. disable the app when the active windows is a browser (chrome, firefox etc...), most user will probably be using an extension already.
  6. avoiding displaying multiple button on top of eachother (screenshot from nautilus)
    image
  7. disable the need to hit spacebar at the end
  8. In nautilus sidebar, the textbox correctly show up. However, when triggered, they just "select" the menu, and not open it. It's suprising since with the mouse a single click is enough to open.
    image
@phil294
Copy link
Owner

phil294 commented Nov 12, 2023

Hi!

like in homerow, the textbox could be shown under the button, not to hide it

not a fan of this personally, but I'll give it a try. If you want you can also already do this yourself by adding something like

y += 15

below this line https://github.com/phil294/vimium-everywhere/blob/master/vimium-everywhere.ahk#L222

In general, this program consists of merely 300 LOC, so modifications shouldn't be too difficult. (In Simple mode, it could be reduced to only 30 or so I guess)

yellow textbox could be better for visibility than gray

sounds right. Is it actually gray for you? Interestingly enough, it's light blue for me, even though I have no idea why

bringing support for slack app

This is an electron app, isn't it? Thus it will probably already work if you configure it appropriately as written in the Readme. If not, what have you tried?

Removing the blinking textbox "Building" which can be disturbing

Actually it shouldn't keep blinking all the time, I think this is an error in the source... I don't use normal mode myself so I'm unsure here

disable the app when the active windows is a browser (chrome, firefox etc...), most user will probably be using an extension already.

this can be configured at the top of the source right now. But indeed it would be better to expose a non-programmatic way of configuring this

avoiding displaying multiple button on top of eachother (screenshot from nautilus)

yes, but this is probably more or less impossible with the current architecture

disable the need to hit spacebar at the end

possible but complicated

In nautilus sidebar, the textbox correctly show up. However, when triggered, they just "select" the menu, and not open it. It's suprising since with the mouse a single click is enough to open.

ok, will try to fix soon!

Thanks for all these suggestions and bug reports, very much appreciated! Don't know when I will get to all of this, but there's definitely still a lot to do.

@joihn
Copy link
Author

joihn commented Nov 16, 2023

Thanks for this complete answer !

This is an electron app, isn't it? Thus it will probably already work if you configure it appropriately as written in the Readme. If not, what have you tried?
You are right, it's electron, will try this, thanks

I see great potential in this tool :) on MacOS, there exist "Homerow" which is quite popular, but on linux this the best attempt i have seen, thanks for your work !

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

2 participants