If panned prior to discoverability demo start, a grey area is visible behind the visible sidebar

VERIFIED FIXED in Firefox 6

Status

Fennec Graveyard
General
VERIFIED FIXED
7 years ago
7 years ago

People

(Reporter: aaronmt, Assigned: mbrubeck)

Tracking

({polish})

Trunk
Firefox 6
polish

Details

Attachments

(2 attachments)

(Reporter)

Description

7 years ago
Created attachment 531056 [details]
Screenshot

Device: Nexus One

Mozilla/5.0 (Android; Linux armv7l; rv:6.0a1) Gecko/20110509; Firefox/6.0a1 Fennec/6.0a1

If a user pans to expose either left or right sidebars prior to the start of the demo animation on about:firstrun, there will be an exposed grey area to either 
the left or right of the associated sidebar.

See attached screenshot.

IRC: mbrubeck mentions -- 'we need to reset the scroll position before starting the demo'
(Reporter)

Updated

7 years ago
Summary: If panned prior to demo start grey area is visible behind sidebars → If panned prior to discoverability demo start, a grey area is visible behind the visible sidebar
(Assignee)

Updated

7 years ago
Assignee: nobody → mbrubeck
Status: NEW → ASSIGNED
Keywords: polish
OS: Android → All
Hardware: ARM → All
(Assignee)

Comment 1

7 years ago
Created attachment 531063 [details] [diff] [review]
patch

This patch just skips the demo if a sidebar is already open when about:firstrun loads.

(As mentioned above, we could also fix this by resetting the scroll position before running the demo.)
Attachment #531063 - Flags: review?(wjohnston)
Comment on attachment 531063 [details] [diff] [review]
patch

I'm fine with this. I think it will be fairly odd that users decide to intentionally visit about:firstrun anyway.
Attachment #531063 - Flags: review?(wjohnston) → review+
(Assignee)

Comment 3

7 years ago
http://hg.mozilla.org/mozilla-central/rev/508c54f63824
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
(Assignee)

Updated

7 years ago
Target Milestone: --- → Firefox 6

Comment 4

7 years ago
Verified fixed on build: Mozilla /5.0 (Android;Linux armv7l;rv:6.0a1) Gecko/20110511 Firefox/6.0a1 Fennec/6.0a1 
Device: LG Optimus 2X (Android 2.2)
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.