ICEfaces
  1. ICEfaces
  2. ICE-11512

ace:dialog seems to take up space when toggling visibility via server

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Won't Fix
    • Affects Version/s: EE-4.3.0.GA_P02, EE-3.3.0.GA_P08
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      Any

      Description

      The dialog seems to take up space on the page when hidden, causing a scrollbar to appear. This seems to happen only when toggling the visibility of the dialog via the server.

        Activity

        Hide
        Arturo Zambrano added a comment -

        Closing as Won't Fix. The issue is only visible if a dialog is taller than the screen's height, which is not a good design practice in the first place. Dialogs, in general, are a convenient way to show a small amount of information or input quickly, without having to load a new page. For bigger amounts of content, it's better to simply use a new page. The issue itself occurs because the 'hidden' CSS class of the dialog uses the styling 'visibility: hidden;' instead of 'display:none;'. However, it is necessary for the dialog to not have the 'display:none;' styling when hidden, because, otherwise, the dialog wouldn't be rendered properly when made visible.

        Show
        Arturo Zambrano added a comment - Closing as Won't Fix. The issue is only visible if a dialog is taller than the screen's height, which is not a good design practice in the first place. Dialogs, in general, are a convenient way to show a small amount of information or input quickly, without having to load a new page. For bigger amounts of content, it's better to simply use a new page. The issue itself occurs because the 'hidden' CSS class of the dialog uses the styling 'visibility: hidden;' instead of 'display:none;'. However, it is necessary for the dialog to not have the 'display:none;' styling when hidden, because, otherwise, the dialog wouldn't be rendered properly when made visible.

          People

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

            Dates

            • Created:
              Updated:
              Resolved: