844206 - [Email]Composing new email: Deleting "cc" recipient clears off the name of the "bcc" recipient

Status: VERIFIED WORKSFORME

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

Description

[Natalya Kot [:nkot]]

Attachment: screenshot 1

Description:
When user deletes "cc" recipient while composing new message, it clears off the name in the "bcc" recipient field

Prerequisites:
*Have a few contacts with email available

Repro Steps:
1) Updated to Unagi Build ID: 20130222070201
2) Launch Email app
3) Select to compose a new message
4) Tap "+" icon in the "cc" field and select contact
5) Tap "+" icon in the "bcc" field and select contact => screenshot 1
6) Tap on "cc" recipient and delete it
7) Verify information of the compose message screen => screenshot 2


Expected:
- "cc" field is cleared off
- other fields are not changed

Actual:
"bcc" recipient name is cleared off, only blank icon is left

Repro frequency:
100%, 2/2 devices

Environmental  Variables:
Kernel Date: Dec 5
Gecko: 9df4d6efd664
Gaia: 5691a16fff8e1403c75ed9d6f3a443b7e58198c6

*screenshots attached

Notes:
*when tapping in the "bcc" field or anywhere below on the screen the name comes back

Comment 1

[Natalya Kot [:nkot]]

Attachment: screenshot 2

Comment 2

[Andrew Sutherland [:asuth] (he/him)]

This seems like some type of CSS/layout issue causing the text in the address bubble to wrap to the next line inside its own container so you can't see it.

I wonder if this is related to our use of display: table-cell?

We might be able to just simplify the DOM implementation of cmp-peep-bubble to avoid the issue.  Right now there are 2 spans in there; one for the name and one for the e-mail address.  This was more appropriate when we used to display both, but now the e-mail address is just being used as a place to stash the user's e-mail address for display when we show it in the onContainerClick event that is used to display the action menu.

If we just used a data attribute to store the name of a JS expando property, that would work just as well and we could have cmp-peep-bubble have no child nodes and just store the user's name directly in the cmp-peep-bubble div node.  Unless I am missing some styling trick we have going on.

cc'ing Pavel because he has done some e-mail UI styling fixes and maybe could take a look?

Comment 3

[Pavel Ivanov [:ivanovpavel][:pivanov] UX]

The problem may actually be due to the use of display: table-cell (repaint/reflow maybe). Unfortunately I can't reproduce the bug.

Comment 4

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

nkot, can you try reproing in a more recent build please?

Comment 5

[Natalya Kot [:nkot]]

(In reply to Naoki Hirata :nhirata from comment #4)
> nkot, can you try reproing in a more recent build please?

yes, i could repro it today on unagi Build ID: 20130228070201
 
Gecko: 3867fca22e3d
Gaia: a41c7921fbf0839b894f277261dad4a22507f979

Comment 6

[Lukas Blakk [:lsblakk] use ?needinfo]

Still reproducible (apparently 100% so) therefore tracking, Pavel can you look into this further?

Comment 7

[Pavel Ivanov [:ivanovpavel][:pivanov] UX]

I will try again.

Comment 8

[Pavel Ivanov [:ivanovpavel][:pivanov] UX]

I still can't reproduce this :(

Comment 9

[Natalya Kot [:nkot]]

issue is no longer reproducible,

- deleting "cc" recipient doesn't affect "bcc" recipient field

tested on unagi running the following build:
Gecko  http://hg.mozilla.org/releases/mozilla-b2g18/rev/d467369d1b0c
Gaia   06e0e5ce42bdfb62bdbe38271de6b5b2d9e40e75
Build  2013-04-03-07-02-04

Comment 10

[Chris Kreinbring [:CKreinbring]]

Verified Worksforme on Unagi 1.1 mozilla RIL

Build ID: 20130429070204
Kernel Date: Dec 5
Gecko: http://hg.mozilla.org/releases/mozilla-b2g18/rev/49a908f052ec
Gaia: e26eadbc598bd09c80192016c1024a4a999a5361
Version #: 18.0