ICEfaces
  1. ICEfaces
  2. ICE-10979

ace:gMap - JavaScript error thrown when setting a certain height

          Details

            Description

            If an ace:gMap component is set with a height higher than 568px, a script error is thrown which is preventing markers from being set on the map:

            Error: Uncaught TypeError: Cannot read property 'addEventListener' of undefined

            Sample code: <ace:gMap style="width: 675px; height: 570px;" ...>

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              Arran Mccullough added a comment -

              Attached test case that shows this issue.

              Steps:

              • Load welcomeICEfaces.jsf
              • The errors are thrown on load of the page and seen in the browser dev console.
              Show
              Arran Mccullough added a comment - Attached test case that shows this issue. Steps: Load welcomeICEfaces.jsf The errors are thrown on load of the page and seen in the browser dev console.
              Hide
              Permalink
              Arran Mccullough added a comment -

              I've also tested with a few different GMap versions but the error is thrown no matter what.

              Show
              Arran Mccullough added a comment - I've also tested with a few different GMap versions but the error is thrown no matter what.
              Hide
              Permalink
              Arturo Zambrano added a comment -

              r48484: modified the approach to initialize address-based markers by first creating an instance of a Marker object that is initially hidden and then its position is updated in the geocoding request callback and is made visible, instead of completely initializing the Marker object from within the geocoding request callback; also modified the infowindows callbacks to be the entire initialization functions instead of just the functions for attaching the windows to markers.

              Show
              Arturo Zambrano added a comment - r48484: modified the approach to initialize address-based markers by first creating an instance of a Marker object that is initially hidden and then its position is updated in the geocoding request callback and is made visible, instead of completely initializing the Marker object from within the geocoding request callback; also modified the infowindows callbacks to be the entire initialization functions instead of just the functions for attaching the windows to markers.
              Hide
              Permalink
              Carmen Cristurean added a comment -

              Verified ICEfaces4 trunk r 48485 on IE11/IE8, Chrome48, FF41 using attached test case.
              Verified showcase > gMap demos on Firefox41.

              The original issue still exists on EE-3.3.0-maintenance branch r48485.

              Show
              Carmen Cristurean added a comment - Verified ICEfaces4 trunk r 48485 on IE11/IE8, Chrome48, FF41 using attached test case. Verified showcase > gMap demos on Firefox41. The original issue still exists on EE-3.3.0-maintenance branch r48485.
              Hide
              Permalink
              Carmen Cristurean added a comment -

              Reopening to port fix to EE-3.3.0-maintenance branch.

              Show
              Carmen Cristurean added a comment - Reopening to port fix to EE-3.3.0-maintenance branch.
              Hide
              Permalink
              Arturo Zambrano added a comment - - edited

              r48489: committed fix to the 3.3 EE maintenance branch

              Show
              Arturo Zambrano added a comment - - edited r48489: committed fix to the 3.3 EE maintenance branch
              Hide
              Permalink
              Carmen Cristurean added a comment -

              Verified EE-3.3.0-maintenance branch r48489 on IE11/IE8, Chrome48, FF41 using attached test case, and showcase > gMap demos on FF41.

              Show
              Carmen Cristurean added a comment - Verified EE-3.3.0-maintenance branch r48489 on IE11/IE8, Chrome48, FF41 using attached test case, and showcase > gMap demos on FF41.

                People

                • Assignee:
                  Arturo Zambrano
                  Reporter:
                  Arran Mccullough
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  3 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: