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
              Arran Mccullough created issue -
              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.
              Arran Mccullough made changes -
              Field Original Value New Value
              Attachment Case13607Example.war [ 22101 ]
              Attachment Case13607Example.zip [ 22102 ]
              Ken Fyten made changes -
              Assignee Arturo Zambrano [ artzambrano ]
              Fix Version/s EE-4.1.0.GA [ 12171 ]
              Fix Version/s EE-3.3.0.GA_P04 [ 12270 ]
              Assignee Priority P1 [ 10010 ]
              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.
              Ken Fyten made changes -
              Assignee Arturo Zambrano [ artzambrano ] Ken Fyten [ ken.fyten ]
              Ken Fyten made changes -
              Assignee Ken Fyten [ ken.fyten ] Arturo Zambrano [ artzambrano ]
              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.
              Arturo Zambrano made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #48484 Thu Mar 03 11:00:55 MST 2016 art.zambrano ICE-10979 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
              Files Changed
              Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/resources/icefaces.ace/gmap/gmap.js
              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.
              Carmen Cristurean made changes -
              Resolution Fixed [ 1 ]
              Status Resolved [ 5 ] Reopened [ 4 ]
              Arturo Zambrano made changes -
              Fix Version/s EE-3.3.0.GA_P04 [ 12270 ]
              Arturo Zambrano made changes -
              Status Reopened [ 4 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Arturo Zambrano made changes -
              Resolution Fixed [ 1 ]
              Status Resolved [ 5 ] Reopened [ 4 ]
              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
              Arturo Zambrano made changes -
              Fix Version/s EE-3.3.0.GA_P04 [ 12270 ]
              Arturo Zambrano made changes -
              Comment [ Removed EE-3.3.0.GA_P04 as a fix version, since the issue is related to the use of the 'address' attribute in ace:gMapMarker, which isn't supported in 3.3. On 3.3, the page should be tested with latitude-longitude values, instead of the address attribute (e.g. latitude="40.5" longitude="-80"). ]
              Arturo Zambrano made changes -
              Status Reopened [ 4 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              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.
              Ken Fyten made changes -
              Fix Version/s EE-4.1.0.BETA [ 13072 ]
              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:
                  Arran Mccullough
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  3 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: