[wp-trac] [WordPress Trac] #46098: Help notice displayed on Privacy Policy page missing in block editor
WordPress Trac
noreply at wordpress.org
Fri Apr 12 20:29:03 UTC 2019
#46098: Help notice displayed on Privacy Policy page missing in block editor
----------------------------------------------+----------------------------
Reporter: aduth | Owner: garrett-
| eclipse
Type: defect (bug) | Status: reopened
Priority: normal | Milestone: 5.2
Component: Privacy | Version: 5.0
Severity: normal | Resolution:
Keywords: has-patch has-screenshots commit | Focuses:
| administration
----------------------------------------------+----------------------------
Comment (by aduth):
Ah, right you are. I'd missed it in my review. The updated patch looks
good. I note the small difference of the space occurring outside the
screen reader `span` where before it'd been contained within. In the
browser, it appears to make no difference and the trailing whitespace is
collapsed. As proposed appears to be the more common convention elsewhere
in the codebase as well.
It occurs to me that the `target` attribute shouldn't need to be inserted
using placeholder substitution. I suppose it was arguably more necessary
when the link was translateable. Do you think it ought to be removed now?
A similar point could be made about the screen reader text element, though
there may be an argument in favor of general code legibility to keeping
the substitution.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/46098#comment:22>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list