ICEfaces
  1. ICEfaces
  2. ICE-10400

ace:panel - disableInputs rendering incorrectly when used in ace:dialog

          Details

            Description

            When using an ace:panel in an ace:dialog and setting disableInputs=true, the greyed out section that should be rendered over the inputs is rendered in the wrong location. The inputs are correctly disabled and can't but submitted so this is a cosmetic issue.

              Activity

              Ascending order - Click to sort in descending order
              Arran Mccullough created issue -
              Hide
              Permalink
              Arran Mccullough added a comment -

              Attached test case and source code.

              Steps:

              • Load welcomeICEfaces.jsf
              • Click the Open button
              • The dialog is rendered with the greyed out portion not located over the inputs.
              Show
              Arran Mccullough added a comment - Attached test case and source code. Steps: Load welcomeICEfaces.jsf Click the Open button The dialog is rendered with the greyed out portion not located over the inputs.
              Arran Mccullough made changes -
              Field Original Value New Value
              Attachment Case13160Example.war [ 17589 ]
              Attachment Case13160Example.zip [ 17590 ]
              Ken Fyten made changes -
              Assignee Mircea Toma [ mircea.toma ]
              Fix Version/s EE-4.0.0.GA [ 11171 ]
              Fix Version/s EE-3.3.0.GA_P03 [ 11572 ]
              Priority Major [ 3 ] Minor [ 4 ]
              Assignee Priority P2 [ 10011 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #43608 Wed Nov 26 13:05:01 MST 2014 mircea.toma ICE-10400 Output the input disabling script at the end of the form to give a chance to the more complex components to fully render and thus calculate the overlay coordinates properly.
              Files Changed
              Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/resources/icefaces.ace/util/blockui.js
              Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/src/org/icefaces/ace/component/panel/PanelRenderer.java
              Hide
              Permalink
              Mircea Toma added a comment -

              Output the input disabling script at the end of the form to give a chance to the more complex components to fully render and thus calculate the overlay coordinates properly.

              Show
              Mircea Toma added a comment - Output the input disabling script at the end of the form to give a chance to the more complex components to fully render and thus calculate the overlay coordinates properly.
              Mircea Toma made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Liana Munroe made changes -
              Attachment FF.PNG [ 17686 ]
              Attachment IE-Chrome.PNG [ 17687 ]
              Ken Fyten made changes -
              Resolution Fixed [ 1 ]
              Status Resolved [ 5 ] Reopened [ 4 ]
              Hide
              Permalink
              Mircea Toma added a comment - - edited
              • "Should the entire panel receive the overlay or just the inputs?"
                The content of the panel (excluding its header) should receive the overlay.
              • "Using FF 33 the overlay is not rendered and the ace:dialog becomes much larger and scrollable."
                I used mainly FF 33 for testing and I don't see the issue you are describing. Please make sure the browser's cache is cleared before running the test.
              Show
              Mircea Toma added a comment - - edited "Should the entire panel receive the overlay or just the inputs?" The content of the panel (excluding its header) should receive the overlay. "Using FF 33 the overlay is not rendered and the ace:dialog becomes much larger and scrollable." I used mainly FF 33 for testing and I don't see the issue you are describing. Please make sure the browser's cache is cleared before running the test.
              Hide
              Permalink
              Liana Munroe added a comment -

              Verified ICEfaces 4 trunk r43613, Tomcat 7, all browsers.
              Not able to reproduce FF issue.

              Show
              Liana Munroe added a comment - Verified ICEfaces 4 trunk r43613, Tomcat 7, all browsers. Not able to reproduce FF issue.
              Ken Fyten made changes -
              Status Reopened [ 4 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Ken Fyten made changes -
              Fix Version/s 4.1 [ 11375 ]
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

                • Assignee:
                  Mircea Toma
                  Reporter:
                  Arran Mccullough
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  3 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: