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

Vim Mode 2022 #439

Open
wants to merge 89 commits into
base: master
Choose a base branch
from
Open

Vim Mode 2022 #439

wants to merge 89 commits into from

Conversation

tombh
Copy link
Member

@tombh tombh commented Jul 26, 2022

Can I finally get Vim Mode merged?

Fixes #236, #246 and #264

tobimensch and others added 30 commits October 7, 2018 14:45
The keybindings are hardcoded for now, but this is going to change.
Added moveTabLeft and moveTabRight functions, which take
a tab ID and try to move the tab as far right or left
in the tabOrder as possible.
Added previouslyVisitedTab function that switches to the
previously selected tab.
with activating input boxes using link hinting.
In Firefox 63 an old syntax for Marionette commands was deprecated.
Updating mostly just meant prepending `WebDriver` to existing commands.

This should fix most problems in #232
Vim mode still needs a lot more tests
…to key combinations only working after a certain number of key strokes.
tobimensch and others added 19 commits June 24, 2019 09:09
with activating input boxes using link hinting.
Vim mode still needs a lot more tests
…to key combinations only working after a certain number of key strokes.
This is because Travis' logs had 2 problems.
  1. it doesn't capture the entire log output
  2. it doesn't show logs when there's a timeout
… event could get interpreted repeatedly. Also some rewrites/improvements of the code. Key mappings can now contain control characters and meta keys with a vim-like notation. There's a hard insert mode, which disables all of browsh's shortcuts and requires 4 hits on ESC to leave. There's a new multiple link opening feature analogous to vimium, that's still incomplete.
After nearly 4 years I'm finally getting round to shipping all the
tremendous hard work of @tobimensch, @j-rewerts and @ed2k. I can't
apologise enough that it's taken so long.
@laughingbiscuit
Copy link

do you want to community to use and test for your fork prior to merge?

@martin-braun
Copy link

vimium in browsh will be a game changer for me!

@jivank
Copy link

jivank commented Jan 19, 2023

I was having trouble building this branch with the provided documentation. I assume the repo has a GitHub Action that successfully builds?

@tombh
Copy link
Member Author

tombh commented Jan 27, 2023

I would love help with this. The only thing that needs fixing to get this merged is the failing tests.

@jivank The local dev setup docs do indeed need updating. If you're on Linux or similar you can use https://github.com/browsh-org/browsh/blob/master/.github/workflows/main.yml for inspiration.

@jivank
Copy link

jivank commented Feb 3, 2023

I see in TTY tests:


Error: test/tty/tty_test.go:4:2: package browsh/interfacer/src/browsh is not in GOROOT (/opt/hostedtoolcache/go/1.18.10/x64/src/browsh/interfacer/src/browsh)

This is a similar error I was facing when I was attempting to build it on my local machine.

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

Successfully merging this pull request may close these issues.

None yet

6 participants