Closed Bug 871449 Opened 11 years ago Closed 11 years ago

[Email] Long email id bubble on composer is out of composer screen

Categories

(Firefox OS Graveyard :: Gaia::E-Mail, defect, P2)

ARM
Gonk (Firefox OS)

Tracking

(blocking-b2g:-, b2g18+)

RESOLVED FIXED
1.1 QE2 (6jun)
blocking-b2g -
Tracking Status
b2g18 + ---

People

(Reporter: leo.bugzilla.gaia, Assigned: leo.bugzilla.gaia)

Details

(Whiteboard: [TD-26515])

Attachments

(1 file, 3 obsolete files)

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:  psingapati@gmail.com
Attached image Long EMail id bubble is cut (obsolete) —
Long email id bubble is cut.
Target Milestone: --- → 1.1 QE2
Assignee: nobody → mihai
Priority: -- → P2
Attached file Pull Request pointer (obsolete) —
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.
Attachment #749248 - Flags: review?(bugmail)
Hi Mihai,
I have uploaded patch to this issue.
Please un assign yourself, I will take it.

Thanks.
Flags: needinfo?(mihai)
(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 :)
Assignee: mihai → psingapati
Flags: needinfo?(mihai)
Assignee: psingapati → leo.bugzilla.gaia
Attachment #749248 - Attachment mime type: text/plain → text/html
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.
Attachment #749248 - Flags: review?(bugmail) → review+
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.
Attachment #749248 - Flags: review+ → review-
Attached file Pull Request pointer (obsolete) —
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.
Attachment #748687 - Attachment is obsolete: true
Attachment #749735 - Flags: review?(bugmail)
Attachment #749735 - Attachment mime type: text/plain → text/html
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!
Attachment #749735 - Flags: review?(bugmail) → review+
Please request approval-gaia-v1 to land to v1.1, since this is not a blocker. Thanks!
blocking-b2g: --- → -
tracking-b2g18: --- → +
Attached file Pull Request pointer
Added New PR with proper commit message and author.
Attachment #749248 - Attachment is obsolete: true
Attachment #749735 - Attachment is obsolete: true
Attachment #750853 - Flags: review?(bugmail)
Attachment #750853 - Attachment mime type: text/plain → text/html
Attachment #750853 - Flags: review?(bugmail) → review+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: