Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

docs(*): don't hide results for middle-clicks #16233

Merged
merged 1 commit into from
Sep 21, 2017

Conversation

mgol
Copy link
Member

@mgol mgol commented Sep 20, 2017

Middle-clicking opens a link in a new tab; it shouldn't close the results list
as the user may want to open more of those links.

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Docs fix.

What is the current behavior? (You can also link to an open issue here)
Middle-clicking on a link in the results list (which opens a link in a new tab) closes the list.

What is the new behavior (if this is a feature change)?
List is kept open in such a case.

Does this PR introduce a breaking change?
No.

Please check if the PR fulfills these requirements

Other information:

@Narretz
Copy link
Contributor

Narretz commented Sep 21, 2017

LGTM

Middle-clicking opens a link in a new tab; it shouldn't close the results list
as the user may want to open more of those links.

Closes angular#16233
@mgol mgol force-pushed the middle-click-no-hide branch from 0d7f74d to da6ae8b Compare September 21, 2017 11:14
mgol added a commit that referenced this pull request Sep 21, 2017
Middle-clicking opens a link in a new tab; it shouldn't close the results list
as the user may want to open more of those links.

Closes #16233
@mgol mgol merged commit da6ae8b into angular:master Sep 21, 2017
@mgol mgol deleted the middle-click-no-hide branch September 21, 2017 12:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants