[dev] [commits] Horde branch master updated. 33c28c5b3e4f09e6c7d750f276186336e7f3a8e0

Jan Schneider jan at horde.org
Mon Nov 5 22:10:17 UTC 2012


Zitat von Michael M Slusarz <slusarz at horde.org>:

> Quoting Jan Schneider <jan at horde.org>:
>
>> Zitat von Michael M Slusarz <slusarz at horde.org>:
>>
>>> 7e2138a Do basic HTML cleanup for Ingo 3.0.x
>>
>> This should be mentioned in the changelog.
>
> I'm not sure "HTML cleanup" is a very useful changelog message  
> though.  These really were only tweaks to get rid of a bunch of  
> useless parameters, move some basic formatting to CSS, and remove a  
> bunch of unneeded  .  A full HTML rewrite or template change  
> would be necessary for me to consider adding an entry.
>
> e.g. fixing a missing closing tag shouldn't go into the changelog.   
> Yes it "fixes" the HTML, but as long as it didn't break display on a  
> browser it's not something an admin is going to care about.

I didn't had a chance to look at the actual output when I wrote this,  
the commit looked like it was actually changing the layout. Nevermind.
-- 
Jan Schneider
The Horde Project
http://www.horde.org/



More information about the dev mailing list