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
(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.
On Wed, Nov 27, 2013 at 5:21 PM, Michal Petrov <
I've created a button group widget (the branch is here (
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
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?
Posted by forums
Original post: https://community.jboss.org/message/847489#847489
richfaces-dev mailing list