Human readable preset names [#594592]

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.

Comment	File	Size	Author
#3	imagecache_preset-names-4.patch	7.31 KB	stborchert
#2	imagecache_preset-names-3.patch	7.36 KB	stborchert
	preset_names-03.png	16.3 KB	stborchert
	preset_names-02.png	42.43 KB	stborchert
	preset_names-01.png	37.59 KB	stborchert
	imagecache_preset-names.patch	26.35 KB	stborchert

Comments

Comment #1

stborchert

he/him

German

commented 2 October 2009 at 21:39

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.

Comment #2

stborchert

he/him

German

commented 3 October 2009 at 09:24

Status	File	Size
new	imagecache_preset-names-3.patch	7.36 KB

Other approach, same result (without renaming column "presetname" to "presetnamespace" and forcing other modules to rewrite their code).

Comment #3

stborchert

he/him

German

commented 3 October 2009 at 09:26

Status	File	Size
new	imagecache_preset-names-4.patch	7.31 KB

One minor comment change.

Comment #4

smk-ka commented 15 October 2009 at 20:43

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. .

Comment #5

sandrewj commented 14 October 2011 at 07:32

Status:

Needs review

» Closed (duplicate)

#737644: Administrative description of a preset

Comment #6

stborchert

he/him

German

commented 14 October 2011 at 07:38

Status:

Closed (duplicate)

» Needs review

@sandrewj: sorry for re-opening, but this patch has been much older and is on a way to make it in core, too (see #606598: Human readable image-style names).
If you may like to join forces, please update this one here and do not open (or feed) another issue.

Comment #7

sandrewj commented 14 October 2011 at 08:12

I don't mind leaving this one open. The patch I submitted on the other issue is pretty similar and solves the same issues.

Does this issue need to stay open because it is referenced in the D8 core issue?
Would the hope be for the patch to be accepted into the D8 core and then backported to D7 core and to this D6 module or can development continue on this issue independently?

Comment #8

sandrewj commented 19 October 2011 at 15:37

Status:

Needs review

» Needs work

For now I am marking this as needs work. When the #606598: Human readable image-style names gets finalized/committed for D8 core it can be ported to this module.