ICEfaces
  1. ICEfaces
  2. ICE-7615

ace:pushButton - style or styleClass only applied to the html button element

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 3.0.RC1, 3.0.RC2
          • Fix Version/s: 3.0
          • Component/s: ACE-Components
          • Labels:
            None
          • Environment:
            ICEfaces 3.0
          • Assignee Priority:
            P1
          • Affects:
            Compatibility/Configuration

            Description

            When setting a style or styleClass on the ace:pushButton the CSS is applied to the html button tag and not the surrounding html elements that are rendered for the pushButton component.

              Activity

              Ascending order - Click to sort in descending order
              Arran Mccullough created issue -
              Arran Mccullough made changes -
              Field Original Value New Value
              Attachment pushButton-style-issue.JPG [ 13907 ]
              Hide
              Permalink
              Arran Mccullough added a comment -

              Attached test case showing issue

              Show
              Arran Mccullough added a comment - Attached test case showing issue
              Arran Mccullough made changes -
              Attachment Case10802Example.zip [ 13908 ]
              Attachment Case10802ExampleWAR.zip [ 13909 ]
              Arran Mccullough made changes -
              Salesforce Case [5007000000KDhxH]
              Ken Fyten made changes -
              Fix Version/s 3.0 [ 10241 ]
              Assignee Priority P2
              Assignee yip.ng [ yip.ng ]
              Hide
              Permalink
              yip.ng added a comment - - edited

              This I think was done on purpose. I remember we had some discussion on this long time ago in a meeting.

              Show
              yip.ng added a comment - - edited This I think was done on purpose. I remember we had some discussion on this long time ago in a meeting.
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #27091 Wed Jan 04 14:45:08 MST 2012 yip.ng ICE-7615: ace:pushButton - style or styleClass only applied to the html button element.
              Files Changed
              Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/component/pushbutton/PushButtonRenderer.java
              Hide
              Permalink
              yip.ng added a comment -

              Style and style class added to root div.

              Modified: C:\svn\ossrepo\icefaces3\trunk\icefaces\ace\component\src\org\icefaces\ace\component\pushbutton\PushButtonRenderer.java
              Completed: At revision: 27091

              Show
              yip.ng added a comment - Style and style class added to root div. Modified: C:\svn\ossrepo\icefaces3\trunk\icefaces\ace\component\src\org\icefaces\ace\component\pushbutton\PushButtonRenderer.java Completed: At revision: 27091
              yip.ng made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Hide
              Permalink
              Ken Fyten added a comment -

              In order to be consistent with the other ACE components, we should change this so that the style and styleClass attributes are always only rendered to the outside containing DIV (or SPAN) and not the internal component.

              This issue may also exist with the other buttons so those should be reviewed and changed as needed.

              Show
              Ken Fyten added a comment - In order to be consistent with the other ACE components, we should change this so that the style and styleClass attributes are always only rendered to the outside containing DIV (or SPAN) and not the internal component. This issue may also exist with the other buttons so those should be reviewed and changed as needed.
              Ken Fyten made changes -
              Resolution Fixed [ 1 ]
              Status Resolved [ 5 ] Reopened [ 4 ]
              Assignee Priority P2 P1
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #27107 Thu Jan 05 16:27:17 MST 2012 yip.ng ICE-7615: ace:pushButton - style or styleClass only applied to the html button element.
              Files Changed
              Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/component/pushbutton/PushButtonRenderer.java
              Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/component/linkbutton/LinkButtonRenderer.java
              Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/component/checkboxbutton/CheckboxButtonRenderer.java
              yip.ng made changes -
              Attachment screenshot-2.png [ 13948 ]
              Hide
              Permalink
              yip.ng added a comment - - edited

              Done. See screenshot-2.png. NB: This may break the styling in existing apps, depending on how the existing CSS rules are formulated.

              Modified: C:\svn\ossrepo\icefaces3\trunk\icefaces\ace\component\src\org\icefaces\ace\component\checkboxbutton\CheckboxButtonRenderer.java
              Modified: C:\svn\ossrepo\icefaces3\trunk\icefaces\ace\component\src\org\icefaces\ace\component\linkbutton\LinkButtonRenderer.java
              Modified: C:\svn\ossrepo\icefaces3\trunk\icefaces\ace\component\src\org\icefaces\ace\component\pushbutton\PushButtonRenderer.java
              Completed: At revision: 27107

              Show
              yip.ng added a comment - - edited Done. See screenshot-2.png. NB: This may break the styling in existing apps, depending on how the existing CSS rules are formulated. Modified: C:\svn\ossrepo\icefaces3\trunk\icefaces\ace\component\src\org\icefaces\ace\component\checkboxbutton\CheckboxButtonRenderer.java Modified: C:\svn\ossrepo\icefaces3\trunk\icefaces\ace\component\src\org\icefaces\ace\component\linkbutton\LinkButtonRenderer.java Modified: C:\svn\ossrepo\icefaces3\trunk\icefaces\ace\component\src\org\icefaces\ace\component\pushbutton\PushButtonRenderer.java Completed: At revision: 27107
              yip.ng made changes -
              Status Reopened [ 4 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Ken Fyten made changes -
              Affects [Compatibility/Configuration]
              Hide
              Permalink
              Carmen Cristurean added a comment - - edited

              Tested fix for PushButton & LinkButton with code revision 27142, in FF3.6, Chrome15 and IE8.

              Show
              Carmen Cristurean added a comment - - edited Tested fix for PushButton & LinkButton with code revision 27142, in FF3.6, Chrome15 and IE8.
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

                • Assignee:
                  yip.ng
                  Reporter:
                  Arran Mccullough
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  1 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: