Merge pull request #461 from lydell/scroll

Make scrolling commands scroll the focused element

Add commands to scroll to the far left/right

Make scrolling commands scroll the focused element

Fixes #11. If the currently focused element is scrollable, it is scrolled by the
scrolling commands. Otherwise the whole document is scrolled.

In other words, you may now choose which element to scroll with the 'f' or 'vf'
command, and then you can scroll it with the scroll commands.

Fixes #399. Using counts with scrolling commands now works reliably.

The scrolling commands respect built-in Firefox prefs:

- general.smoothScroll: Turn smooth scrolling on/off for _all_ scrolling
  commands.
- general.smoothScroll.lines: h, j, k, l only.
- general.smoothScroll.pages: d, u, <space>, <s-space> only.
- general.smoothScroll.other: gg, G only.
- toolkit.scrollbox.horizontalScrollDistance: Scroll step for h, l.
- toolkit.scrollbox.verticalScrollDistance: Scroll step for j, k.

The two last prefs replace the scroll step setting.

A bunch of new (hidden) prefs let you customize the smooth scrolling, in the
same spirit as the built-in `general.smoothScroll.<type>.duration{Min,Max}MS`.
They're called `smoothScroll.<type>.spring-constant` and work like
`layout.css.scroll-behavior.spring-constant`.

This commit also bumps the minimum Firefox version to 36, since
`Element.prototype.scroll{To,By}` are used and first available in that version.

Update readme

Split CONTRIBUTING.md into two files and improve them

CONTRIBUTING.md now only contains issue reporting instructions, to make it look
less overwhelming.

CONTRIBUTING-CODE.md contains the rest. Added information about the current
state of development, about the upcoming 0.6.0 release and the future rapid
release cycle plans.

Remove 0.6.0 upgrade notice

Because the develop branch is now the default. Let's re-add it when 0.6.0 is
actually released.

Improve placement of hint markers when zoomed

Every other zoom level they seem to be a tiny bit off, but this is definitely
much better than it was before.

Add `gulp faster`

Merge pull request #487 from lindhe/develop

Fixed a small but vital typo

Fixed a small but vital typo

Add French translator to PEOPLE.md

Merge pull request #482 from m-r-r/origin/french-translation

Added a french translation (v2)

Added a french translation

Do not treat `<a href="">` as links

Some sites use those as buttons. `element.href === window.location` in that
case. Otherwise several such "buttons" might get the same hint.

Make it easier to run gulp without installing it globally

You may now type `npm run gulp --` instead of `gulp`. Thanks to @marlun for the
suggestion. Closes #467.

Make it possible to distinguish count 1 from no count

This way 'gi' can focus the last focused input, while '1gi' focuses the first
input no matter what.

Fix #47: 'gi' to focus last focused or first text input

'gi' focuses the last focused or the first -- or the count:th to be precise --
text input of the page. Moreover, after you have focused something with 'gi',
<tab> and <s-tab> switch between only text inputs. That's convenient, because
'gi<tab><tab>' is then equivalent to '3gi'. Instead of trying 'gi' followed by
'<esc>2gi' followed by '<esc>3gi' etc. until you get to the input you want, you
can just type 'gi<tab><tab>' instead.

This required updating vim-like-key-notation to support <s-tab> more reliably.

Better vertically centered marker placement

Make sure `gulp release` tags the correct commit

Update dependencies

Reorder vimfx.properties to same order as in commands.coffee

Update CONTRIBUTING.md

Run `gulp sync-locales`

Add `gulp sync-locales`

To make the implementation easier, all translations must be in a single line and
must not contain superfluos whitespace. All .dtd files were updated to reflect
this.

Drastically improve speed up hints generation

We used to apply a class to each element, to work around `border-radius`.
Removing that drastically speeds up hints generation. The "loss" is that the
markers no longer appear in the top-left corner, but rather in the left-middle.
That takes a little bit of getting used to, but is definitely worth it.

Reduce the number of hints on Google Calendar

As noted in https://github.com/akhodakivskiy/VimFx/pull/451#issuecomment-71582664,
Google Calendar gets too many hints. That's because many buttons are implemented
as `<div>`s with lots of `<div>`s inside, all of which have "button" in the
their class names. This commit exludes elements with "button" in the class name
if they contain yet such an element. This works fine on Google Calendar.
Hopefully it does not exlude too much on other pages.

Exclude test scripts from node_modules

Move `Command` into its own file

Refactor `utils.getAllElements` into hints.coffee

It no longer returned all elements as advertised, but instead returned an
optimized result for hints generation, so it belongs in hints.coffee.

Drastically speed up hints generation

By using a Firefox native function for getting all elements inside a given
rectangle. This is way faster than getting _all_ elements, and then manually
checking each element.

Fixes #409. Fixes #329.

Merge branch 'develop' of github.com:akhodakivskiy/VimFx into develop

Fix #459: Don't treat modified numbers as counts

For example, if you press `<c-5>` or `<a-5>` to select the fifth tab, and then
press `J` to select the fourth, it used to be a risk that it was interpreted as
`5J`.

Merge pull request #460 from TheGreenHabi/patch-1

Fix comment typo in utils.coffee

Fix comment typo in utils.coffee

mearues -> measures

Let simulated clicks be cancelable

They used to be, but it was lost in commit 1ee0208. This commit brings it back,
since `<a href="#">` is often used a JavaScript-powered button. If the event
cannot be canceled, clicking such a button will scroll to the top of the page.

(See also commit a4dfda53.)

Merge branch 'TheGreenHabi-patch-1' into develop

Fix #440: Hint markers on feedly.com

Add icon design contributor

Merge pull request #455 from lydell/youtube-space

Youtube space

Update vim-like-key-notation

To support function keys (such as `<F1>`) for shortcuts.

Treat the Youtube video player as an adjustable element

- Pressing `<space>` on the entire player should play/pause.
- Pressing `<space>` on any of its controls should activate that control.
- The arrow keys should let you move the time slider.

Fixes #442.

Enter automatic insert mode when an element is fullscreen

Such as `<video>` tags being in fullscreen.

Merge pull request #451 from lydell/hints

WIP: Hints improvements

Do not consider links without href as proper links

Links ending with a `#` (`<a href="#">`) as well as `javascript:` links are
already excluded, because they're used as buttons, not links. Links without href
at all (`<a>`) should be excluded too, since they are also used as buttons (for
example, the voting buttons on stackoverflow.com). If they're not excluded, all
such links get the same hint, even though all of them do different things
through JavaScript. Moreover, the 'F' and 'gF' commands would try to open them
in new tabs, which of course wouldn't work.

Make upgrade to 0.6.0 notice more visible

Merge branch 'master' into develop

Conflicts:
README.md

Add missing issue reporting instruction

Update readme

Let simulated clicks bubble

They used to, but it was lost in commit 1ee0208. This commit brings it back,
since many buttons are wired up using event delegation, which requires bubbling.

Add Tree Style Tab support for opening new tabs

Since commit 05d60dba5 we no longer open links in new tabs by simulating
ctrl-clicks on them, but instead use Firefox APIs. Before, the new tabs were
detected by the Tree Style Tab extension and opened as new children of the
current tab. After that commit new tabs opened as new _siblings_ after the
current tab instead. This commit opens new tabs as children again, by using the
public [Tree Style Tab API][tst-api].

[tst-api]: http://piro.sakura.ne.jp/xul/_treestyletab.html.en#api.

Make scrollable elements markable

This is the first step at fixing #11. You can now focus scrollable elements and
use the arrow keys to scroll it (which is how Firefox works by default). The
next step is to make VimFx's scrolling commands able to scroll them.

Provide `vim.state`

Instead of just assigning random properties on `vim` objects (such as
`.blacklisted` and `.lastInteraction`) we now assign them to `vim.state`. This
way we can properly initialize those values when new `vim` objects are made, and
easily reset all of them when the URL changes (which means that the state
changes).

Fix #60: Give the same hint to links with the same URL

This results in fewer hints, which results in shorter hints overall.

'vf' is excluded so that everything still is focusable.

`<a href="#">` links, which are commonly used as buttons, are excluded since
clicking them likely does different things via JavaScript.

`<a href="javascript:">` links are also excluded, mainly because
`<a href="javascript:void(0)">` is a commonly used as buttons as well.

Add hint marker support for XUL documents

For example, hint markers may now be used in the Add-ons Manager or in
about:newtab (fixes #51).

Make sure hint markers cannot be line-wrapped

Improve automatic insert mode

- You can now use `<enter>` as a shortcut and still be able to press `<enter>`
  on a focused link or button to activate it (without running the command).
- You can now run commands when for example a radio button is focused, but still
  move the selection with the arrow keys.

Move `rotateOverlappingMarkers()` into marker.coffee

It has nothing to do with hints, but much to do with markers.

Improve markable elements matching

- Each 'f' command now gets its own set of markable elements. For example, it
  does not make sense to open a button in a new tab.
- The above made it really easy to fix #227: Add 'gf' to follow link in new
  foreground/focused tab.
- Links are now opened in new tabs using Firefox APIs instead of simulating
  ctrl-clicks on the links. This is more reliable (and required in order to
  implement the above point).
- XPath is no longer used. Instead, every element in every frame of the page is
  checked with code. This might sound expensive, but it isn't since that's what
  the XPath matcher needs to do behind the scenes anyway. The benefit of doing
  it with code is that we can do more advanced checks (see below), and no
  information is lost along the way: For example, whether or not an element is
  considered "semantic" is saved, instead of needing to be re-calculated
  afterwards.
- The above fixes #320 in a better way than what was already done in commit
  016574212: Elements that have "button" in their class now must not contain an
  `<a>` or `<button>` element, so a marker for a "button-wrapper" is likely not
  even made and therefore cannot cover the real button inside.
- The above also significantly reduced the number of hints on reddit.com, which
  made the all hints shorter.
- While at it, I implemented #450: 'vf' now selects the contents of text inputs.

Correctly determine viewport size

The old method was wrong and caused no hint to show up when viewing an image
directly to toggle zoom on it.

Moreover, if a page would have a very small height and a link absolutely
positioned below that height (but still inside the viewport) that link would not
get a hint either.

These two have been fixed.

Simplify hint matching

Just like normal mode keeps track of the entered keys, hints mode now also keeps
track of the entered keys (chars), instead of letting each marker do it. The
markers now only keep track of themselves; how much they are matched.

Put markers in the browser chrome

Instead of inside the current page. Specifically, the markers are now inserted
at the same place as `alert()` dialogs are. This has a number of advantages:

- We don't need to fight the `z-index`es of the page. The markers will now
  always be on top.
- We don't need to fight any other styling of the page. No reset rules needed.
  The markers will look the same regardless of what page you view.
- The above point means that we can drop `!important` from all declarations.
  This makes it easier to change the styling of the markers using userChrome.css
  or Stylish.
- Just like you cannot interact with the page (such as scrolling) while an
  `alert()` dialog is visible, you cannot interact with the page when hints are
  visible. For example you can no longer scroll, making the hints out of place.

Move lib/mode-hints/* directly into lib/

lib/mode-hints/mode-hints.coffee was very short and fit really well into
lib/modes.coffee. Then there were only two files left in the lib/mode-hints
directory: hints.coffee and marker.coffee, which made that directory not feeling
warranted anymore. Besides, marker.coffee could be refactored into a more
general marker abstraction that could be used for other things than hints as
well.

Remove bloomfilter

The bloomfilter can run out of space, and actually does so pretty quickly:
See #176. When it is full it is not of any help anymore. All it does then is
unconditionally giving better hints to `<a>` elements. But since commit 01657421
that is done in a better way.

In summary, the bloomfilter is not useful anymore.

Give better hints to semantic elements than unsemantic ones

On reddit, each post wrapper has an `onclick` attribute, which makes it
markable. Of course the wrapper has larger area than anything inside of it. This
causes it to get a larger weight than the post title link. The title link is
what the user reads and wants to click, not the entire wrapper. Therefore it
should get a better hint.

There are other similar examples as well. Unsemantic elements (such as those
with `on*` properties, or those whose class contains "button") need to be able
to be clicked, but semantic ones (such as anchors, buttons and form elements)
should be favored.

This commit also fixes #320: Sometimes button elements are wrapped in a div with
a class such as "button-wrapper". Because that contains "button" the wrapper
element gets a marker as well, even though it cannot be clicked. Since it wraps
the actual button it naturally gets a larger weight and therefore a higher
z-index, which makes it overlap the marker for the clickable button. Now,
semantic elements get their z-indexes assigned _after_ the unsemantic ones
instead, which makes them go on top.

Add missing null check for computed style in hints.coffee

Just like commit a0c72d1:

> Fix #447: Regression: Hints generation crashes
>
> Commit e1497ee accidentally removed the null check for `.getComputedStyle()` in
> isElementVisible. This commit adds it back.

Upgrade n-ary-huffman to 2.1

This brings performance improvements.

Merge branch 'develop' of github.com:akhodakivskiy/VimFx into develop

Merge branch 'master' into develop

Conflicts:
CHANGELOG.md
extension/install.rdf

TL;DR-ize the Reporting issues section in CONTRIBUTING.md

So people actually read it.

VimFx v0.5.17

Fix #447: Regression: Hints generation crashes

Commit e1497ee accidentally removed the null check for `.getComputedStyle()` in
isElementVisible. This commit adds it back.

Merge pull request #449 from TheGreenHabi/patch-1

Changed c-escape to s-escape

Changed c-escape to s-escape

Changed keyboard shortcut of __Insert Mode - Return to Normal__ mode from old _c-escape_ to _s-escape_

Merge branch 'master' into develop

Conflicts:
CHANGELOG.md
extension/install.rdf

VimFx v0.5.16

Fix #392: Get vim instance properly onlocationchange

Fix #441: Change default exit insert mode shortcut

We used to have `<c-escape>`, but that conflicts with a standard Windows
keyboard shorcut. Now we use `<s-escape>` instead.

Show actual exit shortcuts in button insert mode tooltip

Merge branch 'master' into develop

Conflicts:
CHANGELOG.md
extension/install.rdf
extension/lib/commands.coffee

VimFx v0.5.15

Make 0.5.x forwards-compatible with 0.6.0

With this commit it will be safe to downgrade from 0.6.0. The new shortcut
format in 0.6.0 will no longer cause VimFx to crash.

Fix broken Customize button in options page

Fix #392: Get vim instance properly onlocationchange

Fix broken Keyboard Shortcuts button menu item

Update localization instructions

Link to the changelog from readme and contributions guide

This is to prevent people from opening issues about backwards-incompatible
changes in the upcoming release.

Update readme

Sync command descriptions in readme with en-US locale

Update commands in readme

Add quote command

Clean up sv-SE locale

Clean up en-US locale

Fix awkward wording for an error message

Merge branch 'lindhe-swedish' into develop

Update sv-SE locale

Created a Swedish translation

Most of it is correct, some minor details are a bit awkward but that's
mostly because not all technical terms ("web terms") has a proper
Swedish translation.

Update commands in readme

Remove alt-shift-v shortcut to toggle VimFx

It has always worked inreliably and cannot be customized. It is better to use
insert mode.

Closes #64. Closes #432.

Don't prevent autofocus in insert mode

This makes insert mode mostly equivalent to disabling VimFx.

Fix #389: Make sure autofocus prevention does not cause infinite loops