ICEfaces
  1. ICEfaces
  2. ICE-10594

showcase-portlet - ace:notificationPanel client-side issue

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: EE-4.0.0.GA
    • Fix Version/s: EE-4.1.0.GA, 4.2.BETA, 4.2
    • Component/s: ACE-Components, Sample Apps
    • Labels:
      None
    • Environment:
      Jenkins EE-4.0 Build# 2
      Browsers: IE11, FF34, Chrome
      Server: Liferay 6.2 w/ Tomcat 7.0.42
    • Assignee Priority:
      P2

      Description

      showcase-portlet:

      NotificationPanel
      > Client Side
      All browsers: component fails to render, and JS errors occur when clicking "Show via client" or "Hide via client" buttons:

      SCRIPT5007: Unable to get property 'show' of undefined or null reference
      File: ice01, Line: 1, Column: 58539
      or:
      SCRIPT5007: Unable to get property 'hide' of undefined or null reference
      File: ice01, Line: 1, Column: 58807

        Activity

        Carmen Cristurean created issue -
        Carmen Cristurean made changes -
        Field Original Value New Value
        Fix Version/s EE-4.0.0.GA [ 11171 ]
        Ken Fyten made changes -
        Security Private [ 10001 ]
        Ken Fyten made changes -
        Priority Major [ 3 ] Minor [ 4 ]
        Fix Version/s 4.1 [ 11375 ]
        Fix Version/s EE-4.0.0.GA [ 11171 ]
        Ken Fyten made changes -
        Fix Version/s EE-4.1.0.GA [ 12171 ]
        Fix Version/s 4.1 [ 11375 ]
        Hide
        Carmen Cristurean added a comment -

        This is still an issue on IF 4.1 trunk r.47579.
        FF41 error message:
        TypeError: ice.ace.instance(...) is null
        http://localhost:8080/web/guest/showcase
        Line 1

        Show
        Carmen Cristurean added a comment - This is still an issue on IF 4.1 trunk r.47579. FF41 error message: TypeError: ice.ace.instance(...) is null http://localhost:8080/web/guest/showcase Line 1
        Hide
        Carmen Cristurean added a comment -

        Still reproducible with ICEfaces4 trunk r48632.

        Show
        Carmen Cristurean added a comment - Still reproducible with ICEfaces4 trunk r48632.
        Hide
        Ken Fyten added a comment -

        This component would need to be constrained to be rendered within the portlet panel it belongs to and not to the top of the page. If this can't be achieved readily this component might not be appropriate for use in portlets.

        Show
        Ken Fyten added a comment - This component would need to be constrained to be rendered within the portlet panel it belongs to and not to the top of the page. If this can't be achieved readily this component might not be appropriate for use in portlets.
        Ken Fyten made changes -
        Assignee Arturo Zambrano [ artzambrano ]
        Assignee Priority P2 [ 10011 ]
        Hide
        Arturo Zambrano added a comment -

        r48866: added component bindings to obtain correct client id in custom Javascript code on the page
        r48867: added code to constrain the notification panel to the portlet it is contained in

        Show
        Arturo Zambrano added a comment - r48866: added component bindings to obtain correct client id in custom Javascript code on the page r48867: added code to constrain the notification panel to the portlet it is contained in
        Arturo Zambrano made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #48866 Tue May 31 11:38:11 MDT 2016 art.zambrano ICE-10594 added component bindings to obtain correct client id in custom Javascript code on the page
        Files Changed
        Commit graph MODIFY /icefaces4/trunk/icefaces/samples/showcase/showcase/src/main/webapp/resources/examples/ace/notificationpanel/notificationPanelClient.xhtml
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #48867 Tue May 31 11:40:01 MDT 2016 art.zambrano ICE-10594 added code to constrain the notification panel to the portlet it is contained in
        Files Changed
        Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/resources/icefaces.ace/notificationpanel/notificationpanel.js
        Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/resources/icefaces.ace/notificationpanel/notificationpanel.css
        Hide
        Liana Munroe added a comment -

        Verified ICEfaces 4 trunk r48867. Liferay-portal-6.2-ce-ga5. IE edge, 11, 10 , 9, 8, FF 43, Chrome 50.

        Show
        Liana Munroe added a comment - Verified ICEfaces 4 trunk r48867. Liferay-portal-6.2-ce-ga5. IE edge, 11, 10 , 9, 8, FF 43, Chrome 50.
        Ken Fyten made changes -
        Fix Version/s 4.2.BETA [ 13091 ]
        Fix Version/s 4.2 [ 12870 ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Arturo Zambrano
            Reporter:
            Carmen Cristurean
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: