Make WordPress Core

Opened 4 years ago

Closed 4 years ago

#51701 closed defect (bug) (fixed)

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

Reported by: desrosj's profile desrosj Owned by: audrasjb's profile 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 4 years ago.
Capture d’écran 2020-11-03 à 18.43.05.png (214.4 KB) - added by audrasjb 4 years ago.
Before patch
Capture d’écran 2020-11-03 à 18.43.27.png (222.0 KB) - added by audrasjb 4 years ago.
After patch

Download all attachments as: .zip

Change History (5)

@desrosj
4 years ago

#1 @audrasjb
4 years 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
4 years 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.