891830 - [SMS/MMS] The screen shown when tapping on '+' to add a contact from contacts is not fine

Status: RESOLVED WONTFIX

(Firefox OS Graveyard :: General, defect, P1)

Description

[isabelrios]

Attachment: Contacts list view

Unagi device 07/09 v1-train build, ref ril:
Gecko-e78450a
Gaia-e251ee6

STR
1. Create a contact on the addressbook
2. Open messaging app
3. Tap on '+' icon to add the contact previously stored

EXPECTED
After tapping on '+' the contacts list is shown correctly

ACTUAL
The contact list is shown but it apperas with a different colour where the keyboard was previously open.
Please see screenshot attached

Comment 1

[Borja Salguero [:borjasalguero]]

Rudy, I think this is something related with the Keyboard. In the STR we are typing in a 'content-editable' item, and when tapping on an activity this is the result. As you can see the 'gap' is the one for the keyboard, so it seems that keyboard space is 'stucked' there. Could you take a look? Thanks!

Comment 2

[John Hammink]

Why do we want to block on this?  Is it preventing the user from doing something?  Or is it just UX?

Comment 3

[Rick Waldron [:rwaldron]]

This isn't a bug in the Messages or Keyboard, it's a repaint issue and likely a dup of bug 811135 (which itself appears to be a dup, so I'm waiting to learn what the original bug was, since it isn't posted on that ticket)

Comment 4

[Rick Waldron [:rwaldron]]

Sorry, didn't mean to make this Gaia::Keyboard

Comment 5

[bhavana bajaj [:bajaj]]

Although not neat to look at, we eat it up this cycle given the time and since its not a regression and take a fix in 1.2

Comment 6

[Lukas Blakk [:lsblakk] use ?needinfo]

See comment 5 - without a champion for the issue at this stage (and given this is a minor polish bug) we see no reason to block here.

Comment 7

[Rudy Lu [:rudyl] (inactive after 2015/7/3)]

No clue right now and does not seem like a Gaia keyboard issue from my point of view.

Comment 8

[滕飞]

hi,
I'm from ZTE corporation,we also get into trouble with the same problem.
We can't add more than one contacts on the sms send message interface.

I find the code from apps/sms/js/thread_ui.js,function updateComposerHeader,line 496,

if (recipientCount > 0) {
   this.contactPickButton.classList.add('disabled'); <------
..

The code means: "recipientCount " defined the contacts list. if the list mumbers are more than 1, the "plus" btn will be disabled.

when I comment this code(//this.contactPickButton.classList.add('disabled');) , We could add more than one contacts and the "plus" btn works well.

But I doult why the engnieer add this code here. Are there any other reasons? 

Because it seems that it's easy to fix the problem on the interface layer. 

Could sb. give me a hand ; 

Thanks a lot && Best Regards.

Comment 9

[BMO Automation]

Firefox OS is not being worked on