1. Title : Long email id bubble on composer is out of composer screen 2. Precondition : Email should be working 3. Tester's Action: Launch Email -> composer -> To field -> enter long email id till the text goes left -> add ; at the end. 4. Detailed Symptom (ENG.) : a. Email address bubble is cut and out of screen. b. This issue also reproduces when we create a contact with long email id and in view contact, select email id c. Issue reproduces in To, cc and bcc fields. 5. Expected :Email id should be shown with properly. 6.Reproducibility: Y 1)Frequency Rate : 100% 7.Gaia Master/v1-train : Reproduced 8.Gaia Revision: 393b3f57822ae0f34055c6a6060f1433136bafa0 9.Personal email id: email@example.com
Created attachment 749248 [details] Pull Request pointer This issue is reported in composer screen. But .msg-peep-bubble is used for composer peep bubble since it has all the styling. So, Set Max width to -moz-available to make sure all the peep bubbles take the required width and set text-overflow: ellipsis; Please review it.
Hi Mihai, I have uploaded patch to this issue. Please un assign yourself, I will take it. Thanks.
(In reply to psingapati from comment #3) > Hi Mihai, > I have uploaded patch to this issue. > Please un assign yourself, I will take it. > > Thanks. Sure, it's all yours :)
Comment on attachment 749248 [details] Pull Request pointer Looks good! r=asuth. Please use rebase to put the bug number explicitly in the commit message rather than just in the pull request and then you can merge that.
Comment on attachment 749248 [details] Pull Request pointer Whoops, have to take the r=asuth back. This regresses the message reader's bubbles and causes them to become vertically unaligned with their labels. Probably for the same reason the attachment patch has trouble because of the filename and its overflow stuff. Please look into that.
Created attachment 749735 [details] Pull Request pointer I have tried the below two approaches 1) having common peep-bubble in mail.css but ended up with many regressions, may be I did in wrong way. so I left that approach 2) Tried with separate text-overflow:ellipsis; styling for .cmp-peep-bubble and added this class to the peep node along with .msg-peep-bubble. I tested thoroughly in both composer and message reader screens. If this approach is okay , please accept the patch. Thanks.
Comment on attachment 749735 [details] Pull Request pointer I would still prefer the cleanup, but I am aware that's a more major undertaking because of the places where the code explicitly uses the classes for identification purposes. r=asuth, but please fix the commit so that it has a real author; right now it's a very verbose build id. While there, if you could also fix up the commit message too. What I usually do is just copy and paste the bug number and title from the top of the bugzilla page. So in this case, it would be: Bug 871449 - [Email] Long email id bubble on composer is out of composer screen The key things to get, though, are the bug number with "Bug " in front of it for regexes we use, and the [Email] bit which helps make it more obvious in the git log what app was being modified. You can merge once that's fixed. Thanks!
Please request approval-gaia-v1 to land to v1.1, since this is not a blocker. Thanks!
Created attachment 750853 [details] Pull Request pointer Added New PR with proper commit message and author.
landed on gaia/master: https://github.com/mozilla-b2g/gaia/pull/9831 https://github.com/mozilla-b2g/gaia/commit/dff274b6689fe444dd71a017e2ccea4bfeb4b88f