ICEfaces
  1. ICEfaces
  2. ICE-10690

Showcase -> mobi:cloudPush demo has button state problems on iOS

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Won't Fix
    • Affects Version/s: EE-4.0.0.GA
    • Fix Version/s: EE-4.2.0.GA
    • Labels:
      None
    • Environment:
      ICEfaces 4, showcase app, mobi:cloudPush demo, iOS device
    • Assignee Priority:
      P1

      Description

      When running the cloud push demo on the showcase on an iOS device, once you receive a native cloud push notification the demo always displays the "Enable Cloud Push" button, even immediately after clicking it.

      To reproduce:

      1. Use icefaces-showcase.icesoft.org (this has native notifications working)
      2. Navigate to the Cloud Push demo on a mobile device
      3. Click EnableCloud Push button
      4. Click Disconnect Push button
      5. Click Priority Push button
      6. Wait for the native notification to arrive. When it does, click it to navigate back to the demo in the browser.
      7. See that the Enable Cloud Push button is rendered again (incorrectly).
      8. Click Enable Cloud Push button. (see that when it returns to the browser the Enable Cloud Push button is still rendered).
      9. At this point, if you touch the browser in the white-space beside the Enable Cloud Push button, the correct buttons will display (or if you close the browser tab and re-open and start from scratch)

        Issue Links

          Activity

          Hide
          Ken Fyten added a comment -

          This issue has been addressed via additional integration with new APIs introduced in PUSH-398.

          Show
          Ken Fyten added a comment - This issue has been addressed via additional integration with new APIs introduced in PUSH-398 .
          Hide
          Mircea Toma added a comment -

          Modified CloudPushRenderer to render button only when cloud push was not yet enabled. The renderer makes use of the new PushContext.hasNotifyBackURI to determine when to render the button.

          Show
          Mircea Toma added a comment - Modified CloudPushRenderer to render button only when cloud push was not yet enabled. The renderer makes use of the new PushContext.hasNotifyBackURI to determine when to render the button.
          Hide
          Mircea Toma added a comment -

          Applied fixes to 4.1.1 BETA branch.

          Show
          Mircea Toma added a comment - Applied fixes to 4.1.1 BETA branch.
          Hide
          Carmen Cristurean added a comment - - edited

          The fix applied on both EE4 trunk and EE 4.1.1.BETA tag causes the regression described above: after enabling Cloud Push, the "Priority Push" and "Simple Push" buttons fail to remain rendered. This occurs on desktop and mobile devices.
          Also, starting at this code revision, the "Enable Cloud Push" button, when rendered, is left-aligned (desktop browsers only). Screen shot attached.

          Showcase from EE4 trunk r48837 and EE-4.1.1.BETA tag r48837 are deployed on Labs1, and this issue can be reproduced on them:
          http://54.245.237.240/tomcat-1/showcase/showcase-mobile.jsf
          http://54.245.237.240/tomcat-1/showcase-B/showcase-mobile.jsf

          Show
          Carmen Cristurean added a comment - - edited The fix applied on both EE4 trunk and EE 4.1.1.BETA tag causes the regression described above: after enabling Cloud Push, the "Priority Push" and "Simple Push" buttons fail to remain rendered. This occurs on desktop and mobile devices. Also, starting at this code revision, the "Enable Cloud Push" button, when rendered, is left-aligned (desktop browsers only). Screen shot attached. Showcase from EE4 trunk r48837 and EE-4.1.1.BETA tag r48837 are deployed on Labs1, and this issue can be reproduced on them: http://54.245.237.240/tomcat-1/showcase/showcase-mobile.jsf http://54.245.237.240/tomcat-1/showcase-B/showcase-mobile.jsf
          Hide
          Mircea Toma added a comment -

          Make "Simple Push" and "Priority Push" buttons always visible. Add CSS style to mobi:cloudPush button.

          Show
          Mircea Toma added a comment - Make "Simple Push" and "Priority Push" buttons always visible. Add CSS style to mobi:cloudPush button.
          Hide
          Ken Fyten added a comment -

          Re-opened to review cloud push showcase demo in light of new Cloud Push demo app.

          Show
          Ken Fyten added a comment - Re-opened to review cloud push showcase demo in light of new Cloud Push demo app.
          Hide
          Liana Munroe added a comment -

          ICEfaces EE-4.1.0.GA Jenkins build 2 - Verified roll back for "Modified CloudPushRenderer to render button only when cloud push was not yet enabled." ios ipad 9.3

          Show
          Liana Munroe added a comment - ICEfaces EE-4.1.0.GA Jenkins build 2 - Verified roll back for "Modified CloudPushRenderer to render button only when cloud push was not yet enabled." ios ipad 9.3
          Hide
          Ken Fyten added a comment -

          Closing as Won't Fix as Cloud Push support is not supported.

          Show
          Ken Fyten added a comment - Closing as Won't Fix as Cloud Push support is not supported.

            People

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

              Dates

              • Created:
                Updated:
                Resolved: