Make WordPress Core

Opened 11 years ago

Closed 9 years ago

#26494 closed defect (bug) (fixed)

Plugins page errors should use 'error' class rather than 'updated'

Reported by: dd32's profile dd32 Owned by: wonderboymusic's profile wonderboymusic
Milestone: 4.4 Priority: normal
Severity: normal Version: 3.8
Component: Plugins Keywords: has-patch commit
Focuses: administration Cc:

Description

Just noticed that some errors on the plugins page are using the Informational notification colours instead of an error colour.

It makes sense that we should update these.

Attachments (3)

26494.diff (815 bytes) - added by dd32 11 years ago.
26494.2.diff (1.9 KB) - added by dd32 11 years ago.
spotted another two cases
26494.3.diff (721 bytes) - added by swissspidy 9 years ago.

Download all attachments as: .zip

Change History (9)

@dd32
11 years ago

#1 @dd32
11 years ago

  • Keywords has-patch commit added

@dd32
11 years ago

spotted another two cases

#2 @nacin
11 years ago

"No out of date plugins were selected." might as well say "All selected plugins are up to date." As in, you wanted to update plugins X, Y, and Z, and they are indeed updated. Arguably not a failure condition; it can go either way.

#3 @dd32
11 years ago

I agree, that particular message is worded as an error at present, but could be re-worded to be more friendly "yay, they're all up to date already" instead.

#4 @nacin
11 years ago

  • Component changed from Administration to Plugins
  • Focuses administration added

@swissspidy
9 years ago

#5 @swissspidy
9 years ago

  • Milestone changed from Awaiting Review to 4.4

The first two cases have already been fixed in #28260.

26494.3.diff changes the wording of No out of date plugins were selected. to All selected plugins are up to date., keeping the notice class as it isn't really an error.

#6 @wonderboymusic
9 years ago

  • Owner set to wonderboymusic
  • Resolution set to fixed
  • Status changed from new to closed

In 33983:

Plugins page: re-word "No out of date plugins were selected." to "All selected plugins are up to date." since it's not really an error condition.

Props swissspidy.
Fixes #26494.

Note: See TracTickets for help on using tickets.