philc / philc/vimium

When searching, spaces are replaced with "+" instead of "%20"

Open
#3,583 9 comments 1 reaction 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
JavaScript
Stars
27k
Forks
2.6k
PR merge metrics
No merged PRs in 30d

Description

Describe the bug

As per the title, spaces are replaced with "+" while some websites require "%20" for space delimiters.

One example is https://www.rottentomatoes.com/search?search=alice%20in%20wonderland

To Reproduce

  1. Configure vimium custom search engine with
r: https://www.rottentomatoes.com/search?search=%s Rotten Tomatoes
  1. Search Rottentomatoes with "Alice in Wonderland"

Incorrect results given.

Brave Browser 1.8.90 and Vimium 1.66

I have a branch with a working fix for this already if this is something you guys would like addressed :)

Edit: Created a PR

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with Vimium's custom search engine configuration and reproduce the Rotten Tomatoes query using the URL and steps in the issue. Verify that searching for "Alice in Wonderland" produces spaces encoded as %20 rather than +, then confirm the corrected search URL returns the expected results.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
web-dev
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.