Closed Bug 1025236 Opened 11 years ago Closed 11 years ago

(vertical-homescreen) Statusbar transition not correct when scrolling

Categories

(Firefox OS Graveyard :: Gaia::Homescreen, defect)

Product:
Firefox OS Graveyard
Component:
Gaia::Homescreen
Platform:
x86
macOS
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

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

Status:
VERIFIED FIXED
Milestone:
2.0 S4 (20june)
Project Flags:
feature-b2g -
Tracking Flags:
Tracking Status
b2g-v2.0 --- verified
b2g-v2.1 --- verified

People

(Reporter: pla, Assigned: kgrandon)

References

Details

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

Attachments

(4 files)

Vertical Scroll.mp4
354.67 KB, video/mp4
Details
StatusbarTransitionProblem.MOV
698.85 KB, video/quicktime
Details
Github pull request
46 bytes, text/x-github-pull-request
kgrandon
: review+
bajaj
: approval-gaia-v2.0+
Details | Review
VIDEO0077.mp4
4.09 MB, video/mp4
Details
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.
Attached video Vertical Scroll.mp4 —
This is the intended transition. A fade from 15% to 100% opacity of the status bar on scroll.
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.
Attached file Github pull request —
Assignee: nobody → kgrandon
Status: NEW → ASSIGNED
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+
Landed: https://github.com/mozilla-b2g/gaia/commit/b439206d688dae365dc116f2ed730f6ff7a60649
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
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)
Attachment #8440112 - Flags: approval-gaia-v2.0?(bbajaj) → approval-gaia-v2.0+
Whiteboard: ux-tracking, visual design, visual-tracking, [ft:systemsfe][systemsfe][2.0-FL-bug-bash] → ux-tracking, visual design, visual-tracking, [ft:systemsfe][systemsfe]
blocking-b2g: --- → 2.0?
feature-b2g: --- → -
blocking-b2g: 2.0? → ---
Status: RESOLVED → VERIFIED
status-b2g-v2.0: fixed → verified
status-b2g-v2.1: fixed → verified
Attached video 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.

Attachment

General

Creator:
Created:
Updated:
Size: