796730 - [calendar] Scroll Bar shouldn't extend to the header

Status: RESOLVED WORKSFORME

(Firefox OS Graveyard :: Gaia::Calendar, defect, P3)

Description

[GH to BZ]

[GitHub issue by patrykdesign on 2012-09-25T16:21:15Z, https://github.com/mozilla-b2g/gaia/issues/5150]
The scroll bar is drawn on top of the header. It should stop right below it. See screenshot = http://cl.ly/image/0S3l1j300k31
This seems to be an issue in Calendar, Email potentially other apps but not Music.
@asutherland @lightsofapollo

Comment 1

[GH to BZ]

[GitHub comment by asutherland on 2012-09-25T16:26:54Z]
@steveck-chung @dominickuo We can probably fix this.  This was done initially as the easiest way to satisfy the "hide the search screen UI" under the top.  We can change the scroll region to just be below the header and have an initial scrolled offset to hide the search UI instead.  (The search UI had been disabled because it was a lie since, however.)

Comment 2

[Dominic Kuo [:dkuo]]

Attachment: Apply header and button BB for email, also fix this bug

Comment 3

[Andrew Sutherland [:asuth] (he/him)]

Landed on gaia master:
https://github.com/mozilla-b2g/gaia/commit/9496abb7ed1cdbefd88a58fcc7bef8f7cd099e1b

Comment 4

[Andrew Sutherland [:asuth] (he/him)]

Oops, just realized this was in the calendar component right now.  Well, consider the email bit fixed as of:
https://github.com/mozilla-b2g/gaia/commit/9496abb7ed1cdbefd88a58fcc7bef8f7cd099e1b

Comment 5

[Eric Pang [:epang] UX]

Dominic, has this bug been fixed for calendar?  I've checked on the 2013-03-11 gaia/master build and it seems to have been fixed cause I can't find a scroll bar that goes into the header.  But I'm not sure what I'm looking for since the original screen cap link is broken.  I'll close this bug if it's fixed. Thx!

Comment 6

[Dominic Kuo [:dkuo]]

Eric, from the bugzilla log above, you can see this bug was for both email and calendar, my patch was for email only, but I believe there is another split bug for calendar to fix this(I cannot find it), and already been fixed just like you said, so I think we can close this one, thanks.