Opened 15 years ago
Closed 13 years ago
#15223 closed Bug (wontfix)
The comments app has 2 similar menu items in action
| Reported by: | Owned by: | Ashutosh Dwivedi | |
|---|---|---|---|
| Component: | contrib.comments | Version: | 1.3-beta |
| Severity: | Normal | Keywords: | actions |
| Cc: | Triage Stage: | Accepted | |
| Has patch: | no | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
There are two menu items:
- Delete selected items
- Remove selected items
I guess one from admin app and one from comments app.
Change History (13)
comment:1 by , 15 years ago
| Triage Stage: | Unreviewed → Accepted |
|---|
comment:4 by , 15 years ago
No doubt originally named after the is_removed field on the Comment model, which seems perfectly reasonable until you see it beside the site-wide delete action. Should be a one-line patch. I think "Unpublish selected" sounds appropriate? (describes the effect better than "deactivate" or "hide").
comment:5 by , 15 years ago
| milestone: | 1.4 → 1.3 |
|---|
Julien, I disagree, this is a bug, even more a one-line translatable literal change. Can be worked on before 1.3 RC.
comment:6 by , 15 years ago
@Ramiro Functionally things are consistent but I agree that this can be seen as a usability bug, and I'm absolutely in favour of fixing this ASAP. The patch may not be as trivial as modifying one string though. The term "removed" is also used in the comment's detail page and in the changelist's table and filters. If we change the term only in some places then things may potentially get even more confusing for the user.
comment:7 by , 15 years ago
| Severity: | → Normal |
|---|---|
| Type: | → Bug |
comment:13 by , 13 years ago
| Owner: | changed from to |
|---|---|
| Status: | new → assigned |
I've made a pull request for the patch here https://github.com/aashudwivedi/django/tree/ticket_15223
comment:14 by , 13 years ago
This is perhaps bikeshedding, but "unpublish/is unpublished" seems pretty clumsy language to me. "hide/is hidden" would be a lot better IMO.
comment:15 by , 13 years ago
| Resolution: | → wontfix |
|---|---|
| Status: | assigned → closed |
django.contrib.comments has been deprecated and is no longer supported, so I'm closing this ticket. We're encouraging users to transition to a custom solution, or to a hosted solution like Disqus.
The code itself has moved to https://github.com/django/django-contrib-comments; if you want to keep using it, you could move this bug over there.
I agree that this is confusing (I had to explain the difference between "Delete" and "Remove" to several clients before). "Remove" should perhaps be changed to "De-activate", or "Un-publish" or something along those lines.