Skip to content

Commit

Permalink
Merge pull request #2877 from smblott-github/fix-ff-opening-multiple-…
Browse files Browse the repository at this point in the history
…tabs

Do not open tab for target "_blank".
  • Loading branch information
smblott-github authored Dec 19, 2017
2 parents 44e65ef + 1b23f70 commit 7b83828
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion lib/dom_utils.coffee
Original file line number Diff line number Diff line change
Expand Up @@ -250,7 +250,7 @@ DomUtils =
eventSequence = ["mouseover", "mousedown", "mouseup", "click"]
for event in eventSequence
defaultActionShouldTrigger = @simulateMouseEvent event, element, modifiers
if event == "click" and defaultActionShouldTrigger and Utils.isFirefox()
if event == "click" and defaultActionShouldTrigger and Utils.isFirefox() and element.target != "_blank"
# Firefox doesn't (currently) trigger the default action for modified keys.
DomUtils.simulateClickDefaultAction element, modifiers
defaultActionShouldTrigger # return the values returned by each @simulateMouseEvent call.
Expand Down

0 comments on commit 7b83828

Please sign in to comment.