Hello again, I have yet another question, do you prefer af|panelBox::dark-content or af|panelBox::content-dark ?
On 8/23/06, Simon Lessard <[EMAIL PROTECTED]> wrote:
Ok, I'll keep it with ::<backgroundValue>-content then. Another question about this issue. Would the skin engine support it if ALL panelBox background were supported as long as the skinner specified that background? Basic skins could continue to support light, medium, dark and transparent. But I see no reason why to not allow skinners to create a "marble" box for example. On 8/22/06, Adam Winer <[EMAIL PROTECTED]> wrote: > > Hrm, technically, I don't consider them states, so much > as attribute selectors. > > If we wanted to be *majorly* sophisticated, we'd support > something like: > > af|panelBox[background="dark"] > > ... but my head hurts trying to imagine how we'd implement that. :) > > -- Adam > > > > > On 8/22/06, Simon Lessard <[EMAIL PROTECTED]> wrote: > > Little semantic question here while playng around with panelBox. > Should > > ::dark, ::light, ::medium and ::transparent be considered states > instead of > > parts since they represent an attribute value? Said otherwise should > we > > rename the following selectors: > > > > af|panelBox::dark --> af|panelBox:dark > > af|panelBox::content-dark --> af|panelBox:dark::content (same for all > -dark, > > -light, ...) > > > > For the added class, they would become: > > af|panelBox:dark::bottom-start-corner and such. > > > > I would not mind renaming it while at it. Furthemore it will allow me > to > > test how well the code I made for :required::content and > :required::label > > works when applied to something else. > > > > > > Regards, > > > > ~ Simon > > > > On 8/22/06, Simon Lessard <[EMAIL PROTECTED]> wrote: > > > > > > Sure! > > > > > > > > > On 8/22/06, Jeanne Waldman <[EMAIL PROTECTED]> wrote: > > > > > > > > Please use "start" instead of "left" and "end" instead of "right". > > > > > This way it still makes sense in rtl languages. > > > > > > > > See this link for selector names for inspiration: > > > > > > > > > http://www.oracle.com/technology/products/jdev/htdocs/partners/addins/exchange/jsf/doc/skin-selectors.html > > > > > > > > - Jeanne > > > > > > > > Adam Winer wrote: > > > > > > > > > +1 to this; and put an "af|panelBox" on the top <table>, > 'course. > > > > > > > > > > -- Adam > > > > > > > > > > > > > > > On 8/22/06, Simon Lessard < [EMAIL PROTECTED]> wrote: > > > > > > > > > >> Hello everyones, > > > > >> > > > > >> I see that image borders around panelBox are no longer > possible, is > > > > >> there a > > > > >> reason for this or it was simply forgotten while switching to > faces > > > > >> major > > > > >> renderer? If that was an image package issue, I think we can > re-add > > > > the > > > > >> support without calling that API simply by generating something > like > > > > >> > > > > >> > > > > >> <table> > > > > >> <tr> > > > > >> <td class="af|panelBox::top-left-corner"/> > > > > >> <td class="af|panelBox::top"/> > > > > >> <td class="af|panelBox::top-right-corner"/> > > > > >> </tr> > > > > >> <tr> > > > > >> <td class="af|panelBox::left"/> > > > > >> <td class="af|panelBox::content"> > > > > >> <!-- Box content --> > > > > >> </td> > > > > >> <td class="af|panelBox::right"/> > > > > >> </tr> > > > > >> <tr> > > > > >> <td class="af|panelBox::bottom-left-corner"/> > > > > >> <td class="af|panelBox::bottom"/> > > > > >> <td class="af|panelBox::bottom-right-corner"/> > > > > >> </tr> > > > > >> </table> > > > > >> > > > > >> with style classes looking like: > > > > >> af|panelBox::top > > > > >> { > > > > >> background-image: url(...); > > > > >> background-repeat: repeat-x; > > > > >> } > > > > >> > > > > >> af|panelBox::left > > > > >> { > > > > >> background-image: url(...); > > > > >> background-repeat: repeat-y; > > > > >> } > > > > >> > > > > >> af|panelBox::bottom > > > > >> { > > > > >> background-image: url(...); > > > > >> background-repeat: repeat-x; > > > > >> } > > > > >> > > > > >> af|panelBox::right > > > > >> { > > > > >> background-image: url(...); > > > > >> background-repeat: repeat-y; > > > > >> } > > > > >> > > > > >> af|panelBox::top-left-corner > > > > >> { > > > > >> background-image: url(...); > > > > >> height: imageHeight; > > > > >> width: imageWidth; > > > > >> } > > > > >> > > > > >> af|panelBox::top-right-corner > > > > >> { > > > > >> background-image: url(...); > > > > >> height: imageHeight; > > > > >> width: imageWidth; > > > > >> } > > > > >> > > > > >> af|panelBox::bottom-left-corner > > > > >> { > > > > >> background-image: url(...); > > > > >> height: imageHeight; > > > > >> width: imageWidth; > > > > >> } > > > > >> > > > > >> af|panelBox::bottom-right-corner > > > > >> { > > > > >> background-image: url(...); > > > > >> height: imageHeight; > > > > >> width: imageWidth; > > > > >> } > > > > >> > > > > >> af|panelBox::content > > > > >> { > > > > >> background-color: rgb(...); > > > > >> } > > > > >> > > > > >> > > > > >> Is there anything I don't see? That would prevent it from > working? > > > > >> > > > > >> > > > > > > > > > > > > > > > > > > > > >
