Shift + Click is not handled

[vict-shevchenko]

Hello!

As I know browsers open links in a new window, when clicking on them with a 'Shift + Click'. But Angular prevents this, and open link in a current tab(window). This can be observed on main angular website.

As I see the issue is in a $locationProvider

if (!html5Mode.rewriteLinks || event.ctrlKey || event.metaKey || event.which == 2) return;

it seems to me that it is missing event.shiftKey

Can you advise, it this a really miss or a default behavior?

Thank You!

[caitp]

it's really difficult, we don't even correctly detect non-primary-clicks all the time.

There is some work being done to do a better job of this (UI Events), but nothing that has really shipped yet.

However, for the shift+click case, we could probably make this work.

[vict-shevchenko]

Thank You!

If my proposition works well, can I make a pull request? May be you can give me a green light if it is ok.

[caitp]

It's going to need an e2e test, which might be a bit difficult to write, but if you can do it before me then i'll certainly review your PR and land it

[caitp]

@vict-shevchenko so are you positive shift-click should open a new window in all supported browsers, regardless of configuration?

[vict-shevchenko]

I don`t think I can do it faster than you, especially e2e test )

i think if we do

if (!html5Mode.rewriteLinks || event.shiftKey || event.ctrlKey || event.metaKey || event.which == 2) return;

this will cause default browser handling of shift + click, what it opening in new window in most browsers.

[caitp]

yes --- the question is really, will this do the right thing for all target browsers. I'm hoping that it does.

It turns out an E2E test isn't really needed, since the browser won't open a new window because of a fake mouse event, so it should be fine to just unit test it