Opened 9 years ago
Closed 9 years ago
#32625 closed defect (bug) (fixed)
About: all admin notices should be hidden
Reported by: | jadpm | Owned by: | obenland |
---|---|---|---|
Milestone: | 4.3 | Priority: | normal |
Severity: | minor | Version: | 4.1 |
Component: | Help/About | Keywords: | has-patch reporter-feedback |
Focuses: | ui | Cc: |
Description
Related to #26519, some admin notices are not being hidden on the About page. That ticket ensured that div.updated, div.error admin notices are hidden, but we also have other classnames like notice-warning and notice-info which were not included.
Also, we should think on normalizing the classnames for admin notices:
- div.update equals .notice-success
- div.error equals .notice-error
- .notice-warning and .notice-info do not have a matching pair.
Attachments (2)
Change History (8)
#2
@
9 years ago
- Milestone changed from Awaiting Review to 4.3
- Severity changed from normal to minor
- Version changed from trunk to 4.1
Culprit: #27418
#3
@
9 years ago
How can I reproduce the notices showing?
Shouldn't it be enough to add .about-wrap .notice
to cover all new cases?
#5
@
9 years ago
Yes, .about-wrap .notice
should produce the same effect and it would also be much more similar to the styling we already have in common.css where we define styling for .wrap .notice, .wrap div.updated, .wrap div.error
on the same place. I'm updating the patch to apply this solution better.
It is easy to get an admin notice on the About page:
add_action( 'admin_notices', 'ek_display_admin_notices' ); function ek_display_admin_notices() { echo '<div id="ek-admin-notice" class="notice notice-warning">'; echo 'Whatever'; echo "</div>"; }
about.css diff