ICEfaces
  1. ICEfaces
  2. ICE-9499

Revamp required/optional styling in showcase to not conflict with themeroller error styling

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 3.3
          • Fix Version/s: 4.0.BETA, 4.0
          • Component/s: Sample Apps
          • Labels:
            None
          • Environment:
            showcase
          • Assignee Priority:
            P1
          • Affects:
            Sample App./Tutorial

            Description

            When demonstrating the ACE components' functionality of required and optional, there is some application specified background colour styling (aqua, yellow, red, green) where the red styling interferes with the red foreground text colouring to make the text invisibly red on red. ace:textEntry and ace:textAreaEntry's Required Styling sections break from the aqua/yellow colouring to use red/green colouring, which causes problems. Other components' samples might have a similar problem.

            This became evident when we fixed some CSS ordering issues.

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              Arturo Zambrano added a comment -

              Committed changes in required/optional styling at revision 37366.

              This summarizes the changes made:

              • The background colours of the input fields are no longer styled by the application, leaving them to use the default theme background.
              • The border colours are now modified by the application to indicate required, optional, and error statuses.
              • Borders were made thicker (2px) to make the colours more noticeable.
              • When there's a validation error, the border is made even thicker (3px) and the colour more intense.
              • The colour palette was modified as well. Now, we only use "web colours". Required styling colours are more intense than optional styling colours. The default optional colour is no longer yellow, since yellow wasn't very noticeable as a border colour; it was changed to gray to communicate that the field is not particularly important.

              These changes play well with all of our pre-packaged Themeroller themes, as well as sam and rime.

              The situation described in this JIRA was present in all text input components, not only ace:textEntry and ace:textAreaEntry.

              Show
              Arturo Zambrano added a comment - Committed changes in required/optional styling at revision 37366. This summarizes the changes made: The background colours of the input fields are no longer styled by the application, leaving them to use the default theme background. The border colours are now modified by the application to indicate required, optional, and error statuses. Borders were made thicker (2px) to make the colours more noticeable. When there's a validation error, the border is made even thicker (3px) and the colour more intense. The colour palette was modified as well. Now, we only use "web colours". Required styling colours are more intense than optional styling colours. The default optional colour is no longer yellow, since yellow wasn't very noticeable as a border colour; it was changed to gray to communicate that the field is not particularly important. These changes play well with all of our pre-packaged Themeroller themes, as well as sam and rime. The situation described in this JIRA was present in all text input components, not only ace:textEntry and ace:textAreaEntry.
              Hide
              Permalink
              Arturo Zambrano added a comment - - edited

              Committed changes at revision 37420 and 37421:

              • Restricted required/optional styling to only Required Styling demos and to Overview textEntry and textAreaEntry demos.
              • Added 'None (Use theme)' checkbox in Required Styling demos (autoCompleteEntry, comboBox, dateTimeEntry, maskedEntry, selectMenu, simpleSelectOneMenu, textEntry, textAreaEntry).
              • Expanded explanation in Required Styling demos.
              • Added background and text colours to error classes.
              Show
              Arturo Zambrano added a comment - - edited Committed changes at revision 37420 and 37421: Restricted required/optional styling to only Required Styling demos and to Overview textEntry and textAreaEntry demos. Added 'None (Use theme)' checkbox in Required Styling demos (autoCompleteEntry, comboBox, dateTimeEntry, maskedEntry, selectMenu, simpleSelectOneMenu, textEntry, textAreaEntry). Expanded explanation in Required Styling demos. Added background and text colours to error classes.
              Hide
              Permalink
              Ken Fyten added a comment -

              The final set of changes to remove the "default" entry from the custom style color list and the addition of the CSS viewer look good. Marking resolved.

              Show
              Ken Fyten added a comment - The final set of changes to remove the "default" entry from the custom style color list and the addition of the CSS viewer look good. Marking resolved.

                People

                • Assignee:
                  Arturo Zambrano
                  Reporter:
                  Mark Collette
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  3 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: