Don't prevent opening themes in new tabs

[jancbeck]

The single theme overlay screen looks gorgeous, however, at times I like to inspect a couple of themes within multiple browser tabs to better compare them side-by-side. What I do is I CMD + Click (Mac) on each of the themes in the result list and I expect each of these to open in a new browser tab just like with any other hyperlink. What happens instead is that the single theme overlay opens in the same tab I'm currently in.

In theme.js around line 61 there should be a check if the user has currently pressed the CMD or CTRL key and if that's the case

event.preventDefault();

should not run. Ideally it should also check for a mousedown event of the middle mouse button since that triggers a new tab in most browsers I'm aware of.

[jancbeck]

Related to that, right-clicking a theme item and choosing "open in new window" does not work because the links point to ​https://wordpress.org/{$themename}/. This could be because of the "preview" state of the new theme directory so I did not include it into the initial ticket. However it would be great to see this working in the final state.

[obenland]

In 1265:

WP.org Themes: Don't prevent opening themes in new tabs.

Wrapps the anchor around all child elements of .theme to allow users to
either right-click and choose to open the link in a new tab, or click the link
with a meta modifier key pressed (command/Ctrl).

Fixes #844.

[obenland]

Replying to jancbeck:

Related to that, right-clicking a theme item and choosing "open in new window" does not work because the links point to ​https://wordpress.org/{$themename}/. This could be because of the "preview" state of the new theme directory so I did not include it into the initial ticket. However it would be great to see this working in the final state.

Faulty URLs were fixed in r1256.
Thanks for the report, @jancbeck!

[mrwweb]

It appears that this feature has regressed and new tabs are again impossible to create. Testing on Windows 10 in both Edge and Chrome, CTRL + click does not work and themes open in the current tab. The right-click context menu option still works.

[jancbeck]

Replying to mrwweb:

It appears that this feature has regressed and new tabs are again impossible to create. Testing on Windows 10 in both Edge and Chrome, CTRL + click does not work and themes open in the current tab. The right-click context menu option still works.

It is working for me on Mac+Chrome. Are we talking about browsing ​https://wordpress.org/themes/browse/popular/ ?

[mrwweb]

Yes. Also ​https://wordpress.org/themes/. Just tested in Firefox and CTRL + Click doesn't work there either.

[Otto42]

Confirmed that it does not work on Chrome + Windows 10.

I think this is because event.metaKey is unreliable, according to most sources I can find with a quick search.

Edit: Checking for event.ctrlKey as well should work, I think.

[Otto42]

In 2450:

Add checking for event.ctrlKey as well, for Windows browsers that don't recognize ctrl as a metaKey. Fixes #844