Human readable preset names
stBorchert - October 2, 2009 - 21:27
| Project: | ImageCache |
| Version: | 6.x-2.x-dev |
| Component: | User interface |
| Category: | feature request |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | needs review |
Description
As long as I'm using ImageCache I always wanted to give some presets more descriptive names (namespace is limited and not really good to describe the preset).
Heres a patch that adds the possibility to enter descriptive names for presets. These names are used in listings (e.g. Views or CCK field output selection).
I've added some screenshots to show the new behaviour.
| Attachment | Size |
|---|---|
| imagecache_preset-names.patch | 26.35 KB |
| preset_names-01.png | 37.59 KB |
| preset_names-02.png | 42.43 KB |
| preset_names-03.png | 16.3 KB |
#1
Btw.: committing this patch would require nearly all modules depending on ImageCache to rewrite some parts of their code.
Maybe ist better to not change the name of column "presetname" to "presetnamespace" but name the new column "presetrealname" (or something like this).
That would still give users the possibility to enter descriptive names but doesn't force other modules to change their code.
On the other hand the columns are now (with this patch) named according to their contents.
#2
Other approach, same result (without renaming column "presetname" to "presetnamespace" and forcing other modules to rewrite their code).
#3
One minor comment change.
#4
I like this. It would also reduce the likelihood to rename presets and having to update all CCK field displays and Views, as described in #596620: Renaming ImageCache preset invalidates formatter display of CCK fields. .