ICEfaces
  1. ICEfaces
  2. ICE-1969

Convert Webmail Project to Facelets Templating

    Details

    • Type: New Feature New Feature
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: Sample Apps
    • Labels:
      None
    • Environment:
      ICEfaces 1.6 and Facelets

      Description

      The Webmail application predates the advent of Facelets. The goal of this bug is to convert the webmail to use Facelets templates and more importantly take advantage of dynamic includes. Webmail currently has a very large component tree as the result of panel stack which will be reduced by using dynamic includes.

        Activity

        Hide
        Patrick Corless added a comment -

        There's a lot to this one. Let me know if you can add it to your plate.

        Show
        Patrick Corless added a comment - There's a lot to this one. Let me know if you can add it to your plate.
        Hide
        Brad Kroeger added a comment -

        Added facelets to webmail at ossrepo\projects\webmail\trunk\webmail.

        Revision 15036
        emailEditViewPanel.jspx - had to remove header facet tags, created ICE-2263 for this issue.
        webmailSplashPanel.jspx - had to remove f:verbatim tags, created ICE-2262 for this issue.
        faces-config.xml - removed <from-view-id> entry as per ICE-1957.
        webmail.jspx - All of the jsp:directive.include references have been replaced with ui:include.

        Next step is to add templating.

        Show
        Brad Kroeger added a comment - Added facelets to webmail at ossrepo\projects\webmail\trunk\webmail. Revision 15036 emailEditViewPanel.jspx - had to remove header facet tags, created ICE-2263 for this issue. webmailSplashPanel.jspx - had to remove f:verbatim tags, created ICE-2262 for this issue. faces-config.xml - removed <from-view-id> entry as per ICE-1957 . webmail.jspx - All of the jsp:directive.include references have been replaced with ui:include. Next step is to add templating.
        Hide
        Brad Kroeger added a comment -

        ui:include now dynamic.

        Revision 15052

        Show
        Brad Kroeger added a comment - ui:include now dynamic. Revision 15052
        Hide
        Brad Kroeger added a comment -

        Revision #15078

        Addition of template for main webmail.jspx page.

        Removal of html tables from the page, regions organized by DIV's instead.

        Show
        Brad Kroeger added a comment - Revision #15078 Addition of template for main webmail.jspx page. Removal of html tables from the page, regions organized by DIV's instead.
        Hide
        Brad Kroeger added a comment -

        Patrick, can you take a look and let me know if this issue is fixed to your satisfaction. Facelets has been added and the webmail.jspx page has been templated.

        Show
        Brad Kroeger added a comment - Patrick, can you take a look and let me know if this issue is fixed to your satisfaction. Facelets has been added and the webmail.jspx page has been templated.

          People

          • Assignee:
            Unassigned
            Reporter:
            Patrick Corless
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: