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

Would it be possible to add an inverted mode? #2

Open
brian6932 opened this issue Dec 13, 2021 · 6 comments
Open

Would it be possible to add an inverted mode? #2

brian6932 opened this issue Dec 13, 2021 · 6 comments

Comments

@brian6932
Copy link
Contributor

brian6932 commented Dec 13, 2021

It being inverted from vim by default slightly catches me off guard whenever I use this, but I really like the addon. I know I can install this manually and just flip j and k (which I have done), but would it be possible to have a toggle for this?

@brian6932 brian6932 changed the title Would it be possible to make add an inverted mode? Would it be possible to add an inverted mode? Dec 13, 2021
@RogueArt
Copy link
Owner

Huh, in the version I'm using, it matches Vim-style bindings, but in the repository code, it's flipped. I'll update it to be Vim-style to be default since that's convention and I definitely don't want to break away from that. Thanks for pointing this out!

@RogueArt
Copy link
Owner

RogueArt commented Dec 14, 2021

Done, just inverted the keybinds and tested it on my end. Feel free to reopen the issue if it doesn't work or if you have any more suggestions!

@brian6932
Copy link
Contributor Author

brian6932 commented Dec 14, 2021

@RogueArt Well I tried to swap the same thing on my end locally and installed it, but it seems to scroll as opposed to go to next item, I think the setFocus function may need to be edited as well? 🤔 Not sure why it happened tho, also the AMO version has the same inverted scroll, so a new release might be necessary.

@RogueArt RogueArt reopened this Jan 23, 2022
@RogueArt
Copy link
Owner

That's strange that it seems to scroll on yours, mine jumps between the next and previous item on the latest versions of Firefox and Chrome.

On the bright side, I made a new release on Firefox inverting the keybind as well as several bug fixes with one on the setFocus function. You could try reinstalling the latest version of the plugin (0.2.2) and see if the scrolling issue is fixed on your end.

I'm back to working on this extension now, so I'll keep this issue open as I want to let users set their own keybinds anyways.

@brian6932
Copy link
Contributor Author

brian6932 commented Jan 23, 2022

@RogueArt I recorded this bug, however, I am unsure of whether I should post it in this issue or create a new one, which is preferable? It occurs both in the new releases, and when I web-ext build it from the repo.

@RogueArt
Copy link
Owner

Feel free to create a new issue for it. Thanks for finding it and taking the time to report it!

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