hints: do not execute `click` in addition to opening hints

When the element which we are about to open has a "href" attribute, we
will execute either `window.open` or set the `window.location.href`
attribute to the target's reference. But as we forgot to call `return`
here, we will also execute the code to open elements without "href"
attribute. In the case of opening a new window, we will now first open
the new window and then also set the current window's URL to the target,
which is obviously wrong.

Fix the issue by putting the excution of `click()` inside an
else-branch.

hints: use `window.location.href` to change current URL

When opening a hint in the current window, we do so by emulating a
`click()` on the element that we want to switch to. This enables us to
open hints when there is no href set, e.g. when the element executes
JavaScript to change the page. Unfortunately, this method of opening
links does not work when JavaScript is disabled in the client.

If the element has a `href` attribute, we can work around this problem
by directly setting `window.location.href` to the element's reference.
Next to being a more obvious method of setting the current URL, this
also fixes hints with JavaScript disabled. Note that we still have to
keep around the old method of using `click()` on the element in case it
has no href attribute.

Cast to OptArgFunction.

Move include of hints.js.h to the right place.

adds insight about shortcut-default #365

adds incsearch to manpage (#365)

Mention runsandbox target in README.

Update change settings in CHANGELOG.

Add settings for spell checking.

Increase default z-index for hint elements.

There are some sites where the hint label where covered by on page
elements using a high z-index. So increase our default too.

fixes serif-font setting name to match manpage

Eliminate dead code from main.c

1. Value stored in 'num' in line 161 was never read
   since it was unconditionally overwritten immediately in 166.

2. Value stored in 'suffix' in line 145 was never read
   since it was overwritten in both branches of 'if(dot_post)'.

3. Value stored in 'req' in line 1157 was never read
   if the branch of the first if statement is taken.
   Storing has been placed int to the second if statement,
   where it is actually used.

Added missed ;x hinting to man page.

Use dashed setting names.

The mixes use of none separated settings names and those containing
multiple dashes looked ugly. So use consequently dashes setting names.

Added current available settings to man page #365.

Added missed x-hint-command setting.

Add space after sed -e option #388.

Remove also whitespace around *%/[.

Close comments in js on end of line.

The multi line comments /* */ must be closed on the same line. Because
the simple script that minifies the scripts a little can only  remove
those on same line closed comments properly.

fixes :eval segfault (closes #397)

js2sh.sh: sed for was GNU-specific.

Mention mailing list also for the contributors.

Moved contribution related stuff from README to CONTRIBUTE.

Added editor to man page #347.

Fixed wrong link identifier.

Fixed wrong client to webpage relation #349.

In case a related webview is created. The relation between webextensions
web page and the client in the ui process where mismatched. In fact the
relation was correct, but when a dbus call was fired from ui to
webextension, the webextension used the last created web page to run
javascript in or to focus input fields.
This patch adds the page_id of the web page to the dbus calls that
target a specific web page. So that th webextension can get the right
page by this id to use to answer the call.
Also the communication from webextension to ui lagged this essential
information. So that a click to a editable filed in a related webview
all related instances into input mode switched.

implements external editor (closes #347)

Removed dead code #393.

Fixed possible null pointer dereference #392.

Added inofficial arch linux pkgbuild.

Updated README.

Also added known packages.

Added hinting to man page.

fixes hintElem opacity for non opaque elements

Forces the hintElem to be fully opaque, even when attached to non opaque
elements (e.g. {opacity:0.5}). #349

Don't give MFLAGS to make explicitly.

The option given to the upper most make call are already given as
MAKEFLAGS on recursive calls to make. So there is no need to give them
explicitly.

hints: open links in async mode

When firing hints, open them (either by simulating a click or through
`window.open()`) in async mode with `window.setTimeout()`. If we don't
do that, the `EvalJS` dbus call will itself timeout and we'll end up
in an inconsistent state, that is, not back to Normal mode.

ref #349

Moved scrolling logic into javascript.

Merge branch 'hints' into webkit2.

Show scroll indicator in status bar #354.

Fix wrong js to scroll to x percent of page.

Use direct click method on element #349.

Previously there was some hacky target="_blank" setting unsetting logic
in the hints click method to force to open a hint into new instance or
the same. But this does not work and the hack was previously removed. So
there is no need to do the click event emulation by ourselves, it's
already available as method on the elements itself.

Clear hints on entering normal mode #349.

Workaround for hanging hints #349.

Set hint css vie web content manager #349.

Add the hint css direct to webkit instead of manipulating the DOM to add
a style node. By the way the default style was removed from config.def.h
into src/scripts directory so that the css is minified during
compilation.

Prefix minified content by file type.

Avoid naming collisions and prefix the minified scripts by JS_ so that
it's obvious what's their content.

Inject global hints script during setting.

In case the user disabled user scripts, also the hints script was
removed from the web content manager. So if the page was reloaded the
hinting did not work. To avoid the disabling of vimb internal used
scripts, we set them always during set up of the settings. So they are
readded in case all scripts are removed from the web content manager.

Run hint script syncron #349.

When we run the hinting asyn over dbus we can't use the return value
of the hinting to check if the action was done or not. So it was not
possible to undo last number filter on pressing backspace.

Fixed missed quotes around filter value #349.

Don't fail on same origin policy violations #349.

Inject hinting script only in top frame #349.

We can not address sub frame scripts at the time and the hinting script
is able to process also frames, when this is allowed according to the
same origin policy. So there is no need to inject the script into
multiple frames.

Allow to quit hint mode via <CTRL-C> #349.

There are some pages where the hinting could not be started and the
input box is filled with ';o'. If the user uses <Esc> this key is also
given to the hinting mode which can't handle this so the user has no way
to get back to normal mode.
Moved the <CTRL-C> case to the beginning of the key handler to make sure
the user can abort the current mode by <CTRL-C>.

Make the ";y" (yanking) hint command work

Make hints wotk in "new window" mode

The old "set target to _blank" trick doesn't work anymore since the
whole "noopener" vulnerability thing
(https://mathiasbynens.github.io/rel-noopener/)

I couldn't find a workaround. We're unfortunately now limited in the
kind of elements we can interact with in "new window" mode.

hints: re-enable hints function return value processing

hints: Re-enable nearly all call_hint_function invocations

Convert old hints.c to the webkit2 era

Lots of it is still disabled, but the basic hint showing happening when
pressing 'f' works.

Add hint display

Pressing 'F' will show hints with the old styling from the master
branch.

Very preliminary work for bringing hints back

Merge branch '375_reject_window_manager_initiated_quit_if_downloads_are_running' of https://github.com/rti/vimb into webkit2

Merge branch 'fix_typos' of https://github.com/rti/vimb into webkit2

Merge branch '383_download_file_name_postfix' of https://github.com/rti/vimb into webkit2

Don't set empty title is none is found.

In case a page without a title or a none html file which can be shown is
opened, webkit informed us about the title change to empty string. But
no window title is no good idea, especial in case vimb runs within
tabbed and there are multiple unnamed tabs.
So we set the webview title only if it's not empty.

Reject window manager quit if downloading

Rejects a window manager initiated quit signal if there are downloads in
progress. This closes #375.

fixes typos

fixes download file name postfix (closes #383)

Changed default-zoom behaviour.

In the previous implementation the default-zoom was only a initial full
content zoom applied to the webview. But in case the user changed the
zooming and reset it back by using 'zz' the webkit zoom level was set to
1.0 (100%) and not to the initial zoom level like on startup of vimb.
This behaviour is strange to under stand and does not fit toe the
setting name 'default-zoom'.
To make the default-zoom to a real default, the zoom is also applied in
case the zoom is reseted by the user via 'zz'.

Fix some typos.

Keep all content before command on completion.

If ':: o' was completed, the white space and ':' before the command
where skipped and the user input was changed into ':open'. Vim keeps the
input also during command completion, so we follow this model and keep
it too.

Allow also completion for whitespace prefixed commands.

We allow to put whitespace or additional ':' before ex commands to avoid
recoding of the commands in the history. But in the completion only the
first ':' was skipped to get the command to apply completion for.
So ': open foo<Tab>' did not start completion, because ' open' is none
known command.
So skip all ':' and whitespace after the first ':' to set the pointer to
the beginning of the command.

Add bookmark completion to manual page #365.

Added section for input mode to man page #365.

Added missed queue file in manual page #365.

Add queue feature to manual page #365.

Moved closed file writing to util #379.

Allow to store multiple closed URLs #379.

Applied changes of 8ae4cadf58e46b3e9951fa89cf084bd2792984c3 in master
branch to new infrastructure.

Fixed wrongs check of g_mkdir_with_parents #381.

Don't run input change on stepping through completion.

fixes incsearch history handling

Refactores parts of command_search() (closes #372)
Fixes search using "/ register (type "/<enter>")

Added missed --config option on spawning new instance.

Show profile name in url bar #343.

Add profile option #343.

Remove FEATURE_HISTORY_INDICATOR.

This does not bring any real benefit to make allow to avoid this but
makes the code hard to maintain.

Update the license year.

Create files needed for full feature support.

Create all those files on startup that are needed to provide all the
documented features.

Don't fail on missing files.

Don't warn about missing scripts file.

It's not required that the user has a scripts file. So don't attempt to
inject it if it is not present.

Don not double inject user script file.

The script file is injected in case 'userscripts' setting is enabled. So
there is no need to do this on creating the webview too.

Removed hsts file because this is not used at the time.

Fixed none printed :open .. on O.

If the input was filled programmatically and the mode where switched the
already filled input was cleaned by the attempt to stop possible search.
So now the input is only cleared if there is really an active search.

Merge branch 'fix_incsearch_paste' of https://github.com/rti/vimb into webkit2.

Clear input after load commited during search.

If a new page is opened during search, the search string was still
present in the input box even if the searching is paused in this case.
So now clear the input box and stop searching in case of a new page
load.

adds TODO for text buffer change warning

adds TODO for deprecated gdkkeysyms

moves incsearch to ex_input_changed (closes #370)

adds call to mode input_changed (closes #369)

Run js by webextension.

Make sure all the scripts we run internally are also evaluated in case
js is disabled.

Run js for scrolling from webextension #367.

Show if directory can't be created.

Added :save command.

Add docu for reading from stdin.

Merge branch '346_html_from_stdin' of https://github.com/rti/vimb into webkit2.