ICEfaces
  1. ICEfaces
  2. ICE-6246

ACE: Rationalize pushButton, checkboxButton, and linkButton skin CSS classes/files used

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0-Beta2
    • Fix Version/s: 2.0.0
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      IF 2 Adv. Comps.
    • Affects:
      Documentation (User Guide, Ref. Guide, etc.), Compatibility/Configuration

      Description

      Currently the CSS styling, classnames, and CSS files being used by the pushButton, checkboxButton, and linkButton are a bit-muddled with duplication and overlap. This is a result of each of these buttons originating from a single YUI button component.

      To clean this up each button type should have it's own distinct CSS classnames and CSS files it references to avoid unintended overlap/influence between each component's styling, and also to optimize the CSS files being included so no duplicates are used.

        Activity

        Hide
        Ken Fyten added a comment -

        It also seems like the same CSS classes are reference/linked multiple times for each component. This needs to be cleaned up.

        Show
        Ken Fyten added a comment - It also seems like the same CSS classes are reference/linked multiple times for each component. This needs to be cleaned up.
        Hide
        Ken Fyten added a comment -

        Also note that it's okay for the Rime and Sam skins for these buttons to use images to represent the button states, as one of the main reasons to use these buttons instead of say, h: or ice: commandButton, etc. is the image-skinning. We do not need these buttons to use default browser styling for the Rime skin.

        Show
        Ken Fyten added a comment - Also note that it's okay for the Rime and Sam skins for these buttons to use images to represent the button states, as one of the main reasons to use these buttons instead of say, h: or ice: commandButton, etc. is the image-skinning. We do not need these buttons to use default browser styling for the Rime skin.

          People

          • Assignee:
            Arturo Zambrano
            Reporter:
            Ken Fyten
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: