You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ofbiz.apache.org by Bilgin Ibryam <bi...@iguanait.com> on 2008/11/19 11:32:31 UTC
Re: svn commit: r718299 - /ofbiz/trunk/framework/images/webapp/images/maincss.css
Adrian,
I wasn´t aware of gray box while doing this change.
The reason for my change was that virtual products which require
amount for some variants use the hidden style, and then use
changeObjectVisibility function to become visible. And this function
only change visibility but not the height, so the amount fields
remains with 0 height and not visualized.
Is it ok to add a new class with height:0 and visibility:hidden to use
in header.ftl (gray bar) ?
Bilgin
On Nov 17, 2008, at 7:56 PM, adrianc@apache.org wrote:
> Author: adrianc
> Date: Mon Nov 17 09:56:44 2008
> New Revision: 718299
>
> URL: http://svn.apache.org/viewvc?rev=718299&view=rev
> Log:
> Reverted style change made in rev 712478. It caused a gray bar to
> appear at the top of the screen.
>
> Modified:
> ofbiz/trunk/framework/images/webapp/images/maincss.css
>
> Modified: ofbiz/trunk/framework/images/webapp/images/maincss.css
> URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/images/webapp/images/maincss.css?rev=718299&r1=718298&r2=718299&view=diff
> =
> =
> =
> =
> =
> =
> =
> =
> ======================================================================
> --- ofbiz/trunk/framework/images/webapp/images/maincss.css (original)
> +++ ofbiz/trunk/framework/images/webapp/images/maincss.css Mon Nov
> 17 09:56:44 2008
> @@ -813,6 +813,7 @@
> }
>
> .hidden {
> +height: 0;
> visibility: hidden;
> }
>
>
>
Re: svn commit: r718299 - /ofbiz/trunk/framework/images/webapp/images/maincss.css
Posted by Adrian Crum <ad...@yahoo.com>.
It would be better to use a CSS descendant selector to reset the height. Changing the existing styles almost always breaks layout. So, do something like:
.variant-amount .hidden {
height: auto;
}
-Adrian
--- On Wed, 11/19/08, Bilgin Ibryam <bi...@iguanait.com> wrote:
> From: Bilgin Ibryam <bi...@iguanait.com>
> Subject: Re: svn commit: r718299 - /ofbiz/trunk/framework/images/webapp/images/maincss.css
> To: dev@ofbiz.apache.org
> Date: Wednesday, November 19, 2008, 2:32 AM
> Adrian,
>
> I wasn´t aware of gray box while doing this change.
> The reason for my change was that virtual products which
> require amount for some variants use the hidden style, and
> then use changeObjectVisibility function to become visible.
> And this function only change visibility but not the height,
> so the amount fields remains with 0 height and not
> visualized.
>
>
> Is it ok to add a new class with height:0 and
> visibility:hidden to use in header.ftl (gray bar) ?
>
> Bilgin
>
> On Nov 17, 2008, at 7:56 PM, adrianc@apache.org wrote:
>
> > Author: adrianc
> > Date: Mon Nov 17 09:56:44 2008
> > New Revision: 718299
> >
> > URL:
> http://svn.apache.org/viewvc?rev=718299&view=rev
> > Log:
> > Reverted style change made in rev 712478. It caused a
> gray bar to appear at the top of the screen.
> >
> > Modified:
> >
> ofbiz/trunk/framework/images/webapp/images/maincss.css
> >
> > Modified:
> ofbiz/trunk/framework/images/webapp/images/maincss.css
> > URL:
> http://svn.apache.org/viewvc/ofbiz/trunk/framework/images/webapp/images/maincss.css?rev=718299&r1=718298&r2=718299&view=diff
> >
> ==============================================================================
> > ---
> ofbiz/trunk/framework/images/webapp/images/maincss.css
> (original)
> > +++
> ofbiz/trunk/framework/images/webapp/images/maincss.css Mon
> Nov 17 09:56:44 2008
> > @@ -813,6 +813,7 @@
> > }
> >
> > .hidden {
> > +height: 0;
> > visibility: hidden;
> > }
> >
> >
> >