[wp-trac] [WordPress Trac] #43481: Add tabs and placeholders to privacy tools page in wp-admin
WordPress Trac
noreply at wordpress.org
Thu Mar 15 00:24:54 UTC 2018
#43481: Add tabs and placeholders to privacy tools page in wp-admin
------------------------------+------------------------------
Reporter: xkon | Owner:
Type: enhancement | Status: new
Priority: normal | Milestone: Awaiting Review
Component: General | Version:
Severity: normal | Resolution:
Keywords: gdpr 2nd-opinion | Focuses:
------------------------------+------------------------------
Comment (by allendav):
A few more things from a review of the code:
- there are inconsistent indents in src/wp-admin/css/forms.css for the
body.privacy-php .privacy-information .tab-content rules
- there is inconsistent whitespace (e.g. around this) in src/wp-
admin/js/xfn.js - codesniffer/wp standards could help
- in src/wp-admin/privacy.php "front-static-pages" is a boo boo from my
old patch - lets use "privacy-policy-page" or some such instead
- also, in privacy.php, I think instead of _e and esc_attr_e in the
requests headings, table cells and buttons you want esc_html_e, no?
- and, still in privacy.php, same thing (esc_html_e instead of _e) for the
privacy policy information tabs and headings
--
Ticket URL: <https://core.trac.wordpress.org/ticket/43481#comment:12>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list