(vertical-homescreen) Statusbar transition not correct when scrolling

VERIFIED FIXED in Firefox OS v2.0

Status

Firefox OS
Gaia::Homescreen
VERIFIED FIXED
3 years ago
3 years ago

People

(Reporter: Peter La, Assigned: kgrandon)

Tracking

unspecified
2.0 S4 (20june)
x86
Mac OS X

Firefox Tracking Flags

(feature-b2g:-, b2g-v2.0 verified, b2g-v2.1 verified)

Details

(Whiteboard: ux-tracking, visual design, visual-tracking, [ft:systemsfe][systemsfe])

Attachments

(4 attachments)

(Reporter)

Description

3 years ago
The status bar transition isn't correct.

Current behaviour:
The way the status bar transitions from 15% opacity to 100% opacity is currently to TRANSLATE a new rectangle that is 100% opacity downwards to cover up the old status bar.

Expected behaviour:
The status bar rectangular region should just FADE from 15% opacity to 100% opacity.  No translation.  It is not moving, it is fading.

Please see following attachments that illustrate the difference.

Note, there was a previous bug filed for status bar transitioning to 100% opacity: https://bugzilla.mozilla.org/show_bug.cgi?id=1020755
Opening new bug to deal with the specific WAY in which the transition occurs.
(Reporter)

Comment 1

3 years ago
Created attachment 8440093 [details]
Vertical Scroll.mp4

This is the intended transition.  A fade from 15% to 100% opacity of the status bar on scroll.
(Reporter)

Comment 2

3 years ago
Created attachment 8440094 [details]
StatusbarTransitionProblem.MOV

This is the actual implementation.  The implementation translates a new rectangle that is 100% opacity on top of the 15% opacity statusbar.  This is incorrect.
(Assignee)

Comment 3

3 years ago
Created attachment 8440112 [details] [review]
Github pull request
Assignee: nobody → kgrandon
Status: NEW → ASSIGNED
(Assignee)

Comment 4

3 years ago
Comment on attachment 8440112 [details] [review]
Github pull request

This is a super trivial CSS patch and I reviewed the original patch, going to go with R=me. Hope that is ok :)
Attachment #8440112 - Flags: review+
(Assignee)

Comment 5

3 years ago
Landed: https://github.com/mozilla-b2g/gaia/commit/b439206d688dae365dc116f2ed730f6ff7a60649
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
(Assignee)

Comment 6

3 years ago
Comment on attachment 8440112 [details] [review]
Github pull request

This is needed for the vertical homescreen. It's rather minor and I don't see it causing any problems.
Attachment #8440112 - Flags: approval-gaia-v2.0?(bbajaj)

Updated

3 years ago
Attachment #8440112 - Flags: approval-gaia-v2.0?(bbajaj) → approval-gaia-v2.0+
v2.0: https://github.com/mozilla-b2g/gaia/commit/4be7d7d65f652dc1c48bce09638883d6bf3928e0
status-b2g-v2.0: --- → fixed
status-b2g-v2.1: --- → fixed
Target Milestone: --- → 2.0 S4 (20june)

Updated

3 years ago
Whiteboard: ux-tracking, visual design, visual-tracking, [ft:systemsfe][systemsfe][2.0-FL-bug-bash] → ux-tracking, visual design, visual-tracking, [ft:systemsfe][systemsfe]

Updated

3 years ago
blocking-b2g: --- → 2.0?
feature-b2g: --- → -

Updated

3 years ago
blocking-b2g: 2.0? → ---

Updated

3 years ago
Status: RESOLVED → VERIFIED
status-b2g-v2.0: fixed → verified
status-b2g-v2.1: fixed → verified

Comment 8

3 years ago
Created attachment 8530127 [details]
VIDEO0077.mp4

This issue has been successfully verified on Flame 2.0:
Gaia-Rev        8d1e868864c8a8f1e037685f0656d1da70d08c06
Gecko-Rev       https://hg.mozilla.org/releases/mozilla-b2g32_v2_0/rev/c756bd8bf3c3
Build-ID        20141127000203
Version         32.0
Device-Name     flame
FW-Release      4.4.2


This issue has been successfully verified on Flame 2.1:
Gaia-Rev        5372b675e018b6aac97d95ff5db8d4bd16addb9b
Gecko-Rev       https://hg.mozilla.org/releases/mozilla-b2g34_v2_1/rev/f34377ae402b
Build-ID        20141127001201
Version         34.0
Device-Name     flame
FW-Release      4.4.2
You need to log in before you can comment on or make changes to this bug.