WordPress.org

Make WordPress Core

Opened 7 months ago

Last modified 4 weeks ago

#39105 new enhancement

Add target="_blank" to author website output in comment moderation

Reported by: joshua.strebel Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version:
Component: Comments Keywords: reporter-feedback target-blank
Focuses: ui, accessibility Cc:

Description

On /wp-admin/edit-comments.php add target="_blank" on the <a href id="author_comment_url_###$" > link. The preview screenshot is not alway sufficient and a click refreshes the current browser page. Would a new window be cleaner?

Change History (7)

#1 in reply to: ↑ description @joshua.strebel
7 months ago

Replying to joshua.strebel:

On /wp-admin/edit-comments.php add target="_blank" on the <a href id="author_comment_url_###$" > link. The preview screenshot is not alway sufficient and a click refreshes the current browser page. Would a new window be cleaner?

Accessibility and other considerations may need to be investigated.

#2 @rianrietveld
3 months ago

  • Focuses accessibility added

This ticket was mentioned in Slack in #accessibility by afercia. View the logs.


8 weeks ago

#4 @afercia
8 weeks ago

  • Keywords reporter-feedback added
  • Version 4.6.1 deleted

@joshua.strebel we've discussed this in today's accessibility meeting, struggling a bit to find this thing :) We suspect it's an Akismet feature, can you confirm please? If so, not related to WordPress core and should be reported to the plugin authors.

Last edited 8 weeks ago by afercia (previous) (diff)

#5 follow-up: @joshua.strebel
8 weeks ago

Checked with and without akismet enabled.

Action with Akisment:
Hover over link, shows a screenshot
Click on link, opens in same tab as wp-admin (replaces)

Action w/out akisment
Hover over link, nothing.
Click on link, opens in same tab as wp-admin (replaces)

See screenshot.
https://www.dropbox.com/s/olc8afyp60kre6w/screenshot407.jpg?dl=0

#6 in reply to: ↑ 5 @joshua.strebel
8 weeks ago

It's Line 688 in wp-admin/includes/class-wp-comments-list-table.php

<?php
//687 if ( ! empty( $author_url_display ) ) {
//688  printf( '<a href="%s">%s</a><br />', esc_url( $author_url ), esc_html( $author_url_display ) );
//689 }

Replying to joshua.strebel:

Checked with and without akismet enabled.

Action with Akisment:
Hover over link, shows a screenshot
Click on link, opens in same tab as wp-admin (replaces)

Action w/out akisment
Hover over link, nothing.
Click on link, opens in same tab as wp-admin (replaces)

See screenshot.
https://www.dropbox.com/s/olc8afyp60kre6w/screenshot407.jpg?dl=0

#7 @afercia
4 weeks ago

  • Keywords target-blank added
Note: See TracTickets for help on using tickets.