app-UI icon indicating copy to clipboard operation
app-UI copied to clipboard

ViewNavigators Don't maintain scroll position

Open triceam opened this issue 12 years ago • 4 comments

ViewNavigators don't maintain scroll position when navigating back using popView(), they always reset back at the top.

triceam avatar Mar 29 '12 01:03 triceam

Any updates on this bug? It still exists in the latest version

gkcgautam avatar Mar 29 '13 15:03 gkcgautam

I'll have to look into it... I fixed this a while ago. It was either not copied back into this repo, or it is commented out. I'm away from my computer today, but can look into it on Monday.

Sent from my iPhone

On Mar 29, 2013, at 11:30 AM, gkcgautam [email protected] wrote:

Any updates on this bug? It still exists in the latest version

— Reply to this email directly or view it on GitHub.

triceam avatar Mar 29 '13 18:03 triceam

Cool. I will also look into the code if I can find the fix. BTW, Is there any documentation yet? I've been digging the code to learn more, but existing docs would be much helpful.

On 3/30/13, Andrew Trice [email protected] wrote:

I'll have to look into it... I fixed this a while ago. It was either not copied back into this repo, or it is commented out. I'm away from my computer today, but can look into it on Monday.

Sent from my iPhone

On Mar 29, 2013, at 11:30 AM, gkcgautam [email protected] wrote:

Any updates on this bug? It still exists in the latest version

— Reply to this email directly or view it on GitHub.


Reply to this email directly or view it on GitHub: https://github.com/triceam/app-UI/issues/6#issuecomment-15654126

Regards, Gautam Chaudhary http://gautamchaudhary.com

gkcgautam avatar Mar 30 '13 04:03 gkcgautam

scrollY attribute value is set in this code: (Line 167) children.attr( 'scrollY', scrollY );

But this code always returns undefined: (Line 315) var scrollY= targetDiv.attr( 'scrollY' );

I have checked value in console and the first code gets the correct value, but 'targetDiv' has no 'scrollY' attribute.

gkcgautam avatar Mar 30 '13 07:03 gkcgautam