myfaces-users mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Adam Winer" <awi...@gmail.com>
Subject Re: [TRINIDAD] Proposal - fieldset/legend component
Date Tue, 04 Sep 2007 18:42:48 GMT
On 9/4/07, Matt Cooper <matt.faces@gmail.com> wrote:
> I like that name too.  Should the text for it be "text" like on other
> components or "caption"?

We already have an example of convenience-attribute+facet
on column;  there it's "header" and "headerText".  So
perhaps the facet should be "caption" and the attribute
"captionText".

-- Adam


>
> Regarding this example mentioned earlier:
>
> <tr:panelFormLayout>
>   <tr:panelCaptionGroup caption="Personal Details">
>       <tr:inputText.../>
>   </tr:panelCaptionGroup>
> </tr:panelFormLayout>
>
> this could potentially be supported but without the lines drawn along the
> sides.  I'm thinking that when placed directly inside of a panelFormLayout,
> it could have the simple separator lines (exactly like tr:group today) but
> with the caption appearing on top of the upper separator line, e.g.:
>
> label1    field1
> -- Caption -----
> label2    field2
> label3    field3
> ----------------
> label4    field4
>
> and when outside of a panelFormLayout (as in your proposal) where you don't
> have to play well with the table structure of the form layout, the
> PanelCaptionGroupRenderer will use the normal fieldset elements.
>
> Regards,
> Matt
>
>
> On 9/4/07, Adam Winer <awiner@gmail.com> wrote:
> > On 9/4/07, Danny Robinson <dannyjrobinson@gmail.com> wrote:
> > > Yeah, I initially went with the enhanced panelFormLayout approach and
> > > decided against it.
> > >
> > > The name is also frustrating, so how about <tr:panelCaptionGroup ...>?
> >
> > I think I like that name.
> >
> > -- Adam
> >
> > >
> > > I'll give it 24hrs for other people to chime in, then I'll go with the
> > > above.
> > >
> > > Thanks,
> > >
> > > Danny
> > >
> > >
> > > On 9/4/07, Adam Winer <awiner@gmail.com> wrote:
> > > > Yep, I agree that the Layout suffix shouldn't be there.
> > > > I'm wondering if we want a name more abstracted from
> > > > the HTML "fieldset" concept, though none really
> > > > come to mind.
> > > >
> > > > Or, as a perhaps more radical suggestion, could this be
> > > > something embedded inside panelFormLayout?  Like if
> > > > we supported "caption" on af:group?
> > > >
> > > > <af:panelFormLayout>
> > > >   <af:group caption="Personal Details">
> > > >       <af:inputText.../>
> > > >   </af:group>
> > > > </af:panelFormLayout>
> > > >
> > > > This has some major disadvantages, though.  It's probably
> > > > significantly harder to implement (more grunge in the
> > > > already grungy PanelFormLayout code) and no way to use it
> > > > outside of panelFormLayout.  So I've probably just talked myself
> > > > out of this suggestion. :)
> > > >
> > > > -- Adam
> > > >
> > > >
> > > >
> > > > On 8/30/07, Simon Lessard <simon.lessard.3@gmail.com > wrote:
> > > > > Hello Danny,
> > > > >
> > > > > True, panelBox does not include the Layout suffix (a fact I somehow
> > > > > forgotten / overlooked), so stripping the Layout suffix is fine with
> me.
> > > I
> > > > > like the attribute shortcut for the legend as well.
> > > > >
> > > > >
> > > > > ~ Simon
> > > > >
> > > > >
> > > > > On 8/30/07, Danny Robinson <dannyjrobinson@gmail.com> wrote:
> > > > > > I knocked together this component to see what it would look
like
> and
> > > what
> > > > > skin selectors would be required.  Here's some tag examples and an
> > > attached
> > > > > screenshot.
> > > > > >
> > > > > > FWIW while Simon's name suggestion was better, I'm not fan of
the
> > > 'Layout'
> > > > > suffix, as it doesn't actually perform layout, and is therefore
> closer
> > > in
> > > > > functionality to panelBox.
> > > > > >
> > > > > >                   <tr:panelFieldsetLayout captionText="Personal
> > > Details">
> > > > > >                     <tr:panelFormLayout>
> > > > > >                       <tr:inputText label="First Name" />
> > > > > >                       <tr:inputText label="Middle Initial"
> > > columns="1"/>
> > > > > >                       <tr:inputText label="Family Name" />
> > > > > >                       <tr:selectOneChoice label="Suffix"
/>
> > > > > >                     </tr:panelFormLayout>
> > > > > >                   </tr:panelFieldsetLayout>
> > > > > >
> > > > > > OR
> > > > > >                 <tr:panelFieldsetLayout
> > > > > >                   captionText="This shouldn't appear">
> > > > > >                   <f:facet name="caption">
> > > > > >                     <tr:icon name="error" />
> > > > > >                   </f:facet>
> > > > > >                   <tr:inputDate label="Date of Birth" />
> > > > > >                 </tr:panelFieldsetLayout>
> > > > > >
> > > > > > Thoughts anyone,
> > > > > >
> > > > > > D.
> > > > > >
> > > > > >
> > > > > >
> > > > > > On 8/29/07, Simon Lessard < simon.lessard.3@gmail.com>
wrote:
> > > > > > > Hello Danny,
> > > > > > >
> > > > > > > I would prefer something along the lines of
> tr:panelFieldSetLayout
> > > with
> > > > > a legend facet.
> > > > > > >
> > > > > > >
> > > > > > > Regards,
> > > > > > >
> > > > > > > ~ Simon
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > > On 8/29/07, Danny Robinson < dannyjrobinson@gmail.com>
wrote:
> > > > > > > > Would there be interest in a skinnable/ajaxified component
> that
> > > > > rendered output using html fieldset/legend.  We'd like to include
> this
> > > in
> > > > > our applications and I'm keen to keep our users away from
> introducing
> > > other
> > > > > component libraries just for the odd extra component.
> > > > > > > >
> > > > > > > > I'm struggling to think of a nice sounding tag name,
other
> than
> > > > > tr:panelBoxAndLegend but it could look something like:
> > > > > > > >
> > > > > > > > <tr:panelBoxAndLegend title="The legend">
> > > > > > > > ...
> > > > > > > > </tr:panelBoxAndLegend>
> > > > > > > >
> > > > > > > > Thanks,
> > > > > > > >
> > > > > > > > Danny
> > > > > > > >
> > > > > > > > --
> > > > > > > > Chordiant Software Inc.
> > > > > > > > www.chordiant.com
> > > > > > >
> > > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > > --
> > > > > > Chordiant Software Inc.
> > > > > > www.chordiant.com
> > > > > >
> > > > >
> > > > >
> > > >
> > >
> > >
> > >
> > > --
> > >
> > > Chordiant Software Inc.
> > > www.chordiant.com
> >
>
>

Mime
View raw message