aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authormrmr19932017-11-11 16:17:03 +0000
committermrmr19932017-11-20 19:42:23 +0000
commit291f52bc319e6790d8fbb1e0384c8c5d023e61c7 (patch)
tree151356464f0d95922fb006841b36ad42069c77d0
parent3f4c9638ac50f31de309b25be49cce4f06b65dc6 (diff)
downloadvimium-291f52bc319e6790d8fbb1e0384c8c5d023e61c7.tar.bz2
Tidy up multiple/redundant checks in LinkHints.activateLink
-rw-r--r--content_scripts/link_hints.coffee13
1 files changed, 7 insertions, 6 deletions
diff --git a/content_scripts/link_hints.coffee b/content_scripts/link_hints.coffee
index 07cec5f4..0b44ea45 100644
--- a/content_scripts/link_hints.coffee
+++ b/content_scripts/link_hints.coffee
@@ -305,7 +305,7 @@ class LinkHintsMode
if linksMatched.length == 0
@deactivateMode()
else if linksMatched.length == 1
- @activateLink linksMatched[0], userMightOverType ? false
+ @activateLink linksMatched[0], userMightOverType
else
@hideMarker marker for marker in @hintMarkers
@showMarker matched, @markerMatcher.hintKeystrokeQueue.length for matched in linksMatched
@@ -359,7 +359,7 @@ class LinkHintsMode
# When only one hint remains, activate it in the appropriate way. The current frame may or may not contain
# the matched link, and may or may not have the focus. The resulting four cases are accounted for here by
# selectively pushing the appropriate HintCoordinator.onExit handlers.
- activateLink: (linkMatched, userMightOverType=false) ->
+ activateLink: (linkMatched, userMightOverType) ->
@removeHintMarkers()
if linkMatched.isLocalMarker
@@ -397,10 +397,11 @@ class LinkHintsMode
# If we're using a keyboard blocker, then the frame with the focus sends the "exit" message, otherwise the
# frame containing the matched link does.
- if userMightOverType and Settings.get "waitForEnterForFilteredHints"
- installKeyboardBlocker (callback) -> new WaitForEnter callback
- else if userMightOverType
- installKeyboardBlocker (callback) -> new TypingProtector 200, callback
+ if userMightOverType
+ if Settings.get "waitForEnterForFilteredHints"
+ installKeyboardBlocker (callback) -> new WaitForEnter callback
+ else
+ installKeyboardBlocker (callback) -> new TypingProtector 200, callback
else if linkMatched.isLocalMarker
DomUtils.flashRect linkMatched.rect
HintCoordinator.sendMessage "exit", isSuccess: true