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

          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.

            People

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

              Dates

              • Created:
                Updated:
                Resolved: