ICEfaces
  1. ICEfaces
  2. ICE-8166

ICEfaces Showcase sample app. - Remove legacy use of NavigationController.refreshPage()

    Details

    • Type: Task Task
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 3.1.0.BETA1
    • Fix Version/s: 3.1.0.RC1, 3.1
    • Component/s: Sample Apps
    • Labels:
      None
    • Environment:
      ICEfaces 3.1
    • Assignee Priority:
      P1
    • Affects:
      Sample App./Tutorial

      Description

      Currently, the ICEfaces Showcase sample application uses a technique to force a refresh/load of the page for certain component demos that historically were unable to dynamically load their resources, such as ice:gmap and ice:inputRichText.

      With the modifications to have these ice components implement the mandatoryResource configuration scheme in ICE-7687 and ICE-7293 it should no longer be necessary to force a reload of the demo pages with these components on them, thus, the entire NavigationController.refreshPage() method should now be removed from the showcase application.

        Issue Links

          Activity

          Ken Fyten created issue -
          Ken Fyten made changes -
          Field Original Value New Value
          Salesforce Case []
          Fix Version/s 3.1 [ 10312 ]
          Affects [Sample App./Tutorial, Compatibility/Configuration]
          Assignee Ken Fyten [ ken.fyten ]
          Ken Fyten made changes -
          Link This issue depends on ICE-7687 [ ICE-7687 ]
          Ken Fyten made changes -
          Link This issue depends on ICE-7293 [ ICE-7293 ]
          Ken Fyten made changes -
          Salesforce Case []
          Ken Fyten made changes -
          Salesforce Case []
          Fix Version/s 3.1.0.RC1 [ 10337 ]
          Ken Fyten made changes -
          Salesforce Case []
          Assignee Priority P1
          Assignee Ken Fyten [ ken.fyten ] Carlo Guglielmin [ carlo.guglielmin ]
          Ken Fyten made changes -
          Fix Version/s 3.1.0.RC1 [ 10337 ]
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #29755 Thu Jun 28 14:49:34 MDT 2012 carlo.guglielmin ICE-8166 - Removed any possible usage of NavigationController.refreshPage, which ended up being 3 instances in the ice:gMap component because their resource loading was improved.
          Files Changed
          Commit graph MODIFY /icefaces3/trunk/icefaces/samples/showcase/showcase/src/main/java/org/icefaces/samples/showcase/example/compat/map/MapZoom.java
          Commit graph MODIFY /icefaces3/trunk/icefaces/samples/showcase/showcase/src/main/webapp/resources/examples/compat/map/mapType.xhtml
          Commit graph MODIFY /icefaces3/trunk/icefaces/samples/showcase/showcase/src/main/webapp/resources/examples/compat/map/mapZoom.xhtml
          Commit graph MODIFY /icefaces3/trunk/icefaces/samples/showcase/showcase/src/main/webapp/resources/examples/compat/map/mapLoad.xhtml
          Commit graph MODIFY /icefaces3/trunk/icefaces/samples/showcase/showcase/src/main/java/org/icefaces/samples/showcase/example/compat/map/MapType.java
          Commit graph MODIFY /icefaces3/trunk/icefaces/samples/showcase/showcase/src/main/java/org/icefaces/samples/showcase/example/compat/map/MapLoad.java
          Hide
          Carlo Guglielmin added a comment -

          NavigationController.refreshPage is still needed in some cases.

          ice:selectInputDate -> Converter Pattern: required because JSF can't switch the f:convertDateTime pattern without a full page refresh
          ice:selectInputDate -> Timezones: required for the same reason as f:convertDateTime can't dynamically switch the timeZone attribute
          ice:dataTable -> Column Widths: the string defining the widths doesn't apply dynamically, it requires a page refresh

          The other instance was in Rich Text Editor -> Disabled, but that demo has been removed a while back because of other issues.

          In terms of the Google Map issues I've fixed their usage of refreshPage so they dynamically update as expected, as of r29755. This applies to Map Zoom, Load XML, and Type.

          Show
          Carlo Guglielmin added a comment - NavigationController.refreshPage is still needed in some cases. ice:selectInputDate -> Converter Pattern: required because JSF can't switch the f:convertDateTime pattern without a full page refresh ice:selectInputDate -> Timezones: required for the same reason as f:convertDateTime can't dynamically switch the timeZone attribute ice:dataTable -> Column Widths: the string defining the widths doesn't apply dynamically, it requires a page refresh The other instance was in Rich Text Editor -> Disabled, but that demo has been removed a while back because of other issues. In terms of the Google Map issues I've fixed their usage of refreshPage so they dynamically update as expected, as of r29755. This applies to Map Zoom, Load XML, and Type.
          Hide
          Carlo Guglielmin added a comment -

          r29755 - See last comment for details.

          Show
          Carlo Guglielmin added a comment - r29755 - See last comment for details.
          Carlo Guglielmin made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Fix Version/s 3.1.0.RC1 [ 10337 ]
          Fix Version/s 3.1 [ 10312 ]
          Affects [Sample App./Tutorial, Compatibility/Configuration] [Sample App./Tutorial]
          Resolution Fixed [ 1 ]
          Ken Fyten made changes -
          Fix Version/s 3.1 [ 10312 ]
          Ken Fyten made changes -
          Status Resolved [ 5 ] Closed [ 6 ]

            People

            • Assignee:
              Carlo Guglielmin
              Reporter:
              Ken Fyten
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: