dmachi / dojox_application

Dojox Application Framework for mobile, tablets, and desktops
Other
27 stars 24 forks source link

Problem with a scrollable view where it scrolls down a little when transitioning to a view which was scrolled to the bottom. #162

Closed edchat closed 11 years ago

edchat commented 11 years ago

There is a problem with scrollable view (app/widgets/Container with data-dojo-props="scrollable:true") where it scrolls down a little when transitioning to a view which was scrolled to the bottom. To see the problem hit the scrollableTestApp with: dojox/app/tests/scrollableTestApp/#Scrollable1 From Scrollable List 1 click the +10, scroll to the bottom of the list, and hit the Scrollable List 1 selection again. The view will scroll down a little so that the last row is partly hidden.

There is a second problem with Scrollable List 4, which uses a dojox/mobile/ScrollableView instead of an app/widgets/Container with data-dojo-props="scrollable:true". To see this problem go to the Scrollable List 4, and then refresh your browser, you will see that the footer is not all the way to the bottom of the screen. If you are on a browser and resize the window it will position the footer correctly.

edchat commented 11 years ago

My testcase did not match the one from the App Center where the problem was first seen, it is using nested views with a Footer, then a Header, then the content of the main scrollable view. In this case the footer is not being found during the calls in app/widgets/_ScrollableMixin from findAppBars because it is looking for children of body and siblings of the scrollable container, but the footer is not found there, it is off of a parent. So it seems we need a better way to find the header and footer that will work with nested views. I have updated the app/tests/longListTestApp (in my master branch) to recreate the problem.

edchat commented 11 years ago

This problem continues, I have code in my footer branch which allows the app to specify a fixedFooter on the app/widgets/Container (with scrollable: true) to get the scroll position to work correctly. https://github.com/edchat/dojox_application/tree/footer

edchat commented 11 years ago

It is fixed now.