[SMS/MMS] The options shown in the dialog when tapping on a contact into ‘To’ field are the wrong way round

VERIFIED FIXED

Status

Firefox OS
Gaia::SMS
VERIFIED FIXED
5 years ago
5 years ago

People

(Reporter: isabelrios, Assigned: gduan)

Tracking

unspecified
ARM
Gonk (Firefox OS)
Bug Flags:
in-moztrap +

Firefox Tracking Flags

(blocking-b2g:leo+, b2g18 fixed, b2g-v1.1hd fixed)

Details

(Whiteboard: MMS_TEF, [u=commsapps-user c=messaging p=0.5], [LeoVB+] )

Attachments

(2 attachments)

(Reporter)

Description

5 years ago
Created attachment 771985 [details]
Remove contact dialog

This is a follow up bug of bug 880624

STR
1.Start creating a SMS or MMS.
2.Tap on ‘+’ to add one recipient from contacts
3.Once the contact is into ‘To’ field, tap on it
4.Check the dialog shown

EXPECTED
The buttons colour is ok but the remove button should be on the left side.

ACTUAL
The two buttons at the bottom are in the wrong place. 
See attached file.
Comment on attachment 772010 [details]
https://github.com/mozilla-b2g/gaia/pull/10835

put cancel button on the right side.
Attachment #772010 - Flags: review? → review?(schung)
Blocking on the assumption that you're reporting this based on system-wide spec criteria and consistency in the OS.
blocking-b2g: leo? → leo+
Flags: needinfo?(firefoxos-ux-bugzilla)

Comment 4

5 years ago
Ayman, can you verify and/or post a link to the system-wide expectation for modals (OK/Cancel).
Flags: needinfo?(firefoxos-ux-bugzilla) → needinfo?(aymanmaat)
Whiteboard: MMS_TEF → MMS_TEF, [u=commsapps-user c=messaging p=0]
Assignee: nobody → gduan

Comment 5

5 years ago
Test cases available in MozTrap #8025
Flags: in-moztrap+
George, I've sent you a PR with small tweaks. Could you take a look? Thanks!
Flags: needinfo?(gduan)
OS: Windows 7 → Gonk (Firefox OS)
Hardware: x86_64 → ARM
Whiteboard: MMS_TEF, [u=commsapps-user c=messaging p=0] → MMS_TEF, [u=commsapps-user c=messaging p=0.5]
Thanks Borja.
Flags: needinfo?(gduan)
Comment on attachment 772010 [details]
https://github.com/mozilla-b2g/gaia/pull/10835

Taking this review!
Attachment #772010 - Flags: review?(schung) → review?(fbsc)
Comment on attachment 772010 [details]
https://github.com/mozilla-b2g/gaia/pull/10835

George, R+! Thanks for your patch :)
Attachment #772010 - Flags: review?(fbsc) → review+
Uplifted d0c9d97070ee6e46a2396e598b1123f0f7ac58c4 to:
v1-train: f5e809d3ee6f624cce7b35d72cf187f4fe2dada5
status-b2g18: --- → fixed
(Reporter)

Comment 12

5 years ago
The options appear in the right place now.
Verified with unagi v1-train build: 
Gecko-2b310a1
Gaia-2e711c1
Status: RESOLVED → VERIFIED

Updated

5 years ago
Whiteboard: MMS_TEF, [u=commsapps-user c=messaging p=0.5] → MMS_TEF, [u=commsapps-user c=messaging p=0.5], [LeoVB+]
v1.1.0hd: f5e809d3ee6f624cce7b35d72cf187f4fe2dada5
status-b2g-v1.1hd: --- → fixed

Comment 14

5 years ago
removing ni to me as fixed/resolved/verified
Flags: needinfo?(aymanmaat)
You need to log in before you can comment on or make changes to this bug.