ICEfaces
  1. ICEfaces
  2. ICE-2750

Extra div added to end of document on certain submit clicks.

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Critical Critical
          • Resolution: Fixed
          • Affects Version/s: 1.7
          • Fix Version/s: 1.7
          • Component/s: None
          • Labels:
            None
          • Environment:
            All
          • Support Case References:
            4590

            Description

            An extra (hidden) div is added to the end of the document body on certain initial submit clicks (e.g. collapsible panel, tree node), which may cause the browser scrollbars to appear where there was none initially.

              Activity

              Ascending order - Click to sort in descending order
              yip.ng created issue -
              yip.ng made changes -
              Field Original Value New Value
              Support Case References 4590
              Fix Version/s 1.7 [ 10080 ]
              Hide
              Permalink
              yip.ng added a comment -

              Screenshots showing problem occurring in a screen of collapsible panels.

              Show
              yip.ng added a comment - Screenshots showing problem occurring in a screen of collapsible panels.
              yip.ng made changes -
              Attachment CollapsiblePanelProblem.doc [ 10818 ]
              Hide
              Permalink
              yip.ng added a comment -

              screenshot showing the added extra div.

              Show
              yip.ng added a comment - screenshot showing the added extra div.
              yip.ng made changes -
              Attachment screenshot-1.jpg [ 10819 ]
              Ken Fyten made changes -
              Assignee Ted Goddard [ ted.goddard ]
              Hide
              Permalink
              Ted Goddard added a comment -

              Do we know if this is a new bug? Yip, please try some older versions
              of ICEfaces for this problem to help narrow down when it occurred.

              Show
              Ted Goddard added a comment - Do we know if this is a new bug? Yip, please try some older versions of ICEfaces for this problem to help narrow down when it occurred.
              Ted Goddard made changes -
              Assignee Ted Goddard [ ted.goddard ] Yip Ng [ yip.ng ]
              Ken Fyten made changes -
              Assignee Priority P1
              Ken Fyten made changes -
              Assignee Yip Ng [ yip.ng ] Arturo Zambrano [ artzambrano ]
              Arturo Zambrano made changes -
              Status Open [ 1 ] In Progress [ 3 ]
              Hide
              Permalink
              Arturo Zambrano added a comment -

              I couldn't reproduce this issue. I used both ICEfaces 1.6.2 and 1.7 beta1 on Tomcat 6, on IE 6 and FF2. There was no horizontal scrollbar at all, only the vertical one when expanding a panel but this happened normally, no extra space below.

              I used five of these on a page:
              <ice:panelCollapsible>
              <f:facet name="header">
              <ice:outputText value="title" />
              </f:facet>
              <ice:panelGroup>
              <ice:selectInputDate />
              </ice:panelGroup>
              </ice:panelCollapsible>

              Could you give me more details about how you reproduced it? Which browsers, which servers, exactly which components besides the collapsible panel (if any)? In conjunction with which other components? Was the change permanent or only for a while?

              Show
              Arturo Zambrano added a comment - I couldn't reproduce this issue. I used both ICEfaces 1.6.2 and 1.7 beta1 on Tomcat 6, on IE 6 and FF2. There was no horizontal scrollbar at all, only the vertical one when expanding a panel but this happened normally, no extra space below. I used five of these on a page: <ice:panelCollapsible> <f:facet name="header"> <ice:outputText value="title" /> </f:facet> <ice:panelGroup> <ice:selectInputDate /> </ice:panelGroup> </ice:panelCollapsible> Could you give me more details about how you reproduced it? Which browsers, which servers, exactly which components besides the collapsible panel (if any)? In conjunction with which other components? Was the change permanent or only for a while?
              Ken Fyten made changes -
              Assignee Arturo Zambrano [ artzambrano ] Yip Ng [ yip.ng ]
              Hide
              Permalink
              yip.ng added a comment -

              Test app. to reproduce the problem. Run in Windows XP, Tomcat 5.5.25, IE 7. See the previously attached screenshot for the outcome. First use the IE Developer Toolbar to inspect the DOM tree, then click on a collapsible panel, then click on the refresh button in the Developer Toolbar, then inspect the DOM tree again. The result is as seen in the screenshot.

              Show
              yip.ng added a comment - Test app. to reproduce the problem. Run in Windows XP, Tomcat 5.5.25, IE 7. See the previously attached screenshot for the outcome. First use the IE Developer Toolbar to inspect the DOM tree, then click on a collapsible panel, then click on the refresh button in the Developer Toolbar, then inspect the DOM tree again. The result is as seen in the screenshot.
              yip.ng made changes -
              Attachment support-4590.zip [ 10867 ]
              Ken Fyten made changes -
              Assignee Priority P1 P2
              Assignee Yip Ng [ yip.ng ] Adnan Durrani [ adnan.durrani ]
              Priority Major [ 3 ] Minor [ 4 ]
              Ken Fyten made changes -
              Assignee Priority P2 P1
              Assignee Adnan Durrani [ adnan.durrani ] Mircea Toma [ mircea.toma ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #16102 Tue Mar 25 11:25:58 MDT 2008 mircea.toma Take the hidden DIV out of the flow.
              ICE-2750
              Files Changed
              Commit graph MODIFY /icefaces/trunk/icefaces/bridge/lib/element.js
              Hide
              Permalink
              Mircea Toma added a comment -

              The DIV is used by the bridge to generate new elements from the received updates. Taken the hidden DIV out of the flow since it should not interact with the displayed content.

              Show
              Mircea Toma added a comment - The DIV is used by the bridge to generate new elements from the received updates. Taken the hidden DIV out of the flow since it should not interact with the displayed content.
              Mircea Toma made changes -
              Status In Progress [ 3 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]
              Assignee Priority P1
              Assignee Mircea Toma [ mircea.toma ]
              Michael Thiem made changes -
              Resolution Fixed [ 1 ]
              Status Closed [ 6 ] Reopened [ 4 ]
              Michael Thiem made changes -
              Attachment TestLayout.zip [ 10924 ]
              Michael Thiem made changes -
              Fix Version/s 1.7 [ 10080 ]
              Affects Version/s 1.7 [ 10080 ]
              Affects Version/s 1.7Beta1 [ 10121 ]
              Affects Version/s 1.6.2 [ 10111 ]
              Priority Minor [ 4 ] Critical [ 2 ]
              Ken Fyten made changes -
              Status Reopened [ 4 ] Closed [ 6 ]
              Fix Version/s 1.7 [ 10080 ]
              Resolution Fixed [ 1 ]
              Ken Fyten made changes -
              Comment [ issue is not yet fixed, the problem is still the same ]
              Ken Fyten made changes -
              Comment [ A comment with security level 'icesoft-internal-developers' was removed ]
              Ken Fyten made changes -
              Comment [ Created a new JIRA for this follow-up issue: ICE-3010.

              Re-closing. ]
              Ken Fyten made changes -
              Attachment TestLayout.zip [ 10924 ]

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: