562545 - Shift-reload keeps the scroll position stored

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Martijn Wargers (dead)]

To reproduce:
- Visit http://jquery.malsup.com/form/#ajaxForm
- Scroll to the bottom of the page
- Hit ctrl-shift-R

Expected result:
A complete reload, browser cache is bypassed, the scroll position is not restored.

Actual result:
A complete reload, browser cache is bypassed, but the scroll position is restored.

Comment 1

[Martijn Wargers (dead)]

And it makes me wonder whether the zoom level should be remembered on a shift-reload.

Comment 2

[Martijn Wargers (dead)]

Note to self: I added test for this in https://litmus.mozilla.org/show_test.cgi?id=9802

Comment 3

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

(In reply to comment #1)
> And it makes me wonder whether the zoom level should be remembered on a
> shift-reload.

It looks like firefox keep it.

Comment 4

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

Attachment: Patch

Low risk patch that affect a not very common use functionality.

Comment 5

[Martijn Wargers (dead)]

(In reply to comment #3)
> (In reply to comment #1)
> > And it makes me wonder whether the zoom level should be remembered on a
> > shift-reload.
> 
> It looks like firefox keep it.

Yes, but in firefox, this is part of the site specific preferences. If you zoom in on a page, close Firefox, restart, then the zoom in is also remembered. The use case is for sites that consistently have small font sizes on their site, that some people have trouble reading.
I think Fennec has a different kind of usage pattern, so you might want a different behavior there. Also zoom is not restored on back-forward in history, btw.

Anyway, perhaps a new bug should be filed for this discussion?

Comment 6

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

(In reply to comment #5)
> (In reply to comment #3)
> > (In reply to comment #1)
> > > And it makes me wonder whether the zoom level should be remembered on a
> > > shift-reload.
> > 
> > It looks like firefox keep it.
> 
> Yes, but in firefox, this is part of the site specific preferences. If you zoom
> in on a page, close Firefox, restart, then the zoom in is also remembered. The
> use case is for sites that consistently have small font sizes on their site,
> that some people have trouble reading.
> I think Fennec has a different kind of usage pattern, so you might want a
> different behavior there. Also zoom is not restored on back-forward in history,
> btw.

I'm agree that the use case is different, as an example because the screen is small you're _force_ to zoom to read the text of a website.

You're probably right after all, but I would like to have Madhava's feedback on this behavior (though Ctrl+shift+R is _not_ a very large audience)

> 
> Anyway, perhaps a new bug should be filed for this discussion?

There is a start of a discussion about per domain zoom in bug 559838, but I won't mind implementing the correct behavior (in term of zoom) for Ctrl + shift + R in this bug.

Comment 7

[Madhava Enros [:madhava]]

I think that Shift-reload is a "completely reload the page / reset" function, so not going back to a new state is what we want.  In other words, not remembering zoom level or position.

Comment 8

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

Attachment: Patch

Comment 9

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

Obviously the last patch address the zoom/scroll/everything case.

Comment 10

[Mark Finkle (:mfinkle) (use needinfo?)]

Nice clean patch

pushed m-b:
http://hg.mozilla.org/mobile-browser/rev/b0f738eaff02

shift-reload is not on m-1.1, so I am not pushing there.

Comment 11

[Aakash Desai [:aakashd]]

verified FIXED on build:
Mozilla/5.0 (X11; U; Linux armv7l; Nokia N900; en-US; rv:1.9.2.5pre) Gecko/20100503 Namoroka/3.6.5pre Fennec/1.1b2pre