1051202 - SMS new message notice sometimes never disappears

Status: RESOLVED WORKSFORME

(Firefox OS Graveyard :: Gaia::SMS, defect, P2)

Description

[:gerard-majax]

[Blocking Requested - why for this release]: Bothering UX

This is something bothering on master and 2.0, whatever the device. It's present for some time already, but it may be tricky to reproduce. So far, the STR I could come with would be:

STR:
 0. Sit in an existing thread
 1. Put SMS app in background
 2. Receive two SMS in sequences
 3. Go back in SMS (tapping the icon or notification)
 4. See the banner notifying of a new SMS (new-message-notice-a-text)

Expected:
 I see the new message banner, and when scrolling to the bottom it disappears.

Actual:
 I see the new message banner and when scrolling to the bottom it stays there.

Comment 1

[Julien Wajsberg [:julienw]]

I didn't reproduce on master but I did in v2.0. That does not mean it does nt exist in master though, I just got it reprduce on v2.0 by chance. I can't reproduce on v2.0 now either ;)

I did notice something else on v2.0 when I reproduced it: when the thread was scrolled down, and I focus the input box, then the keyboard appears, the window resizes, and we should still see the most recent message at the bottom; however when the bug happened, we didn't see the most recent message at this moment.

What probably happens is that the measurement in [1] is failing.

[1] https://github.com/mozilla-b2g/gaia/blob/3c4f1370b4c62ba2316423164567568990fcd820/apps/sms/js/thread_ui.js#L986

However as I see it I wouldn't block on it because wen the bug happens the banner still disappears when the user taps on "read it".  Does it work like this for you as well, Alexandre?

Comment 2

[:gerard-majax]

Maybe not blocking, then. Maybe it's a UX call.

Comment 3

[Wesley Huang [:wesley_huang] (EPM) (NI me)]

I reproduced this with UX-Jenny. We agree that this is not blocking because the banner goes away if "read it" is tapped, or user exits SMS screen.

Comment 4

[Wesley Huang [:wesley_huang] (EPM) (NI me)]

do we still have this problem? 
If we can reproduce it, we may want to set P1.

Comment 5

[Julien Wajsberg [:julienw]]

I don't know, we never reproduced reliably.

Alexandre, do you see this lately?

Comment 6

[:gerard-majax]

I have not reproduced this for a while.