[dev] [commits] Horde branch master updated. d21fdd520f93a0b63ff9a6579f67726cebf1e0de

Michael M Slusarz slusarz at horde.org
Thu May 16 20:13:27 UTC 2013


Quoting Jan Schneider <jan at horde.org>:

> Zitat von Michael M Slusarz <slusarz at horde.org>:
>
>> Quoting Jan Schneider <jan at horde.org>:
>>
>>> commit 232f65ed0160c52e8e90d8173851b5e463691435
>>> Author: Jan Schneider <jan at horde.org>
>>> Date:   Wed Apr 24 16:07:44 2013 +0200
>>>
>>>   Fix link colors and remove address group background.
>>>
>>> imp/themes/default/dynamic/screen.css |    6 +-----
>>> imp/themes/default/screen.css         |    3 +--
>>> 2 files changed, 2 insertions(+), 7 deletions(-)
>>>
>>> http://git.horde.org/horde-git/-/commit/232f65ed0160c52e8e90d8173851b5e463691435
>>
>> Why?  This is necessary to distinguish these group addresses.  I'd  
>> agree that the formatting can be improved, but it absolutely  
>> shouldn't be eliminated.
>
> Why is it necessary to distinguish those? I actually removed it to  
> resemble the look of a regular address list. Is this not wanted for  
> some reason?

Yes.  Take this example, as displayed in dynamic view:

Group 1: Foo Foo2

Each of these three elements is clickable.  But is Foo2 part of Group  
1?  There needs to be some sort of indication which addresses belong  
in the group.

michael

___________________________________
Michael Slusarz [slusarz at horde.org]



More information about the dev mailing list