ICEfaces
  1. ICEfaces
  2. ICE-11123

Support using buttonGroup with none-nested buttons via "group" attribute

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 4.2.BETA, 4.2
    • Component/s: ACE-Components, Sample Apps
    • Labels:
      None
    • Environment:
      ICEfaces 4.x
    • Assignee Priority:
      P2
    • Affects:
      Sample App./Tutorial

      Description

      The original intention for ace:buttonGroup was to support two distinct modes of operation:
      1. Directly nested child checkboxButton and radioButton components.
      2. None-nested buttons via specifying the id of a buttonGroup in the "group" attribute of radioButton/checkboxButton.

      The 1st (nested) scenario has been working since ICEfaces 4.0, but the 2nd (none-nested) scenario has not worked until now.

        Activity

        Hide
        Judy Guglielmin added a comment -

        rev 49344 has two new examples plus code to support the buttonGroup that can be used before the buttons on the page (from any location).
        Note that the number of rows is something for QA to copy to their tests and then should be removed from the showcase example.

        Show
        Judy Guglielmin added a comment - rev 49344 has two new examples plus code to support the buttonGroup that can be used before the buttons on the page (from any location). Note that the number of rows is something for QA to copy to their tests and then should be removed from the showcase example.
        Hide
        Judy Guglielmin added a comment -

        found an unused attribute I added when first starting this work so cleaning it up.

        Show
        Judy Guglielmin added a comment - found an unused attribute I added when first starting this work so cleaning it up.
        Hide
        Judy Guglielmin added a comment -

        rev 49345

        Show
        Judy Guglielmin added a comment - rev 49345
        Hide
        Liana Munroe added a comment - - edited

        Tested with ICEfaces 4 trunk r49346, Tomcat 7/8 WIndows7, FF 47, Chrome 53, IE 11.
        Added QA test applications /ICE-11123.jsf and /ICE-11123-2.jsf to:
        http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces4/Sparkle/Nightly/buttonGroup.

        In the QA test applications and the showcase demos the 500 and 1000 row dataTables performed poorly.
        The loading of 500 rows took ~6 seconds. Each selection of a checkbox or radioButton took > 1 second.
        -The loading of 1000 rows was only accomplished with Chrome browser, taking about 20 seconds. FF and IE both locked up with long running script errors. See attached screen shot. Making a radioButton or checkboxButton selection can take from 4 to 15 seconds depending on the browser.

        Show
        Liana Munroe added a comment - - edited Tested with ICEfaces 4 trunk r49346, Tomcat 7/8 WIndows7, FF 47, Chrome 53, IE 11. Added QA test applications / ICE-11123 .jsf and / ICE-11123 -2.jsf to: http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces4/Sparkle/Nightly/buttonGroup . In the QA test applications and the showcase demos the 500 and 1000 row dataTables performed poorly. The loading of 500 rows took ~6 seconds. Each selection of a checkbox or radioButton took > 1 second. -The loading of 1000 rows was only accomplished with Chrome browser, taking about 20 seconds. FF and IE both locked up with long running script errors. See attached screen shot. Making a radioButton or checkboxButton selection can take from 4 to 15 seconds depending on the browser.
        Hide
        Judy Guglielmin added a comment -

        need to remove QA tests from showcase example

        Show
        Judy Guglielmin added a comment - need to remove QA tests from showcase example
        Hide
        Judy Guglielmin added a comment -

        rev 49588...removed ability to change number of rows in examples of column and row. (they are now in QA tests)

        Show
        Judy Guglielmin added a comment - rev 49588...removed ability to change number of rows in examples of column and row. (they are now in QA tests)

          People

          • Assignee:
            Judy Guglielmin
            Reporter:
            Ken Fyten
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: