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
comment:2
robmiller
— 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...");
?
comment:3
robmiller
— 7 years ago
Second patch added that alerts the user if there are no checkboxes selected when they try to mass-delete.
comment:6
robmiller
— 7 years ago
- 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.