[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