Add es locale

Fix typo in the en-US locale

Merge branch 'lint'

Update code style guidelines

Replace `when ... then` with multliners

For the sake of smaller diffs, simpler merges and consistency.

Reduce unnecessary vertical code alignment

While it may look nice many times, it causes annoying merge conflicts.
Especially aligning all `require(...)` calls has bitten me many times, since
adding or removing one often requires reformatting the entire block.

This commit attempts to remove vertical alignment (especially around `=`) for
the sake of smaller diffs and simpler merges.

Vertical alignment has been kept, though, where it really makes the code easier
to read and you're likely to edit all of the aligned lines anyway if changing
one of them. The most common example is code dealing with X and Y coordinates
and having to do the same thing to both of them. Only the words X, Y, left,
right, top, bottom, width, height etc. differ between the lines. It's easier to
see that they are really up-to-date with each other if the common parts are
aligned.

Enable the `colon_assignment_spacing` CoffeeLint rule

Exceptions are made for the DSL in defaults.coffee and migrations.coffee.

Work around a CoffeeLint bug

A `no_implicit_braces` false positive.

Enable the `no_implicit_braces` CoffeeLint rule

Exceptions are made for the DSL in defaults.coffee and migrations.coffee.

Enable the `no_plusplus` CoffeeLint rule

Enable the `newlines_after_classes` CoffeeLint rule

Tweak issue templates

Replace CONTRIBUTING.md with issue template

Tweak code contribution documentation

Improve error messages for l10n.coffee

Fix `vim.mode` being `undefined` in 'modeDisplayChange'

Fix uncaught exception after dragging tabs between windows

Document a NoScript bug

Closes #588.

Improve and add more documentation

- Improved wording and consistency.
- Add short summaries of each section in the table of contents.
- Document how to install VimFx. Fixes #686.
- Document modes.
- Don't refer to the "Public API" anymore, but the "config file API". It is no
  longer intended to be used by other add-ons. (The plan is to load config files
  in a different way in the future.)

Improve CONTRIBUTING.md

- Remove "Use English". That's understood since the bug template is in English.
- Put the instruction about bugs at the top.
- Make the paragraph about the changelog a bullet point. It makes the
  instructions look shorter. Since the "Use English" bullet point was removed,
  the list does not look too long either.
- Move the "Contributing code?" link off to the right to get it out of the way.
- Use fewer words in a few places. The less to read the better.

Also: Link to CONTRIBUTING.md instead of directly to the issue tracker in the
readme, to increase the chance of people actually reading it. Therefore a "Go to
the issue tracker" link was added in CONTRIBUTING.md (it goes to the issues
list, not a new issue, to increase the chance of people looking for duplicates).

Apply the blacklist faster on some pages

Fixes #688.

VimFx v0.12.0

Make the `H` and `L` commands more reliable

It seems like using `window.BrowserBack()` and `window.BrowserForward()` are the
most reliable ways of navigating the history, rather than having to wait for
`SessionStore.getSessionHistory()` to finish and then going to a certain history
index. This commit optimizes the case where the count is 1 to use the mentioned
functions instead. Some extensions also override those, so calling them results
in better interoperability.

(Hopefully/Likely) fixes #687.

To keep the code changes simple, the support for Firefox < 43 was dropped.
That's fine, since Firefox 44 has already been released.

Silence `addons-linter` warnings

Merge branch 'master' into develop

Add `npm run addons-linter`

s/Keyboard layout/Keyboard language/ in CONTRIBUTING.md

It's really the language (en-US, en-UK, sv-SE, de, ru, etc.) of the keyboard
that is relevant. (Many of those examples are variations of QWERTY.)

Convert most of CONTRIBUTING.md into copy-paste template

Recognize Codeacademy terminals as text inputs

Fixes #680.

Add handy links to location bar docs

Related: #540.

Use async messaging in bootstrap.coffee

The initial message from new frame scripts is now async. This is possible since
commit bd00a85e0. Other than following the recommendation to avoid sync
messaging whenever possible, this fixes #679 and fixes #624.

Use explicit `return`s in message-manager.coffee

Update copyright notices

Consistently name the help dialog is user-facing code

Always call it the "Keyboard Shortcuts help dialog".

Use strings instead of functions in the `modes` object

Since we run things properly in the `startup()` function of bootstrap.js since
last commit, the `chrome://` URI to the string bundles is now available when the
`modes` object is constructed. This means that there's no need anymore to let
mode names, category names and command descriptions be functions. Instead they
can be the sane thing: Simply strings.

Note that this is a backwards incompatible change. Luckily it does not affect
the `.addCommand()` API method, so propably won't impact anyone.

Use `chrome://` URIs for `require()` calls

Apparently, `chrome://` URIs registered in chrome.manifest cannot be used in
bootstrap.js until its `startup()` function is called. Therefore this commit
reorganizes the code of bootstrap.coffee a bit, so that actual work is done
inside `startup()` (where it is supposed to be, after all). Outside of that
function we now mostly simply declare variables that the `shutdown()` function
needs to access as well. Some things were moved into main.coffee as well.

- Using `chrome://` URIs is the way add-ons are "supposed" to do it.
- If a `require()` call fails, we now get much shorter URIs in the error
  message, making things easier to debug.
- This allows to get rid of the hack of sending `__SCRIPT_URI_SPEC__` to frame
  scripts.

Improve CPU usage in Hints mode

Raising the "check for marker element movements" interval from 0 to 15 results
in no noticeable CPU usage change on my system, while still allowing markers to
move smoothly. Setting it to 20 results in noticeable stutter.

This commit also add the `hints_sleep` pref to control the interval, or disable
the feature altogether.

Additionally, the interval now doesn't run at all for the `zF` command.
Previously, it ran as usual, but didn't do anything (as marker movement isn't
supported (YAGNI) for the `zF` command).

Fixes #678.

Fix missing hint markers for tabs on caniuse.com

The "feature" tabs for a specific feature on caniuse.com look like this:

    ___________ ___________
    |         :\|         :\
    |  Tab 1  :+\  Tab 2  : \
    |_________:__\________:__\

The "real" tab elements are the rectangles to the left of the dotted vertical
lines. The triangle parts are made using the `::before` or `::after`
pseudo-elements of the tabs.

When calling `.elementFromPoint()` on a pseudo-element, its parent element is
returned. But calling `.getBoundingClientRect()` on the parent element returns a
rectangle only around the the parent element, excluding the pseudo-element.

Because of the pseudo-elements overlap the next tab, the second tab is
considered covered by the first tab.

Previously, a point to the right of the first tab would then be tried. But since
the pseudo-element isn't included in `.getBoundingClientRect()` that would mean
the point marked with a `+`, which is actually to the _left_ of the original
point, not to the right! That obviously didn't work.

With this commit, if the "try-right point" is to the left of the initial point
and the covering element and the queried element are in the same frame, the
point is tried again but this time forcing the element to be considered located
at that point, solving the issue.

Using pseudo-elements to create non-rectangular tabs is quite common, so this
will help on other sites as well.

Add tips about the menu key and access keys

Fix the empty notification for the `gH` command

Briefly mention "click to edit" in the readme

Merge branch 'master' into develop

Fix AMO formatting issue by forcing a line break

Simplify "Questions & Answers" a bit

Streamline the readme

Mention "Questions & Answers" directly in the readme

s/Google Chrome/Chrome/ in the readme

Improve discoverability of Ignore mode

See #593.

Remove check for 'close' class for clickable elements

It gave too many false positives because of the word 'closed' in many classes.
Instead of trying to switch to a regex to avoid those cases, I'm trying to
completely remove the check. Hopefully the other matching rules are enough.

Add count support to `gl`

Remove hints and help on shutdown

Previously, if you disabled VimFx by using the `f` command to click VimFx's
"Disable" button in the Add-ons Manager, the hint markers would be left behind
(though losing their styling). This meant that the entire page would be covered
by a transparent element impossible to remove (other than closing the tab) and
impossible to click through.

A similar thing happened if you left the help dialog open in another tab while
disabling VimFx.

This commit makes sure that both those pieces of UI are removed on shutdown.

Comment out "copy help HTML" to clipboard

Enabling it was accidentally committed in commit 6f8cb627c.

Merge pull request #676 from akhodakivskiy/more-hints

Recognize more elements as clickable

Recognize more elements as clickable

- Elements with `data-` attributes used for clickable things by Bootstrap are
  now recognized. This is a huge win since Bootstrap is very popular and many
  developers use it with semantically unclickable elements (even though the
  Bootstrap documentation always reminds of accessibility concerns). Since these
  elements are or _should have been_ semantically clickable elements, they are
  always treated as such, in order not to give them too bad hints.

- A few `aria-` attributes are now recognized as semantically clickable.

- ARIA roles are now treated as _semantically_ clickable, instead of just
  clickable.

- "Close" buttons are now attempted to be recognized. This is useful for many of
  those obtrusive EU cookie law notices. Recognizing Bootstrap elements also
  helps greatly here.

- Finally, elements with 'click' event listeners (added using
  `element.addEventListener('click', ...)`) are now recognized, by using a
  Firefox API. This is really useful on DuckDuckGo's image search. Fixes #671
  and fixes #672.

  Note that event delegation is not supported, since that would require
  inspecting the code of the event listeners. One _could_ inspect the global
  `jQuery` object (if any) to get to solve that problem to a great extent (like
  Firefox's devtools do), but I'm not sure if that's a good idea.

It should also be noted that I tried recognizing elements matching
`/\bjs-/.test(element.className)`, but that proved give too many false
positives.

Never prevent autofocus in XUL pages

- Every time I go to `about:config` I wish that the filter field was focused.
- Even though `vim.markPageInteraction()` was called, the "click to go to
  command" feature of the help dialog sometimes failed to autofocus the text
  input for the command.

Thus, I think it's sensible to simply _not_ prevent autofocus in XUL pages. Such
pages are few and well-behaved (it seems).

Get XPCOM interfaces once at the top of files

Improve autofocus prevention after `[` and `]`

When you go to the next page of GitHub’s code search results, the page is loaded
with AJAX. GitHub then annoyingly focuses its search input. This autofocus
cannot be prevented in a reliable way, because the case is indistinguishable
from a button whose job is to focus some text input. However, after the `[` and
`]` command we know for sure that we can prevent the next focus. This commit
resets the `vim.state.hasInteraction` flag in those commands to allow for this
behavior, by adding the `value=true` parameter to `vim.markPageInteraction()`
and passing in `false`.

Fixes #675.

Improve "click to edit" in help dialog in non-multi-process

Allow to click commands in the help dialog to edit them

Remove unnecessary hint markers in the Add-ons Manager

Fix uncaught exception when opening the devtools

Fix `gulp help.html`

Remove confusing link from the readme

Code style nitpicks

Properly clean up public API on shutdown (fix typo)

Merge pull request #669 from mozillazg/zh-CN

updated zh-CN locale

updated zh-CN locale

VimFx v0.11.0

Fix uncaught exception when loading some pages

In frame scripts, `document.activeElement` may be null while the page is
loading.

Reduce AMO validator warnings regarding mouse events

Future-proof the `Cu.import` for `console`

Fix support for Firefox < 44

Update dependencies

Update copyright notices

Merge branch 'master' into develop

Make the tests run on newer Firefox versions

Merge branch 'master' into develop

Conflicts:
extension/lib/marker.coffee

Fix missing hint for "Browser styles" checkbox in devtools

Merge pull request #668 from akhodakivskiy/devtoolszoom

Fix misplaced devtools hint markers when zoomed

Fix misplaced hint markers when zoomed

Hint markers are supposed to be centered vertically on the non-covered point.
This didn't work properly when zoomed, but now does. For simplicity, that fix
removed the "make sure the marker stays within its element" constraint, which
turned out to be nice and more consistent. The hint markers are now _always_
centered vertically on the non-covered point, even for very small links.

This commit also fixes hint marker placement in the devtools when the devtools
have been independently zoomed. The above issue needed to be fixed before this
could be fixed. Fixes #667.

Improve docs regarding adding shortcuts to custom commands

Fix marker movement when zoomed

Merge pull request #666 from just-barcodes/master

Update de locale

Update de locale

Fix memory leak in non-multi-process Firefox

Apparently, 'overflow' and 'underflow' events in web page content can be caught
in UI listeners in non-multi-process. `event.target` of 'overflow' events in the
browser UI are saved as scrollable elements, which accidentaly included web page
elements as well. Holding on to such elements after their owner tab is closed
caused a memory leak (ghost windows). This commit checks that the overflown
elements really are UI elements before saving them.

Fixes #664.

Don't use the same hint for GitHub's diff expansion buttons

Links with a `data-url` attribute no longer get the same hint. They're likely
JavaScript-powered even on other sites.

Support the "Default FullZoom Level" extension

Fixes #665.

Fix Hints mode crash on some pages

`elementAtPoint` might be `null`.

Merge branch 'master' into develop

Improve `zf` hover support: Emit mouse events

Fix `:hover` clearing on click in multi-process

Because clicks simulated by VimFx apparently cannot be caught in non-frame
listeners (which is good), _two_ real clicks were needed to clear `:hover`. This
commit fixes that, which allowed for a nice code cleanup. Instead, a temporary
hack is used for non-multi-process to make it work there.

Make simulated clicks look more like real clicks

Fix broken `scroll_to_mark` command

Fixes #663

Make markers move along with its element

Fixes #503.

Clear locked `:hover` on click

Fix missing hints in the Add-ons Manager

Merge branch 'master' into develop

Add a command to select the most recent tab

Fixes #319.

Lock `:hover` when clicking with the `f` commands

Fixes #388.

Note that this commit also adds `pointer-evets: none;` to the hints container,
allowing to generate hints for elements only visible because of the mouse
pointer hovering something. Getting hints for hover-menus opened by `zf` but not
for hover-menus opened by moving the mouse was too confusing. The reason
`pointer-events: none;` wasn’t added before was that it prevented from scrolling
while hints were shown. While that was nice, it was just an edge case that
doesn't really matter. It is better to allow hints for hover-menus.

Add support for various fancy text editors

Fixes #335.