ButtonGroup RichWidget

[Lukáš Fryč]

Hey Michal,

I have reviewed your impl quickly

and there is one conceptual problem I have found:

in order to achieve as slick performance as possible on client-side,
we should prefer CSS-only components over JS+CSS ones.

I believe ButtonGroup is a sample of component that doesn't have to use JS
(Widget Factory)
at all.

(Note that JSF Bridge code may be required though).

Even in case where JS is required, it is good practice to build whole
component on simple and CSS-styled markup
and the continue to tie in JS things like event handling.

~ Lukas


On Wed, Nov 27, 2013 at 5:21 PM, Michal Petrov <
richfaces-dev at lists.jboss.org> wrote:

> Hi all,
>
> I've created a button group widget (the branch is here (
> https://github.com/richwidgets/richwidgets/tree/buttongroup)). Based on
> what's showcased on the Bootstrap site (
> http://getbootstrap.com/components/#btn-groups) I've implemented sizing,
> justification, vertical grouping and button styling. The widget also turns
> nested lists into dropdowns (which I guess can be expanded to process any
> nested elements).
>
> I'd ask for a code review (since this is my first work on RichWidgets) and
> I also have a few questions:
> * How much styling are we expecting on user side? Right now I'm only
> applying style to buttons if they weren't styled already (and if they
> aren't styled the group doesn't work).
> * Should I write any tests?
> * Any other functionality to add?
>
> Michal
>
> Posted by forums
> Original post: https://community.jboss.org/message/847489#847489
> _______________________________________________
> richfaces-dev mailing list
> richfaces-dev at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/richfaces-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/richfaces-dev/attachments/20131128/eb17db98/attachment.html