Skip to content

Commit

Permalink
Disable urlseparator by default, for #319 (#747)
Browse files Browse the repository at this point in the history
* Disable urlseparator by default

As explained in #319, this separator is too common in URLs and search strings to be a sensible default separator.
* When copying and pasting some content to look it up with some search engine (for instance a title or an address);
* A comma followed by spaces is also legitimate in a URL! For instance if you put some JSON as a GET parameter in a URL.

* Update NEWS: disable urlseparator by default
  • Loading branch information
wetneb authored and timss committed Mar 28, 2017
1 parent f3d5824 commit 63f8808
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion common/content/browser.js
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,7 @@ const Browser = Module("browser", {

options.add(["urlseparator"],
"Set the separator regex used to separate multiple URL args",
"string", ",\\s");
"string", "");

options.add(["yankencodedurl"],
"Set the yank mode copying encoded URL",
Expand Down
2 changes: 1 addition & 1 deletion vimperator/NEWS
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
201x-xx-xx:
* create README.md
* fix search highlighting (hlsearch) in firefox 51

* disable urlseparator by default as its unexpected behaviour irritates users

2017-02-01:
* add option 'newtaburl' to set default URL in new tabs
Expand Down

0 comments on commit 63f8808

Please sign in to comment.