I have several suggestions for improving (in my eyes) the user interface:
I'd like to have the settings as a tab instead of a submenu (it is better to handle especially with the DHMTL menu module as it by default doesn't display menu items that have sub-items). Also the settings are kind of hidden under a menu entry title 'Revision deletion'. The menu title (better a tab title!) should be renamed to 'Settings'
In the settings there is a string 'Age in days of revisions that should be deleted.' which I would change into 'Age of revisions that should be deleted.' as there are not only option in days but also in weeks.
There is also an untranslatable string 'No Node Revisions to delete at this time. ' on ?q=admin/content/revision_deletion.
Finally there is a string 'Select node types for revision deletion. Multiple select enabled.' which ends with a dot although there is a hardcoded ':' after it.
Thanks for the module anyway!
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | hardcopy.PNG | 17.24 KB | FrankT |
Comments
Comment #1
nancydruPlease check the patch in #547650: Coding standards. I believe it takes care of all your concerns, but still needs to be ported to 6.x.
Comment #2
nancydruComment #3
FrankT commentedThanks so far. After updating to the latest version there is still a menu entry under 'Revisions to Mass Delete' (/admin/content/revision_deletion), named 'Revision Deletion' which points to the settings tab (/admin/content/revision_deletion/settings).
The menu entry 'Revision Deletion can be removed, or is it a bug in my installation?
I also suggest to replace the setting 'Show conditional labels' by 'Show additional labels', so far I do not understand why the labels should be conditional.
Comment #4
nancydruThis is what you should see
I assume you did run update.php, which is necessary with the menu changes.
I called them "conditional" because they depend on the other settings as to whether or not they are produced. But I went ahead and changed them. BTW, have you looked at translation and whether there are any fixes needed for that?
Comment #5
FrankT commentedThanks, I always run update.php after any updates, but the menu system sometimes behaves strange sometimes. I think I will uninstall the module once and reinstall it again.
I will have a look at the changes this weekend (they are not yet in the dev version of Aug 22!?), I will update my translation and post it shortly then.
Comment #6
FrankT commentedThere is just another thing: there are different date/time formats in the revisions listing, see attached hardcopy.
Comment #7
FrankT commentedI reinstalled the module, after that there is no more menu entry under 'Revisions to mass delete', so that's solved.
Comment #8
nancydruChanging the title to reflect the new point. I figured you had run update, but just wanted a little reminder because even I forget once in a while.
Comment #9
nancydruOkay, changed the format.
Comment #10
nancydruComment #11
nancydru