Details
Description
When selecting an example on the iPad or desktop browsers, change the theme to Android and the scrolling no longer works on the menu panel.
Activity
Judy Guglielmin
created issue -
Judy Guglielmin
made changes -
Field | Original Value | New Value |
---|---|---|
Fix Version/s | 1.1 Final [ 10343 ] | |
Fix Version/s | EE 1.1 [ 10339 ] |
Steve Maryka
made changes -
Assignee | Steve Maryka [ steve.maryka ] | Patrick Corless [ patrick.corless ] |
Repository | Revision | Date | User | Message |
ICEsoft Public SVN Repository | #30379 | Thu Aug 02 13:42:53 MDT 2012 | patrick.corless | |
Files Changed | ||||
MODIFY
/icemobile/trunk/icemobile/samples/mobileshowcase/src/main/webapp/resources/javascript/showcase.js
|
Patrick Corless
made changes -
Status | Open [ 1 ] | Resolved [ 5 ] |
Resolution | Fixed [ 1 ] |
Migration
made changes -
Status | Resolved [ 5 ] | Closed [ 6 ] |
Looks like this is a side effect of MOBI-273 refactoring. The original implementation make an incorrect call:
window.addEventListener(orientationEvent, resizeElementHeight('#
{cc.id}'), false);
Where resizeElementHeight is a function. So as Philip noticed resizeElementHeight() would only get called once and not actually get registered as an event listener. The new code addressed the listener assignment by assigning the function to a variable and passing the variable into window.addEventListener().
The interesting side effect is that resizeElementHeight now only gest called on dom update or on a resize or orientation change. The function should also run on first load to make sure the panel size is correctly calculated.