Merge branch 'master' into develop

Conflicts:
extension/lib/commands.coffee

Enable the `braces_spacing` coffeelint rule

The more lint rules enabled, the easier to collaborate.

Thank god for regex substitutions.

Increase the `max_line_length` rule to error

Enable the `missing_fat_arrow` coffeelint rule

To prevent mistakes like the one fixed in commit 5d2a01bf5.

Fix error in the 'underflow' event

Simplify `utils.removeDuplicates`

Make `gi` find text inputs in all frames

Fix nothing happening when clicking some buttons

... using `f`. This was broken by commit 7b69d5f6.

Fix missing hints for links with floated children

Previously, we used `nsIDOMWindowUtils#nodesFromRect()` to get markable element
candidates, because of its speed. However, it turned out that that method does
not return elements with zero area, such as links with floated children. That
caused those links to not get a hint.

This commit switches back to `document.getElementsByTagName('*')`. Due to some
reordering of element filtering there seems to be no difference in performance!

Fixes #584.

Make Facebook's comment fields focusable using `f`

Fixes #524.

Fix un-blurrable comment fields on Facebook

We used to only blur focusable elements (`.tabIndex > -1`) to "interfere with
the browser as little as possible." However, this made it impossible to blur
Facebook's comment fields. _Not_ being able to blur things is what interferes.

Fixes #604.

Improve count and wrap semantics for tab select/move

When using one of `J`, `K`, `gJ` and `gK` with a count, you usually don't want
to wrap around the tab bar, in case you counted too large a number. Then it's
less confusing what happened if it stops at the edge of the tab bar. This is how
it already works.

However, if you're at one of the ends of the tab bar and cannot move in the
chosen direction, there is no reason to forbid a count. Before this commit we
did, making the command a no-op. Now it is allowed. As an example, you can
select the second to last tab by pressing `2J` if you're currently on the first
tab.

The commands only wrap _once._ In other words, if you're on the first of five
tabs, pressing `6J` takes you back to the first tab (five moves), discarding the
last move instead of wrapping again to the last tab. This is useful if you count
too large a number.

See #578.

Handle {,non-}pinned tabs separately in `gJ` and `gK`

Previously, it was possible to move a non-pinned tab into the pinned tabs,
making it pinned, and to move a pinnned tab into the non-pinned tabs, making it
non-pinned.

Now, that's not the case anymore. The first non-pinned tab wraps to the last
tab, and the last tab wraps to the first non-pinned tab, and vice versa for
pinned tabs.

In other words, you can no longer use `gJ` and `gK` to change the pinned state
of a tab. Only `gp` can.

See #606.

Fix issues with scrolling and frames

Since commit 4958beb3b we use `MutationObserver`s to keep
`vim.state.scrollableElements` up-to-date when scrollable elements are removed.
However, there are two more ways elements can be removed if the are located
inside a frame:

- If the whole frame is removed.
- If the `src` attribute of the frame is changed.

This caused "TypeError: can't access dead object" errors on gmail.com.

This commit takes care of those two frame cases as well.

See #605.

Remove `utils.timeIt(fn)`

It required `fn` to be synchronous. Since the multi-process support, very little
is anymore, making it easier to add `console.time('foo')` and
`console.timeEnd('foo')` where needed.

Add count support to `g0`, `g^` and `g$` like Vimium

See #610.

Fix problem with counts and shortcut including `0`

For instance, `5g0` didn't trigger the `g0` command. This was broken in commit
e552942c.

Fix lint errors

Fix quirks after dragging tab to another window

The quirks only happened in multi-process, where a new `vim` instance is not
instantiated for the dragged tab. We already update `.browser` and `.window`.
However, all frame script listeners were still using the message manager of the
old `<browser>`, making the `vim` instance miss out on its frame script
messages. For example, this caused the blacklist not to apply when reloading the
page in a tab dragged to another window. This commit adds the listeners for the
new `<browser>` as well.

Don't lose state when loading tabs in the background

'locationChange' used to be detected in a rather complicated fashion. While this
worked mostly it had problems with tabs loading in the background.
'locationChange' wouldn't fire until the a background tab was _selected,_
causing any state, such as scrollable elements, collected up to that point to be
lost. It also meant that the blacklist patterns were applied to late, which
could cause unwanted autofocus prevention, for example.

Using the 'readystatechange' inside frame scripts seems to do exactly what we
want. This commit switches to that technique, which also simplifies the code a
lot, removing many slightly hacky parts.

See #605.

Fix error in api.md

Style hints for scrollable elements differently

Making it easier to understand which hints are for scrollable elements.

See #605.

Allow to style markers based on the type of element

... by adding `[data-type="scrollable"]` etc. to them.

See #605.

Keep the largest scrollable element up-to-date

We used to not try to find a new largest scrollable element if the currently
largest scrollable element underflowed (stopped being scrollable), because it
was considered "unlikely". That might be true for "classic static" web pages,
but it is not for JavaScript-heavy sites, such as groups.google.com, which may
replace most of the DOM at any time. Therefore, we now _do_ try to find a new
largest scrollable element when the old stops being scrollable.

The above required changing the storage for the scrollable elements from a
`WeakSet` to a `Set`, in order to be able to enumerate the set. The reason
`WeakSet` was used before was to avoid memory leaks. However, that should not be
a problem, because we now remove elements removed from the DOM from the set,
because of the next paragraph. Also, all state is wiped when the page is left.

Unfortunately, the 'underflow' event is not fired for scrollable elements
removed from the DOM. Therefore that is tracked separately using
`MutationObserver`s.

See #605.

Choose which element to scroll more wisely

'overflow' events _are_ triggered for `<html>`, if it overflows. This means that
there is no need to try to detect if `<html>` is scrollable in `commands.scroll`
using `.scrollHeight` and `.clientHeight`; we can simply use
`vim.state.largestScrollableElement`.

The above is not only a code cleanup; it also fixes a bug, mentioned in #605:

On some sites (such as groups.google.com), `<html>` might overflow, but not be
scrollable because of `overflow: hidden;`. The old `.scrollHeight` vs.
`.clientHeight` technique did not catch this, and wrongly tried to scroll
`<html>` when there was a more appropriate scrollable element on the page.
`overflow: hidden;` _is_ taken care of for `vim.state.scrollableElements` and
`vim.state.largestScrollableElement`, though, fixing the problem.

This clears the confusion from commit 59cb2a9a:

> Revert "Remove unnecessary code in commands.follow{,_focus}"
>
> This reverts commit 1a77a5d62036ee0d6029b36ee3c2ef33355b0d85.
>
> Apparently, I must be wrong. I get a hint marker for the entire page (if it is
> scrollable) after this commit. So confused now.

As noted above, `<html>` (`document.documentElement`) _does_ get 'overflow'
events, and thus ends up in `vim.state.scrollableElements`, so we _should_
exclude `document.documentElement` from getting a marker. There is no need to
deal with quirks mode here, though. In quirks mode, `<body>` is considered the
root element and must be scrolled instead of `<html>`, but 'overflow' events are
still triggered for `<html>`, and _not_ for `<body>`.

Revert "Remove unnecessary code in commands.follow{,_focus}"

This reverts commit 1a77a5d62036ee0d6029b36ee3c2ef33355b0d85.

Apparently, I must be wrong. I get a hint marker for the entire page (if it is
scrollable) after this commit. So confused now.

Don't stay in "`gi` mode" if only one text input

Don't consider `<select>`s as text inputs for `gi`

Remove unnecessary code in commands.follow{,_focus}

`document.documentElement` never emits an 'overflow' event even if it is
scrollable, and will thus never exist in `vim.state.scrollableElements`.

Fix prev/next patterns order not respected

Fixes #602.

Add missing word in options.md

Update scroll documentation

Prepare for Tab Groups being removed from Firefox

- Make sure that nothing crashes if `window.TabView` ceases to exist.
- Add TODO comments reminding to remove unnecessary when it is time.

<https://support.mozilla.org/kb/tab-groups-removal>

VimFx v0.6.2

Fix `P` compatibility with InstandFox

Fixes #595.

Fix commands not working on slowly-loading pages

This was broken in commit b50f5ea5.

See #588.

Consistently use 2-space indentation in style.css

Don't give the same hint to all `<a href="#?">`

Fixes #596.

Many websites use `<a href="#">` instead of `<button>`. Apparently, some also
use `<a href="#?">`.

Fix toolbar button icon size in high dpi

Fixes #591.

Add an example to prev/next page patterns docs

Enhance scrolling when entire page isn't scrollable

... by scrolling the largest scrollable element instead.

Fixes #585.

Fix hint markers placement when using Zoom text only

Fixes #594.

Allow to use numbers as shortcuts, overriding counts

... properly handling the special-case for `0`:

- `0J` is _not_ a count, because a count of zero makes no sense. It first
  invokes the `0` command, and then the `J` command.
- `20J` _is_ a count, repeating twenty times. In other words, pressing `2` and
  then `0` does _not_ run the `0` command (scroll to the far left) with a count
  of two. Instead, it sets the waiting count to twenty.

Fixes #587.

Improve formatting of `<li>`s in `gulp {changelog,readme}`

Document `gulp --silent`

VimFx v0.6.1

Fix `<tab>` behavior in browser UI

Fixes #571. Also see #572, which suggested the method used in this commit.

Add migration to add `<force>` to `mode.normal.esc`

We already migrate old shortcut customizations to the new 0.6.0 format. However,
I forgot that if the user had customized the shortcut for the esc command, it
most likely needs `<force>` at the beginning as well.

This commit adds `<force>` to every shortcut for that command (if needed).
Ideally this should have been done already in the 0.6.0 release. Now we run the
tiny risk of adding `<force>` to a shortcut that the user actually intended not
to have `<force>` in, but it's worth it: See #577, #571, #582 and #583.

Add an advanced option to disable notifications

Fixes #576.

Add a question to question-and-answers.md

Reset `text-shadow` for the help dialog

The dialog may inherit it from light-weight themes, making it almost unreadable.

See also commit 238dc6c.

Reset `text-shadow` for hint markers

They may inherit it from light-weight themes, making them almost unreadable.

Fixes #574.

Use consistent casing for all pref titles in en-US

See #571.

Make `gulp readme` and `gulp changelg` AMO compatible

AMO only allows a small subset of HTML, and converts newlines to `<br>`. This
commit makes the mentioned commands output such text.

VimFx v0.6.0

Silence AMO `Function` warnings

The AMO validator thinks that if `Function` appears anywhere, it is a type of
`eval`. However, we use it to access `Function.prototype.call`. This commit
replaces the `Function` occurances with other methods in order to silence those
warnings.

Fix invalid Firefox maximum version

The version has to be one of those listed here:

https://addons.mozilla.org/en-US/firefox/pages/appversions/

Commit c1bc315c tried to simplify the compatible Firefox version management by
using `*` as the maximum version. However, that is not allowed when uploading to
AMO.

Fix invalid Firefox minimum version

The version has to be one of those listed here:

https://addons.mozilla.org/en-US/firefox/pages/appversions/

Apparently, `40` is invalid, while `40.0` is valid.

Fix error in `gulp release`

One shouldn't _both_ return a stream (thanks to CoffeeScript's implicit returns)
_and_ call the callback.

Update API stability note

Update CONTRIBUTING-CODE.md

Update information about master vs. develop and versioning.

Point links in readme to master rather than develop

Tiny code cleanup

Add missing `return`s after `for` loops

Fix mailing list link error in readme

Add changelog for 0.6.0

Tiny documentation fixes

Merge pull request #570 from atilacamurca/locale-pt-br

Update locale messages of pt-BR.

Update locale messages of pt-BR.

Merge pull request #567 from mymedia2/develop

Update russian locale

Update russian locale

Fix scrolling in pages without a doctype

Make it possible to blur scrollable elements

Scrollable elements _are_ focusable (both by using `<tab>` and by using VimFx's
`f` commands), but they aren't marked as such by the browser. In order to
interfere as little as possible, VimFx only blurs elements marked as focusable.
This made it impossible to blur scrollable elements, preventing you from
scrolling the entire page again. This commits now also allows scrollable
elements to blurred as a special-case.

Remove trailing space in "it" locale

Update dependencies

Fix global variable leaks in frame scripts

Apparently, frame scripts for the same `<browser>` but from different add-ons
share the same scope. Previously, bootstrap.coffee used to set up a few global
variables. However, those were then leaked to other add-ons in frame scripts.
Now they are explicitly set in the `scope` of each `require`d module instead.

Make checkboxes in about:preferences markable

Merge pull request #561 from cbertoldi/develop

Added italian translation.

Added italian translation.

Update copyright notices

Document custom commands that access web page content

Suggest using vimfx.js as a config file

- bootstrap.js cannot handle non-ASCII characters, but a loaded sub-script can.
- It's cleaner.

Fix "tab dragged to other window" detection

Previously, it detected some tabs opened in the same window as just dragged to
another window.

Fix uncaught excecption on startup

Play nicer with the dev tools

- Previously, events that occured in the dev tools were erraneously marked as
  frame events, which made it impossible to blur the dev tools using VimFx.
- Improved Escape key handling: Blurring the dev tools using the Escape key no
  longer toggles the split console.

API loading: Better safe than sorry

Make hints mode more robust

- Revive `hints.removeHints()`. While looking for an element that happens to
  have the VimFx container ID and removing it perhaps is not very elegant, it is
  very robust. If the reference to the hints container is ever lost (bugs
  inevitably slip through, or if `gulp` is run while in hints mode), unremovable
  hints are left on the screen.

- Enter hints mode _immediately_ when invoking one of the `f` commands. Hints
  mode used to be entered first when the frame script had found all markable
  elements. Remember that communicating with a frame script is asynchronous, and
  that finding all markable elements may take time. In that time span, user key
  presses should be handled in hints mode, not in normal mode.

Increase current mode display {extensi,hacka}bility

This commit makes it easy to highlight the current mode using custom CSS and/or
JS, by exposing the logic used for the button.

Add documentation for the button

Use the red icon for ignore mode (for the button)

Simply mkaing it grayscale didn't stand out enough. I had a hard time seeing the
difference sometimes. Besides, it's a shame having that beautiful red version
lying around but not being used.

Optimize the API

- Before, if an add-on called `vimfx.set()` around 200 times it could delay
  Firefox's startup time with almost a second. Now there's no delay at all.

- Marks internal variables in public.coffee with a leading underscore. There's
  nothing stopping people from importing them, but no they at least look
  private.

- Remove `vimfx.refresh()`. That is done automatically now, in an efficient way.

- This also makes the test's teardown more robust.

Fix typos in api.md

Fix typo in vimfx.coffee

Make tests more robust by providing teardown hook

Fix broken setting of the API URL pref

Accidentally broken in commit 33ed14df. I hadn't noticed before, since the
correct pref still remained in my profiles. Testing in a new profile revealed
the error.

Fix crash when installing VimFx after `gulp -t`

Get the public API the same way in tests as in docs

Add `vimfx.getDefault()` to the public API

Increase hackability of the help dialog

Provide `data-mode`, `data-category` and `data-command` attributes, that can be
used by custom CSS to, for example, hide unwanted modes, categories and/or
commands from the dialog.

Fix handling of tabs dragged to new/other windows

See #57. This was originally fixed in commit 35522821, but regressed after
adding multi-process support. This fixes it again.