I disagree. The stuff we have doesn't work just fine. Try removing the clearing element and see what happens. The multi-column layout you're working on looks great, but now everything else is broken.
Our existing multi-column layout is extremely fragile and easily broken. That's why I suggested the column widgets. It's a bigger issue than dragging columns or changing sizes. -Adrian --- On Tue, 1/12/10, David E Jones <d...@me.com> wrote: > From: David E Jones <d...@me.com> > Subject: Re: Screen Widget screenlet and the hard coded br class="clear" > To: dev@ofbiz.apache.org > Date: Tuesday, January 12, 2010, 10:20 PM > > The point isn't to accommodate multi-column layouts, we > already have basic stuff for that which works just fine. I'm > not talking about anything like dragging columns or > configuring sizes or anything like that. It's an interesting > idea, but a different one. > > The question I have is do we really want this <br > class="clear"/> tag always inserted when you use the > screen widget screenlet element? > > Right now there is a bit of a "hack" in place to cause this > tag to do nothing, and that is by wrapping the screenlet in > a container with a style of "no-clear". > > So, the point is that is seems like in most cases this > clear isn't needed and we can avoid the no-clear hack by > having something explicit when it is needed. > > -David > > > On Jan 13, 2010, at 12:13 AM, Adrian Crum wrote: > > > We had a discussion about accommodating multi-column > layouts. Search the dev list for Subject = "Discussion: > Column Widgets (was: How to add a splitter to adjust the > left column width by dragging it?" > > > > -Adrian > > > > --- On Tue, 1/12/10, David E Jones <d...@me.com> > wrote: > > > >> From: David E Jones <d...@me.com> > >> Subject: Screen Widget screenlet and the hard > coded br class="clear" > >> To: dev@ofbiz.apache.org > >> Date: Tuesday, January 12, 2010, 10:07 PM > >> > >> I am looking into an issue with formatting using > a > >> CSS-based multiple column layout and with > screenlets in each > >> column. > >> > >> The problem is that in the > htmlScreenMacroLibrary.ftl file > >> there is a hard-coded clear on line 148: > >> > >> <br class="clear" /> > >> > >> With most browsers these clears are no > context-sensitive, > >> they clear ALL floats. This means that it messes > up the > >> multi-column formatting. > >> > >> Would anyone be too disappointed if we removed > this, and if > >> you need a clear below your screenlet then just > use a > >> container element after the screenlet element in > your screen > >> definition? > >> > >> Also, does anyone know the reason this was put > there in the > >> first place? > >> > >> Can anyone think of any issues removing this might > cause? > >> > >> -David > >> > >> > >> > > > > > > > >