Make WordPress Core

Opened 19 years ago

Closed 19 years ago

#2508 closed enhancement (fixed)

Make Comment Purging Warning More Informative

Reported by: schestowitz's profile schestowitz Owned by: robmiller's profile robmiller
Milestone: 2.1 Priority: lowest
Severity: normal Version: 2.0
Component: Administration Keywords: bg|has-patch bg|tested bg|commit
Focuses: Cc:

Description

(nightly 25-02-2006) When viewing comments in 'mass edit' mode, deletion of comments is a common and risky task. However, the prompt that appears when deleting all selected items does not include the number of selected items. I suggest making it more verbose. User confidence and usability could be improved here, in line with other applications, e.g. mail clients.

Attachments (2)

edit-comments.php.diff (1.6 KB) - added by robmiller 19 years ago.
edit-comments.php.2.diff (1.6 KB) - added by robmiller 19 years ago.
Second patch implementing checking for no checkboxes checked (check!)

Download all attachments as: .zip

Change History (12)

#2 @robmiller
19 years ago

Patch working fine on my local install.

There's a minor issue, I guess, with the plural form of "comment" when deleting just one comment, although why a user would check use the mass editing mode to delete just one comment I don't know.

Also, whilst changing this it struck me as a bit strange that the form can be submitted when there are 0 comments checked. Perhaps with this new functionality we can do

if(numchecked == 0) return false; else return confirm("blah blah...");

?

@robmiller
19 years ago

Second patch implementing checking for no checkboxes checked (check!)

#3 @robmiller
19 years ago

Second patch added that alerts the user if there are no checkboxes selected when they try to mass-delete.

#4 @robmiller
19 years ago

  • Milestone set to 2.1

#5 @robmiller
19 years ago

  • Keywords bg|has-patch added; comment moderation removed

#6 @robmiller
19 years ago

  • Owner changed from anonymous to robmiller
  • Status changed from new to assigned

#7 @davidhouse
19 years ago

  • Keywords bg|tested bg|commit added

#8 @schestowitz
19 years ago

Thanks for the patch, Rob.

#9 @robmiller
19 years ago

May it be the first of many, hopefully :)

#10 @ryan
19 years ago

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

(In [3652]) Show number of comments selected for deletion. Props robmiller. fixes #2508

Note: See TracTickets for help on using tickets.