Closed
Bug 992994
Opened 11 years ago
Closed 11 years ago
[VDR][Email] Apply Buttons and Input areas [BB]
Categories
(Firefox OS Graveyard :: Gaia::E-Mail, defect)
Tracking
(b2g-v2.0 fixed)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
b2g-v2.0 | --- | fixed |
People
(Reporter: pivanov, Assigned: pivanov)
References
Details
(Whiteboard: ux-tracking, visual design, visual-tracking, bokken )
Attachments
(1 file, 1 obsolete file)
No description provided.
Assignee | ||
Comment 1•11 years ago
|
||
Attachment #8402754 -
Flags: feedback?(arnau)
Assignee | ||
Updated•11 years ago
|
Attachment #8402754 -
Attachment is obsolete: true
Attachment #8402754 -
Flags: feedback?(arnau)
Assignee | ||
Comment 2•11 years ago
|
||
Attachment #8402756 -
Flags: feedback?(arnau)
Assignee | ||
Updated•11 years ago
|
Attachment #8402756 -
Flags: review?(bfrancis)
Assignee | ||
Comment 3•11 years ago
|
||
Comment on attachment 8402756 [details]
patch for Gaia/master
oops ... Sorry Ben
Attachment #8402756 -
Flags: review?(bfrancis)
James,
Could you please check this patch? I think there's something wrong.
We need to link to buttons and input areas BB in style unstable folder.
For input areas, there's no problem, but buttons looks like it's not being copied inside email app.
So buttons.css is not imported in mail.css
Thanks!
Flags: needinfo?(jrburke)
Comment 5•11 years ago
|
||
Chiming in after IRC discussion with Arnau: It appears the style_unstable files work properly in Nightly but do not seem to get loaded on-device after make reset/install-gaia. I have not done any further investigation.
Comment 6•11 years ago
|
||
mcav's comment triggered something for me. I think the following line needs to be added to the apps/email/Makefile, after cp line for shared/style:
cp -rp ../../shared/style_unstable $(STAGE_APP_DIR)/shared
Flags: needinfo?(jrburke)
\o/ that's it James!
Will add this line to that PR.
Assignee | ||
Comment 8•11 years ago
|
||
Hey Arnau looks that this works ... :) can you check :) I updated the PR :)
Comment 9•11 years ago
|
||
Note that this patch touches .scrollregion-below-header but seems to be doing what :jrburke is proposing as part of his review feedback for bug 993215 to fix toolbar overlap. (change it to subtract off 5rem).
Assignee | ||
Comment 10•11 years ago
|
||
Thanks Andrew :) I made the changes
Blocks: 1001298
(In reply to James Burke [:jrburke] from comment #6)
> mcav's comment triggered something for me. I think the following line needs
> to be added to the apps/email/Makefile, after cp line for shared/style:
>
> cp -rp ../../shared/style_unstable $(STAGE_APP_DIR)/shared
James, your suggestion worked great, but looks like Makefile has been update.
Could you please update the file to make it work with style_unstable folder, or let us what should we include in that PR?
Thanks!
Flags: needinfo?(jrburke)
Comment on attachment 8402756 [details]
patch for Gaia/master
LGTM, just need to fix the makefile :)
Attachment #8402756 -
Flags: feedback?(arnau) → feedback+
Comment 13•11 years ago
|
||
As this patch is the first new one I know of that wants unstable styles now, I would put the Makefile change in this pull request.
Also, I am sure this was already planned, but be sure to rebase over latest master and remove the style in apps/email/style/common.css as that style is now already in message_cards.css.
Flags: needinfo?(jrburke)
Assignee | ||
Updated•11 years ago
|
Attachment #8402756 -
Flags: review?(jrburke)
Comment 14•11 years ago
|
||
Comment on attachment 8402756 [details]
patch for Gaia/master
Can you rebase on top of current master? The build system and the email app have changed noticeably since this patch branch, and I am seeing some weird behavior when trying to apply it, the unstable stuff is not coming over. Once the rebase is done, flip the review back to me.
Attachment #8402756 -
Flags: review?(jrburke)
Assignee | ||
Comment 15•11 years ago
|
||
Comment on attachment 8402756 [details]
patch for Gaia/master
Hey sorry ... I miss to push ... now I think is ok
Attachment #8402756 -
Flags: review?(jrburke)
Comment 16•11 years ago
|
||
Comment on attachment 8402756 [details]
patch for Gaia/master
So something is still not correct for builds, but it looks like the changes in the email Makefile as part of bug 968661 may be the source of the issue.
I am still sorting that out, but also trying to sort out a more serious data display issue for another bug. Hope to get this sorted in the next day or two.
Assignee | ||
Comment 17•11 years ago
|
||
Thanks James :) if I can help with something just ping me :)
Comment 18•11 years ago
|
||
Comment on attachment 8402756 [details]
patch for Gaia/master
I dug in a little more, and looks like the build system did change enough so to get this patch to work, you'll need to add link tags in the email/index.html for the new css files mentioned in the @import calls. Redundant, but unfortunately necessary for now.
Locally, I added these two lines to the email/index.html commented-out CSS section and that seemed to fix it:
<link rel="stylesheet" type="text/css" href="shared/style_unstable/input_areas.css" >
<link rel="stylesheet" type="text/css" href="shared/style_unstable/buttons.css" >
If you include that change in the pull request, r+ from me.
Attachment #8402756 -
Flags: review?(jrburke) → review+
Assignee | ||
Comment 19•11 years ago
|
||
I just updated the PR but want to be sure before merge ... can you take a look at:
https://github.com/mozilla-b2g/gaia/pull/18045/files#diff-6141e0195f9eca04c4c1480275d69945L14
Flags: needinfo?(jrburke)
Comment 20•11 years ago
|
||
Just pulled the latest from the pull request and pushed to device, did the same reset of the build as before, and still looks good, thanks!
Flags: needinfo?(jrburke)
Assignee | ||
Comment 21•11 years ago
|
||
Assignee | ||
Updated•11 years ago
|
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
status-b2g-v2.0:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•