On 07/13/2011 03:42 PM, Mark McLoughlin wrote:
Hey,
Hi,
In the pools pretty list, the link to the default pool details looks
like:
default_pool Pool details>>
Which looks pretty bad
How about we:
1) Rename default_pool to "Default Pool" - that seems like a much
more human friendly name
renaming should work (note that default_pool name is probably used both
in seeds and migrations).
2) Add a helper method to make the link "Default Pool
Details" rather
than "Default Pool Details Details" e.g.
name = pool.name.endswith("Pool") ? pool.name : "#{pool.name}
Pool"
Looks like a hack. Why not just change link text to:
1) "#{pool.name} details" or
2) "Pool #{pool.name} details" - yes, in this case "Pool" is twice,
but
if it's a problem, then better to rename default_pool to "Default"
Any thoughts on that?
Cheers,
Mark.
Jan
_______________________________________________
aeolus-devel mailing list
aeolus-devel(a)lists.fedorahosted.org
https://fedorahosted.org/mailman/listinfo/aeolus-devel