ICEfaces
  1. ICEfaces
  2. ICE-7607

showcase - ace:linkButton & ace:pushButton do not render properly in IE browsers

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 3.0.RC2
    • Fix Version/s: 3.0
    • Component/s: Sample Apps
    • Labels:
      None
    • Environment:
      Browsers: IE 7/8/9
      Code Rev: 3.0.0.RC2 build 8
      Server: Tomcat 7
    • Assignee Priority:
      P1

      Description

      These two components do not render properly when first loading their pages in the showcase application:
      http://localhost:8080/showcase/showcase.jsf?grp=aceMenu&exp=linkButton
      http://localhost:8080/showcase/showcase.jsf?grp=aceMenu&exp=pushButton

      IE 7:
      ace:pushButton & ace:linkButton - Overview : when first loading the page the component is not rendered on the page inside the tab.

      IE 8:
      ace:pushButton & ace:linkButton - Overview : when first loading the page the component render over the tabs (screen shots attached). After switching tabs, or just moving the mouse over the page in case of the pushButton, they will be in the right place on the page.

      IE 9: both components render in the wrong place when first loading the page.

        Issue Links

          Activity

          Carmen Cristurean created issue -
          Carmen Cristurean made changes -
          Field Original Value New Value
          Attachment 1.png [ 13899 ]
          Attachment 2.png [ 13900 ]
          Ken Fyten made changes -
          Salesforce Case []
          Fix Version/s 3.0 [ 10241 ]
          Assignee Priority P2
          Assignee Mark Collette [ mark.collette ]
          Ken Fyten made changes -
          Security Private [ 10001 ]
          Ken Fyten made changes -
          Salesforce Case []
          Assignee Priority P2 P1
          Show
          Nils Lundquist added a comment - http://social.msdn.microsoft.com/Forums/en-SG/iewebdevelopment/thread/267ec272-7976-4062-9993-b107af87b1b9 possibly related.
          Hide
          Cruz Miraback added a comment -

          I've attached a war file that is potentially related to this problem.
          This test app has some h:inputText components inside an ace:tabSet. The input components aren't visible until switching tabs.

          Show
          Cruz Miraback added a comment - I've attached a war file that is potentially related to this problem. This test app has some h:inputText components inside an ace:tabSet. The input components aren't visible until switching tabs.
          Cruz Miraback made changes -
          Attachment tabset.war [ 13968 ]
          Ken Fyten made changes -
          Assignee Mark Collette [ mark.collette ] Cruz Miraback [ cmiraback ]
          Ken Fyten made changes -
          Salesforce Case []
          Assignee Cruz Miraback [ cmiraback ] Nils Lundquist [ nils.lundquist ]
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #27192 Fri Jan 13 14:48:42 MST 2012 nils.lundquist ICE-7607 - Reverting styling change made for ICE-6728 to fix TabSet IE rendering issues.
          Files Changed
          Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/tabset/tabset.css
          Hide
          Nils Lundquist added a comment - - edited

          26336
          11-11-12 5:59 AM
          mark.collette
          ICE-6738 : Add client-side cache support for ace:tabPane within a 'clientSide=false'

          Is the Grimlock revision causing this regression.

          actual JIRA number is ICE-6728

          Show
          Nils Lundquist added a comment - - edited 26336 11-11-12 5:59 AM mark.collette ICE-6738 : Add client-side cache support for ace:tabPane within a 'clientSide=false' Is the Grimlock revision causing this regression. actual JIRA number is ICE-6728
          Hide
          Nils Lundquist added a comment -

          .yui-navset .yui-content .ui-tabs-hide

          { border:0; height:0; width:0; position:absolute; left:-999999px; overflow:hidden; visibility:hidden; padding:0; }

          Is the specific rule causing this error. Trying to look into its usage is difficult however. It is only referenced in code once, in JS, and there it is suppled to the YUI Tab object, which as far as i can tell never uses it again explicitly. It is also referenced a second time in JS, but this has been commented out.

          Is this even needed?

          Show
          Nils Lundquist added a comment - .yui-navset .yui-content .ui-tabs-hide { border:0; height:0; width:0; position:absolute; left:-999999px; overflow:hidden; visibility:hidden; padding:0; } Is the specific rule causing this error. Trying to look into its usage is difficult however. It is only referenced in code once, in JS, and there it is suppled to the YUI Tab object, which as far as i can tell never uses it again explicitly. It is also referenced a second time in JS, but this has been commented out. Is this even needed?
          Hide
          Nils Lundquist added a comment -

          Tentatively fixed by reverting the styling rule in question. Still await Marks input about possible regressions caused by this change.

          Show
          Nils Lundquist added a comment - Tentatively fixed by reverting the styling rule in question. Still await Marks input about possible regressions caused by this change.
          Nils Lundquist made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Hide
          Mark Collette added a comment -

          It should be fine pulling out some of that styling. Really, as long as it passes testing in all browsers, we're good to go.

          Show
          Mark Collette added a comment - It should be fine pulling out some of that styling. Really, as long as it passes testing in all browsers, we're good to go.
          Ken Fyten made changes -
          Link This issue blocks ICE-7602 [ ICE-7602 ]
          Ken Fyten made changes -
          Link This issue blocks ICE-7639 [ ICE-7639 ]
          Hide
          Carmen Cristurean added a comment -

          I re-tested the showcase app with ICEfaces3 trunk revision 27232, in IE7 and IE8, and found this issue has been resolved.

          Show
          Carmen Cristurean added a comment - I re-tested the showcase app with ICEfaces3 trunk revision 27232, in IE7 and IE8, and found this issue has been resolved.
          Ken Fyten made changes -
          Status Resolved [ 5 ] Closed [ 6 ]

            People

            • Assignee:
              Nils Lundquist
              Reporter:
              Carmen Cristurean
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: