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

        Patrick Corless created issue -
        Patrick Corless made changes -
        Field Original Value New Value
        Link This issue blocks ICE-1966 [ ICE-1966 ]
        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.
        Patrick Corless made changes -
        Assignee Brad Kroeger [ brad.kroeger ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #15036 Thu Oct 25 17:29:39 MDT 2007 brad.kroeger ICE-1969 Added facelets to webmail application.

        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.
        Files Changed
        Commit graph MODIFY /projects/webmail/trunk/webmail/web/webmail.jspx
        Commit graph MODIFY /projects/webmail/trunk/webmail/web/WEB-INF/faces-config.xml
        Commit graph MODIFY /projects/webmail/trunk/webmail/web/WEB-INF/web.xml
        Commit graph MODIFY /projects/webmail/trunk/webmail/web/inc/contentPanels/emailEditViewPanel.jspx
        Commit graph MODIFY /projects/webmail/trunk/webmail/build.xml
        Commit graph MODIFY /projects/webmail/trunk/webmail/web/inc/contentPanels/webmailSplashPanel.jspx
        Commit graph MODIFY /projects/webmail/trunk/webmail/web/login.jspx
        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.
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #15052 Wed Oct 31 10:27:16 MDT 2007 brad.kroeger ICE-1969 ui:include now dynamic.
        Files Changed
        Commit graph MODIFY /projects/webmail/trunk/webmail/web/webmail.jspx
        Commit graph MODIFY /projects/webmail/trunk/webmail/src/com/icesoft/applications/faces/webmail/navigation/NavigationTree.java
        Commit graph MODIFY /projects/webmail/trunk/webmail/src/com/icesoft/applications/faces/webmail/contacts/ContactManager.java
        Commit graph MODIFY /projects/webmail/trunk/webmail/src/com/icesoft/applications/faces/webmail/tasks/TasksManager.java
        Commit graph MODIFY /projects/webmail/trunk/webmail/src/com/icesoft/applications/faces/webmail/mail/MailManager.java
        Hide
        Brad Kroeger added a comment -

        ui:include now dynamic.

        Revision 15052

        Show
        Brad Kroeger added a comment - ui:include now dynamic. Revision 15052
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #15078 Thu Nov 01 16:30:56 MDT 2007 brad.kroeger ICE-1969 Addition of template for main webmail.jspx page.

        Removal of html tables from the page, regions organized by DIV's instead.
        Files Changed
        Commit graph MODIFY /projects/webmail/trunk/webmail/web/webmail.jspx
        Commit graph MODIFY /projects/webmail/trunk/webmail/web/inc/contentPanels/emailEditViewPanel.jspx
        Commit graph MODIFY /projects/webmail/trunk/webmail/web/css/styles.css
        Commit graph ADD /projects/webmail/trunk/webmail/web/webmail-template.jspx
        Commit graph MODIFY /projects/webmail/trunk/webmail/web/inc/navigation.jspx
        Commit graph MODIFY /projects/webmail/trunk/webmail/web/inc/header.jspx
        Commit graph MODIFY /projects/webmail/trunk/webmail/web/inc/controls.jspx
        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.
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #15091 Mon Nov 05 11:39:43 MST 2007 brad.kroeger ICE-1969 Change width to 985px.
        Files Changed
        Commit graph MODIFY /projects/webmail/trunk/webmail/web/css/styles.css
        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.
        Brad Kroeger made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Assignee Brad Kroeger [ brad.kroeger ]

          People

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

            Dates

            • Created:
              Updated:
              Resolved: