Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jetpack Submenu Icons Don't Open New Window #32601

Open
sdixon194 opened this issue Aug 21, 2023 · 9 comments
Open

Jetpack Submenu Icons Don't Open New Window #32601

sdixon194 opened this issue Aug 21, 2023 · 9 comments
Labels
Customer Report Issues or PRs that were reported via Happiness. Previously known as "Happiness Request". [Feature] Backup & Scan [Feature] Scan [Feature] Subscriptions All subscription-related things such as paid and unpaid, user management, and newsletter settings. [Focus] Accessibility Improving usability for all users (a11y) Groundskeeping Worked on by Dotcom Groundskeeping [Plugin] Jetpack Issues about the Jetpack plugin. https://wordpress.org/plugins/jetpack/ [Pri] Normal [Status] Stale Triaged [Type] Bug When a feature is broken and / or not performing as intended

Comments

@sdixon194
Copy link
Contributor

Impacted plugin

Jetpack

What

When looking at the Jetpack Submenu items that indicate a new window will be opened, they don't actually open a new window.

How

Surfaced during internal testing: p8oabR-1jo-p2#comment-7542

Some submenu items like Subscribers and Scan don't open the window in a new tab. I believe we're using this indicator because the link goes outside of the WP Site (to cloud.jetpack.com in the case of Scan), but it opens in the same time.

Screenshot 2023-08-21 at 10 56 30 AM

@sdixon194 sdixon194 added the [Type] Enhancement Changes to an existing feature — removing, adding, or changing parts of it label Aug 21, 2023
@github-actions github-actions bot added the [Plugin] Jetpack Issues about the Jetpack plugin. https://wordpress.org/plugins/jetpack/ label Aug 21, 2023
@jeherve jeherve added [Feature] Subscriptions All subscription-related things such as paid and unpaid, user management, and newsletter settings. [Feature] Backup & Scan [Feature] Scan labels Aug 28, 2023
@danjjohnson
Copy link

This seems related enough to add here, but let me know if a new issue is preferred.

Same thing happens if you go to My Jetpack, and click on Manage:

image

The transfer ownership link has a 'new window' icon, but it opens in the same window.

image

Ref: p1698052981757229-slack-CDLH4C1UZ

@jeherve jeherve added [Type] Bug When a feature is broken and / or not performing as intended [Pri] Normal [Focus] Accessibility Improving usability for all users (a11y) and removed [Type] Enhancement Changes to an existing feature — removing, adding, or changing parts of it labels Nov 6, 2023
@keoshi
Copy link
Contributor

keoshi commented Jan 4, 2024

Related: https://github.com/Automattic/jetpack-scan-team/issues/792

Also on P2: p1HpG7-qtT-p2

Copy link
Contributor

github-actions bot commented Jan 4, 2024

Support References

This comment is automatically generated. Please do not edit it.

  • p8oabR-1jo-p2#comment-7542
  • pekYwv-4Jd-p2#comment-4304

@github-actions github-actions bot added the Customer Report Issues or PRs that were reported via Happiness. Previously known as "Happiness Request". label Jan 4, 2024
Copy link
Contributor

This issue has been marked as stale. This happened because:

  • It has been inactive for the past 6 months.
  • It hasn’t been labeled `[Pri] BLOCKER`, `[Pri] High`, `[Type] Feature Request`, `[Type] Enhancement`, `[Type] Janitorial`, `Good For Community`, `[Type] Good First Bug`, etc.

No further action is needed. But it's worth checking if this ticket has clear reproduction steps and it is still reproducible. Feel free to close this issue if you think it's not valid anymore — if you do, please add a brief explanation.

@davemart-in
Copy link
Contributor

If a link points to a site within our trusted network (Calypso, wp-admin, Jetpack, A4A) we should not be showing these icons.

@davemart-in davemart-in added the Groundskeeping Worked on by Dotcom Groundskeeping label Oct 18, 2024
@keoshi
Copy link
Contributor

keoshi commented Oct 20, 2024

@davemart-in Would you mind sharing the reasoning behind that opinion?

@davemart-in
Copy link
Contributor

We did a full audit of external links and came up with these rules as a result of our observations:

  • The external link icon signifies a link will open a new tab. external link
  • We should consider all links within our system (Calypso, wp-admin, Jetpack) internal links.
  • All links outside of our system should open in a new tab. (YouTube, Fiverr, etc.)
  • If linking to a support page, we should open the Help Center; no external link icon is necessary.
  • When in doubt, if Automattic owns the property, open the link in _self; no external link icon is necessary.

Here are a few links with additional context:

  • pe7F0s-23W-p2
  • pekYwv-4Jd-p2

@keoshi
Copy link
Contributor

keoshi commented Oct 21, 2024

Thanks @davemart-in, will add a comment of the latest post.

@keoshi
Copy link
Contributor

keoshi commented Oct 21, 2024

Left a comment here: pekYwv-4Jd-p2#comment-4304

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Customer Report Issues or PRs that were reported via Happiness. Previously known as "Happiness Request". [Feature] Backup & Scan [Feature] Scan [Feature] Subscriptions All subscription-related things such as paid and unpaid, user management, and newsletter settings. [Focus] Accessibility Improving usability for all users (a11y) Groundskeeping Worked on by Dotcom Groundskeeping [Plugin] Jetpack Issues about the Jetpack plugin. https://wordpress.org/plugins/jetpack/ [Pri] Normal [Status] Stale Triaged [Type] Bug When a feature is broken and / or not performing as intended
Projects
Development

No branches or pull requests

6 participants