WordPress.org

Make WordPress Core

Opened 3 weeks ago

Closed 3 weeks ago

#51701 closed defect (bug) (fixed)

Auto-update setting notice should be `.notice-success`

Reported by: desrosj Owned by: audrasjb
Milestone: 5.6 Priority: normal
Severity: normal Version: 5.6
Component: Upgrade/Install Keywords: has-patch has-screenshots commit
Focuses: administration Cc:

Description

When saving the new auto-update setting, an info notice (blue) is displayed. This should be a success notice (green), as it indicates a successful update.

Attachments (3)

51701.diff (499 bytes) - added by desrosj 3 weeks ago.
Capture d’écran 2020-11-03 à 18.43.05.png (214.4 KB) - added by audrasjb 3 weeks ago.
Before patch
Capture d’écran 2020-11-03 à 18.43.27.png (222.0 KB) - added by audrasjb 3 weeks ago.
After patch

Download all attachments as: .zip

Change History (5)

@desrosj
3 weeks ago

#1 @audrasjb
3 weeks ago

  • Keywords has-screenshots commit added
  • Owner set to audrasjb
  • Status changed from new to accepted

Nice catch @desrosj. Tested well on my side (see above screenshots).
Marking for commit.

#2 @desrosj
3 weeks ago

  • Resolution set to fixed
  • Status changed from accepted to closed

In 49490:

Upgrade/Install: Change the notice displayed after saving auto-update settings to .notice-success.

This better conveys the message of the notice and brings consistency with other success notices throughout Core.

Props audrasjb.
Fixes #51701.

Note: See TracTickets for help on using tickets.