[wp-trac] [WordPress Trac] #23432: Review usage of target="_blank" in the admin
WordPress Trac
noreply at wordpress.org
Tue Sep 13 11:36:05 UTC 2016
#23432: Review usage of target="_blank" in the admin
----------------------------+--------------------------------
Reporter: theadityajain | Owner: rianrietveld
Type: enhancement | Status: assigned
Priority: normal | Milestone: Future Release
Component: Administration | Version: 3.2
Severity: normal | Resolution:
Keywords: has-patch | Focuses: ui, accessibility
----------------------------+--------------------------------
Comment (by rianrietveld):
Investigated the use of target="_blank" in the admin today.
Almost all target="_blank" links are in the help-wrap at the top of almost
each admin-view.
And if no permalink structure is set, a button Change permalink is shown
with a target=_blank link to the set permalink page.
I left out the media and press-this for this review.
Most links can be changed easily by just removing the target.
On other pages an alert needs to be set for possible unsaved data when a
user leaves, as done in the edit.php, but that can be done later.
The (old) link functionality is still in core, you can activate it with
the the plugin Link Manager, which has 100,000+ active installs. This also
has "_blank in the help-wrap.
The plan is to open new tickets per functionality (like
users/comments/network) and make patches for those. We can discuss per
ticket what needs to go and what can stay. But for the links in help: I
think all blank can go.
A few questions:
- Then there is the preview button. Should this open in a new window or in
the same? I prefer stay with a visual warning.
- Some functionality, like the menus and the old link functionality, gives
the user the option to add a link target="_blank". Maybe we should
reconsider this? Related to this is #36809
--
Ticket URL: <https://core.trac.wordpress.org/ticket/23432#comment:33>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list