Closed
Bug 910351
Opened 11 years ago
Closed 11 years ago
[e.me][feature] Apply full bg image new design
Categories
(Firefox OS Graveyard :: Gaia::Everything.me, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: sonmarce, Assigned: evyatar)
References
Details
Attachments
(2 files)
No description provided.
Comment 1•11 years ago
|
||
Additional information:
Currently full bgimage mode (swipe results down till they disappear) displays a footer with info.
1. Attached is the new design.
2. Clicking the "wallpaper" pops up an alert for saving the image as wallpaper.
Comment 2•11 years ago
|
||
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → evyatar
Assignee | ||
Comment 3•11 years ago
|
||
Assignee | ||
Updated•11 years ago
|
Status: NEW → ASSIGNED
Comment 4•11 years ago
|
||
Attachment #810574 -
Flags: review?(crdlc)
Comment 5•11 years ago
|
||
r+ for the attachment ;)
Assignee | ||
Comment 6•11 years ago
|
||
Addressed comments on GH and Travis passed.
Cristian, are we good to land (no r+ yet)?
Flags: needinfo?(crdlc)
Comment 7•11 years ago
|
||
It is not r+ because I am not agree with the markup
See https://github.com/mozilla-b2g/gaia/pull/12475#issuecomment-25172408
Flags: needinfo?(crdlc)
Comment 8•11 years ago
|
||
I don't want to see a separator as image, please review with detail the header bb
Assignee | ||
Comment 9•11 years ago
|
||
as far as I can tell the BB also uses an image... am I looking at the wrong place?
https://github.com/mozilla-b2g/gaia/blob/master/shared/style/headers.css#L239
Comment 10•11 years ago
|
||
Sorry, I didn't see you comment, so r+
Updated•11 years ago
|
Attachment #810574 -
Flags: review?(crdlc) → review+
Assignee | ||
Comment 11•11 years ago
|
||
great! thanks, merged into master
ced4f2a44e81a61405d45f5a8298f4d5d1870162
https://github.com/mozilla-b2g/gaia/commit/ced4f2a44e81a61405d45f5a8298f4d5d1870162
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
blocking-b2g: --- → koi?
Assignee | ||
Comment 12•11 years ago
|
||
removing koi? since e.me 1.2 features are delayed for future version
blocking-b2g: koi? → ---
You need to log in
before you can comment on or make changes to this bug.
Description
•