Tickets :: [#8769] Incorrectly merged files with CSS caching

6.0.0-alpha14

6/30/25

Summary	Incorrectly merged files with CSS caching
Queue	Horde Framework Packages
Queue Version	Git master
Type	Bug
State	Resolved
Priority	1. Low
Owners	jan (at) horde (dot) org
Requester	jan (at) horde (dot) org
Created	12/11/2009 (5680 days ago)
Due
Updated	12/17/2009 (5674 days ago)
Assigned
Resolved	12/17/2009 (5674 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch	No

12/17/2009 10:24:14 AM	Jan Schneider	State ⇒ Resolved

12/17/2009 10:24:08 AM	CVS Commit	Comment #11	Reply to this comment
Changes have been made in Git for this ticket: Reset locale when parsing (~~Bug #8769~~). http://git.horde.org/diff.php/framework/Text_Filter_Csstidy/lib/Horde/Text/Filter/Csstidy/class.csstidy.php?rt=horde-git&r1=a54513db23be4198dd011d1477e8ce144d4d32c2&r2=4c302f54b80f927d66a358f56b78de92ecd8c70c

12/17/2009 10:10:11 AM	Jan Schneider	Version ⇒ Git master Queue ⇒ Horde Framework Packages

12/17/2009 10:09:42 AM	Jan Schneider	Comment #10	Reply to this comment
Gotcha, it's a locale issue, mine is: .tabset li a{color:#366;background-color:#e9e9e9;display:block;border:0;background:url(data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAAkAAACWCAAAAAApyv1aAAAACXBIWXMAAAsTAAALEwEAmpwYAAAAB3RJTUUH1wUfDx8KJabuMwAAARNJREFUOMuNkzFyxDAMAxcazySPSLrk/6+4T9wDrrk+nYUUsi3dCKPEFQcyQRAktcPzBqCd++P7DaH9/vMJWOX5+KBWu5bbV63GUHjH2NUbGIAescERlzNgA6qOSA3cwA4ZGOyGGbm0wJUCgnpUk7HZAOODRVdku/FJsjlY3JjdfiyAeFVaOL/yokqaMPnETMhV4uPKnWv4ytCQsawx+KyVAs/MnU9iWWOVmzpy6tyXa5qV2p76HVR51dFfjs8OSTM2ag7zTXUVOh88vabaO0oODXwOPvfcsCVp5mm+HVNyI/nctYT9S9Na12C5G3GHQo3kM7PPg/p0ofEquruaseizw9b9b9IKFz++ri4+3mrayQb+AiwrnW287gZ8AAAAAElFTkSuQmCC) no-repeat left top;border-bottom:1px solid ~~#999~~;padding:4px ,8em 3px;

12/17/2009 09:25:28 AM	Michael Slusarz	Comment #9	Reply to this comment
The ".tabset li a" padding, e.g. from the silver theme (line 196) is missing in the merged css. Not on my server. Here's the excerpt: .tabset li a{color:#366;background-color:#e9e9e9;display:block;border:0;background:url(data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAAkAAACWCAAAAAApyv1aAAAACXBIWXMAAAsTAAALEwEAmpwYAAAAB3RJTUUH1wUfDx8KJabuMwAAARNJREFUOMuNkzFyxDAMAxcazySPSLrk/6+4T9wDrrk+nYUUsi3dCKPEFQcyQRAktcPzBqCd++P7DaH9/vMJWOX5+KBWu5bbV63GUHjH2NUbGIAescERlzNgA6qOSA3cwA4ZGOyGGbm0wJUCgnpUk7HZAOODRVdku/FJsjlY3JjdfiyAeFVaOL/yokqaMPnETMhV4uPKnWv4ytCQsawx+KyVAs/MnU9iWWOVmzpy6tyXa5qV2p76HVR51dFfjs8OSTM2ag7zTXUVOh88vabaO0oODXwOPvfcsCVp5mm+HVNyI/nctYT9S9Na12C5G3GHQo3kM7PPg/p0ofEquruaseizw9b9b9IKFz++ri4+3mrayQb+AiwrnW287gZ8AAAAAElFTkSuQmCC) no-repeat left top;border-bottom:1px solid ~~#999~~;padding:4px .8em 3px;}

12/16/2009 11:28:24 PM	Jan Schneider	Comment #8	Reply to this comment
The ".tabset li a" padding, e.g. from the silver theme (line 196) is missing in the merged css.

12/16/2009 06:11:37 PM	Jan Schneider	Comment #7	Reply to this comment
I already compared the different CSS rules but didn't find an obvious difference.

12/16/2009 05:30:41 PM	Michael Slusarz	Comment #6	Reply to this comment
Still not seeing this. Tested with bluewhite and silver on FF 3.5 (Ubuntu and Win XP). Here is the tabset CSS as computed by firebug: .tabset ul { list-style-image:none; list-style-position:outside; list-style-type:none; margin-top:4px; padding-left:8px; } .tabset li { float:left; margin:-3px 2px 0 0; white-space:nowrap; }

12/16/2009 10:14:43 AM	Jan Schneider	New Attachment: with_cache.png

12/16/2009 10:14:30 AM	Jan Schneider	Comment #5 New Attachment: without_cache.png	Reply to this comment
I haven't pinned this down yet, but when enabling CSS caching in a static file and using the bluewhite theme, something is wrong with the generated CSS file. The tabs that are using .tabset are smaller than they are supposed to be. I don't see this. This is not even restricted to the bluewhite theme, but seems to happen with tabs in any theme.

12/15/2009 08:48:53 PM	Michael Slusarz	Comment #4	Reply to this comment
JS caching is broken for me too, the sidebar tree isn't rendered. In both cases I use the internal compressor. This is a separate issue that Mike R. had noticed previously. The problem is that the Horde_Tree javascript code contains PHP code. But it doesn't belong in this ticket.

12/15/2009 08:48:11 PM	Michael Slusarz	Comment #3 Assigned to Jan Schneider Taken from Michael Slusarz	Reply to this comment
I haven't pinned this down yet, but when enabling CSS caching in a static file and using the bluewhite theme, something is wrong with the generated CSS file. The tabs that are using .tabset are smaller than they are supposed to be. I don't see this.

12/11/2009 03:53:29 PM	Jan Schneider	Comment #2	Reply to this comment
JS caching is broken for me too, the sidebar tree isn't rendered. In both cases I use the internal compressor.

12/11/2009 03:21:29 PM	Jan Schneider	Comment #1 Patch ⇒ No State ⇒ Assigned Milestone ⇒ Assigned to Michael Slusarz Queue ⇒ Horde Base Summary ⇒ Incorrectly merged files with CSS caching Type ⇒ Bug Priority ⇒ 1. Low	Reply to this comment
I haven't pinned this down yet, but when enabling CSS caching in a static file and using the bluewhite theme, something is wrong with the generated CSS file. The tabs that are using .tabset are smaller than they are supposed to be.