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

              depends on

              Improvement - An improvement or enhancement to an existing feature or task. ICE-7293 Add ability to avoid having ice:inputRichText JS loaded on every page when not using it.

              • Major - Major loss of function.
              • Closed

              Improvement - An improvement or enhancement to an existing feature or task. ICE-7687 Add ability to avoid having the ice:gmap JavaScript load on every page when not using it.

              • Major - Major loss of function.
              • Closed

                Activity

                Ascending order - Click to sort in descending order
                Hide
                Permalink
                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
                Permalink
                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: