Opened 7 years ago
Closed 7 years ago
#2508 closed enhancement (fixed)
Make Comment Purging Warning More Informative
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Priority: | lowest | Milestone: | 2.1 |
| Component: | Administration | Version: | 2.0 |
| Severity: | normal | Keywords: | bg|has-patch bg|tested bg|commit |
| 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)
Change History (12)
comment:1
davidhouse — 7 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...");
?
Second patch added that alerts the user if there are no checkboxes selected when they try to mass-delete.
- Owner changed from anonymous to robmiller
- Status changed from new to assigned
comment:7
davidhouse — 7 years ago
- Keywords bg|tested bg|commit added
comment:8
schestowitz — 7 years ago
Thanks for the patch, Rob.
comment:10
ryan — 7 years ago
- Resolution set to fixed
- Status changed from assigned to closed

+1.