Details

    • Type: New Feature New Feature
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 4.3
    • Component/s: None
    • Labels:
      None
    • Environment:
      Java 7

      Description


      It'd be really useful to have a Dashboard component like the one in PrimeFaces (http://www.primefaces.org/showcase/ui/dashboard.jsf), where you can move your panels around and it'll save the state in the model.

      Thanks

        Activity

        Hide
        Arturo Zambrano added a comment -

        r52437: introduced table layout in order to render footer at the bottom of the pane and make content div take all available height; removed css resources from Meta class, since they are already included in combined.css; change resize handler to use fontawesome, so it looks well on any theme; added destroy function; added custom event classes for ajax events; changed close event to silent mode, so that panes below the one closed stay where they are

        Show
        Arturo Zambrano added a comment - r52437: introduced table layout in order to render footer at the bottom of the pane and make content div take all available height; removed css resources from Meta class, since they are already included in combined.css; change resize handler to use fontawesome, so it looks well on any theme; added destroy function; added custom event classes for ajax events; changed close event to silent mode, so that panes below the one closed stay where they are
        Hide
        Ken Fyten added a comment - - edited

        Review Notes

        • In the demos it's possible for the dashboard panes to be positioned partially off the right side of the visible demo area. Can we prevent that or make it a scrollable region?
        • In the Dynamic Panes demo if you add more panes than will fit on the page it stops working, you can't close the panes or add any, even if you reload the page. Should use a vertically scrollable region for the dashboard to live inside?
        • It would be more flexible if each dashboardPane could be set to resizable, moveable, or closable. That way you could have panes that are fixed in size or location and cannot be removed, plus others that can. Is that a feasible improvement?
        • Missing Wiki docs
        Show
        Ken Fyten added a comment - - edited Review Notes In the demos it's possible for the dashboard panes to be positioned partially off the right side of the visible demo area. Can we prevent that or make it a scrollable region? In the Dynamic Panes demo if you add more panes than will fit on the page it stops working, you can't close the panes or add any, even if you reload the page. Should use a vertically scrollable region for the dashboard to live inside? It would be more flexible if each dashboardPane could be set to resizable, moveable, or closable. That way you could have panes that are fixed in size or location and cannot be removed, plus others that can. Is that a feasible improvement? Missing Wiki docs
        Hide
        Arturo Zambrano added a comment -

        Regarding the first bullet point, I don't see that issue on Chrome, FF, and Edge on Windows 10. The whole dashboard fits perfectly with enough white space on both sides. Did you mean make it more centered?

        Regarding the third bullet point, each dashboardPane already has its own 'closable' attribute. As for resizable and draggable, the underlying javascript library (gridster.js) doesn't offer that possibility. Either resizing is enabled for all panes or not at all and either dragging is enabled for all panes or not at all. One particular situation is that, by dragging or resizing a pane, it's possible to displace other panes that are on the way.

        Show
        Arturo Zambrano added a comment - Regarding the first bullet point, I don't see that issue on Chrome, FF, and Edge on Windows 10. The whole dashboard fits perfectly with enough white space on both sides. Did you mean make it more centered? Regarding the third bullet point, each dashboardPane already has its own 'closable' attribute. As for resizable and draggable, the underlying javascript library (gridster.js) doesn't offer that possibility. Either resizing is enabled for all panes or not at all and either dragging is enabled for all panes or not at all. One particular situation is that, by dragging or resizing a pane, it's possible to displace other panes that are on the way.
        Hide
        Arturo Zambrano added a comment - - edited

        r52457: demo modifications for correctness and to avoid issues; created all new panes with the same default minumum size of 1, in order to make correct size and position calculations and this avoid the issue of the dashboard becoming unusable due to javascript errors; also added logic to avoid adding more panes than the maximum number of rows supported by gridster.js (15 rows)

        r52458: added code to assume a minimum size of 1, if somehow the x and y sizes are less than 1, in order to make correct size and position calculations

        With these fixes the second bullet point is solved.

        Show
        Arturo Zambrano added a comment - - edited r52457: demo modifications for correctness and to avoid issues; created all new panes with the same default minumum size of 1, in order to make correct size and position calculations and this avoid the issue of the dashboard becoming unusable due to javascript errors; also added logic to avoid adding more panes than the maximum number of rows supported by gridster.js (15 rows) r52458: added code to assume a minimum size of 1, if somehow the x and y sizes are less than 1, in order to make correct size and position calculations With these fixes the second bullet point is solved.
        Hide
        Arturo Zambrano added a comment -

        Added wiki pages for ace:dashboard and ace:dashboardPane, including some important observations for their use.

        Show
        Arturo Zambrano added a comment - Added wiki pages for ace:dashboard and ace:dashboardPane, including some important observations for their use.

          People

          • Assignee:
            Arturo Zambrano
            Reporter:
            Leo Milhouse
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: