ICEmobile
  1. ICEmobile
  2. MOBI-307

JSP Resource Loading and Consolidation Strategy

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.1 Final
    • Fix Version/s: 1.2 Beta
    • Component/s: Spring
    • Labels:
      None
    • Environment:
      jsp

      Description

      The resource loading strategy for JSP tags could be improved. Currently, if not using Spring MVC resource mapping the developer must manually include the needed CSS, JavaScript and image resources in the application project structure.

      The location of the various tag resources also currently lacks a standard structure. For instance, the iscroll.js resource is located under the JSF component resources, and in the ICEmobile Spring MVC demo. It would be better if there were only one location for this resource.

      We may also consider a resource loading utility servlet that could be used in straight JSP (non-Spring MVC) environments, so that users are not forced to manually include the resources in their project structure.

      It's also a bit odd that the folder name for the skins is 'org.icefaces.component.skins'. We may want to remove the ICEfaces reference here. I think 'skins' would suffice.

        Issue Links

          Activity

          Hide
          Philip Breau added a comment -

          Revision: 30524
          Author: philip.breau
          Date: August-16-12 1:59:24 PM
          Message:
          MOBI-307 JSP Resource Loading and Consolidation Strategy

          • add new resource servlet for jsp resource loading

            Added : /icemobile/trunk/icemobile/jsp/src/main/java/org/icemobile/jsp/servlet
            Added : /icemobile/trunk/icemobile/jsp/src/main/java/org/icemobile/jsp/servlet/ResourceServlet.java
            Modified : /icemobile/trunk/icemobile/jsp/src/main/java/org/icemobile/jsp/tags/DeviceResourceTag.java
            Added : /icemobile/trunk/icemobile/jsp/src/main/java/org/icemobile/jsp/util
            Added : /icemobile/trunk/icemobile/jsp/src/main/java/org/icemobile/jsp/util/Constants.java
            Added : /icemobile/trunk/icemobile/jsp/src/main/java/org/icemobile/jsp/util/Util.java
            Modified : /icemobile/trunk/icemobile/samples/icemobilespring/src/main/webapp/WEB-INF/web.xml

          Show
          Philip Breau added a comment - Revision: 30524 Author: philip.breau Date: August-16-12 1:59:24 PM Message: MOBI-307 JSP Resource Loading and Consolidation Strategy add new resource servlet for jsp resource loading Added : /icemobile/trunk/icemobile/jsp/src/main/java/org/icemobile/jsp/servlet Added : /icemobile/trunk/icemobile/jsp/src/main/java/org/icemobile/jsp/servlet/ResourceServlet.java Modified : /icemobile/trunk/icemobile/jsp/src/main/java/org/icemobile/jsp/tags/DeviceResourceTag.java Added : /icemobile/trunk/icemobile/jsp/src/main/java/org/icemobile/jsp/util Added : /icemobile/trunk/icemobile/jsp/src/main/java/org/icemobile/jsp/util/Constants.java Added : /icemobile/trunk/icemobile/jsp/src/main/java/org/icemobile/jsp/util/Util.java Modified : /icemobile/trunk/icemobile/samples/icemobilespring/src/main/webapp/WEB-INF/web.xml
          Hide
          Philip Breau added a comment -

          Revision: 30532
          Author: philip.breau
          Date: August-16-12 4:11:18 PM
          Message:
          MOBI-307 JSP Resource Loading and Consolidation Strategy

          • fix jsp jar build to include all javascript resources
          • remove org.icemobile.resource.root paramter
          • clean up DeviceResourceTag link output
          • fix CarouselTag using a non-absolute path link (wouldn't work anywhere but the root folder)
          • remove iscroll from icemobilespring application resources
          • move icemobile.js under javascript folder

            Modified : /icemobile/trunk/icemobile/jsp/build.xml
            Modified : /icemobile/trunk/icemobile/jsp/src/main/java/org/icemobile/jsp/servlet/ResourceServlet.java
            Modified : /icemobile/trunk/icemobile/jsp/src/main/java/org/icemobile/jsp/tags/CarouselTag.java
            Modified : /icemobile/trunk/icemobile/jsp/src/main/java/org/icemobile/jsp/tags/DeviceResourceTag.java
            Modified : /icemobile/trunk/icemobile/jsp/src/main/java/org/icemobile/jsp/util/Constants.java
            Modified : /icemobile/trunk/icemobile/jsp/src/main/java/org/icemobile/jsp/util/Util.java
            Deleted : /icemobile/trunk/icemobile/jsp/src/main/resources/META-INF/web-resources/icemobile.js
            Added : /icemobile/trunk/icemobile/jsp/src/main/resources/META-INF/web-resources/javascript
            Added : /icemobile/trunk/icemobile/jsp/src/main/resources/META-INF/web-resources/javascript/icemobile.js
            Added : /icemobile/trunk/icemobile/jsp/src/main/resources/META-INF/web-resources/javascript/iscroll.js
            Added : /icemobile/trunk/icemobile/resources/javascript
            Added : /icemobile/trunk/icemobile/resources/javascript/iscroll.js
            Modified : /icemobile/trunk/icemobile/samples/icemobilespring/src/main/webapp/WEB-INF/web.xml
            Deleted : /icemobile/trunk/icemobile/samples/icemobilespring/src/main/webapp/resources/org.icefaces.component.util

          Show
          Philip Breau added a comment - Revision: 30532 Author: philip.breau Date: August-16-12 4:11:18 PM Message: MOBI-307 JSP Resource Loading and Consolidation Strategy fix jsp jar build to include all javascript resources remove org.icemobile.resource.root paramter clean up DeviceResourceTag link output fix CarouselTag using a non-absolute path link (wouldn't work anywhere but the root folder) remove iscroll from icemobilespring application resources move icemobile.js under javascript folder Modified : /icemobile/trunk/icemobile/jsp/build.xml Modified : /icemobile/trunk/icemobile/jsp/src/main/java/org/icemobile/jsp/servlet/ResourceServlet.java Modified : /icemobile/trunk/icemobile/jsp/src/main/java/org/icemobile/jsp/tags/CarouselTag.java Modified : /icemobile/trunk/icemobile/jsp/src/main/java/org/icemobile/jsp/tags/DeviceResourceTag.java Modified : /icemobile/trunk/icemobile/jsp/src/main/java/org/icemobile/jsp/util/Constants.java Modified : /icemobile/trunk/icemobile/jsp/src/main/java/org/icemobile/jsp/util/Util.java Deleted : /icemobile/trunk/icemobile/jsp/src/main/resources/META-INF/web-resources/icemobile.js Added : /icemobile/trunk/icemobile/jsp/src/main/resources/META-INF/web-resources/javascript Added : /icemobile/trunk/icemobile/jsp/src/main/resources/META-INF/web-resources/javascript/icemobile.js Added : /icemobile/trunk/icemobile/jsp/src/main/resources/META-INF/web-resources/javascript/iscroll.js Added : /icemobile/trunk/icemobile/resources/javascript Added : /icemobile/trunk/icemobile/resources/javascript/iscroll.js Modified : /icemobile/trunk/icemobile/samples/icemobilespring/src/main/webapp/WEB-INF/web.xml Deleted : /icemobile/trunk/icemobile/samples/icemobilespring/src/main/webapp/resources/org.icefaces.component.util
          Hide
          Philip Breau added a comment -

          Revision: 30533
          Author: philip.breau
          Date: August-16-12 4:55:27 PM
          Message:
          MOBI-307 JSP Resource Loading and Consolidation Strategy

          • have jsf build use the shared iscroll.js file

            Modified : /icemobile/trunk/icemobile/jsf/components/component/build.xml
            Deleted : /icemobile/trunk/icemobile/jsf/components/component/resources/org.icefaces.component.util/iscroll-lite.js
            Deleted : /icemobile/trunk/icemobile/jsf/components/component/resources/org.icefaces.component.util/iscroll.js

          Show
          Philip Breau added a comment - Revision: 30533 Author: philip.breau Date: August-16-12 4:55:27 PM Message: MOBI-307 JSP Resource Loading and Consolidation Strategy have jsf build use the shared iscroll.js file Modified : /icemobile/trunk/icemobile/jsf/components/component/build.xml Deleted : /icemobile/trunk/icemobile/jsf/components/component/resources/org.icefaces.component.util/iscroll-lite.js Deleted : /icemobile/trunk/icemobile/jsf/components/component/resources/org.icefaces.component.util/iscroll.js

            People

            • Assignee:
              Philip Breau
              Reporter:
              Philip Breau
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: