Mailing List Archive

Re: [8555] The clearboth class was causing rendering issues on the login and
I know I'm a little late to the discussion here but why are you
replacing valid CSS with invalid CSS? There are IE hacks that are valid.

On Apr 6, 2009, at 9:12 PM, brewt@bricolage.cc wrote:

> Revision
> 8555
> Author
> brewt
> Date
> 2009-04-06 18:12:27 -0700 (Mon, 06 Apr 2009)
> Log Message
>
> The clearboth class was causing rendering issues on the login and
> new story/media/etc pages (amongst others) in IE6. This hacky fix
> is to just turn off what clearboth does in IE6 and set zoom:1 (or
> height: 1%) in the places where it does need it. Also changed the
> clear class to remove IE-mac support and use a different CSS filter.
> Modified Paths
>
> bricolage/trunk/comp/media/css/style.css
> Diff
>
> Modified: bricolage/trunk/comp/media/css/style.css (8554 => 8555)
>
> --- bricolage/trunk/comp/media/css/style.css 2009-04-07 01:08:32 UTC
> (rev 8554)
> +++ bricolage/trunk/comp/media/css/style.css 2009-04-07 01:12:27 UTC
> (rev 8555)
> @@ -699,8 +699,10 @@
> #containerprof ul.elements li.element {
> margin: 0 0 10px 0;
> position: relative;
> +/* Clearing for IE6 */
> + _zoom: 1;
> }
> -#containerprof ul.elements li.element h3.name {
> +#containerprof ul.elements li.element h3.name {
> margin: 0; clear: both;
> float: left; width: 75px;
> text-align: right;
> @@ -933,27 +935,13 @@
> .desk_item .head,
> form .row,
> .paginate,
> -.viewmeta { display: inline-block;}
> -/* Hides from IE-mac \*/
> -* html .clearboth,
> -* html .desk-top,
> -* html .desk .item,
> -* html .desk .item .head,
> -* html .desk .item .actions,
> -* html .desk_item .head,
> -* html form .row,
> -* html .paginate,
> -* html .viewmeta {height: 1%;}
> -.clearboth,
> -.desk-top,
> -.desk .item,
> -.desk .item .head,
> -.desk .item .actions,
> -.desk_item .head,
> -form .row,
> -.paginate,
> -.viewmeta {display: block;}
> -/* End hide from IE-mac */
> +.viewmeta {
> + *height: 1%;
> +}
> +/* Turn off clearboth for IE6 */
> +.clearboth {
> + _height: auto;
> +}
>
> /* double list manager. */
> table.dlman select { width: 40em; } /* IE-specific setting */
Re: [8555] The clearboth class was causing rendering issues on the login and [ In reply to ]
Marshall Roch <marshall@mroch.com> wrote:
> I know I'm a little late to the discussion here but why are you
> replacing valid CSS with invalid CSS? There are IE hacks that are valid.

I'm up for using valid CSS hacks if they're simple and clean. I did a quick
search to see if anything new had come up since I last looked, and didn't find
anything. Did you have something in mind?

Adrian