Skip to content
This repository has been archived by the owner on Nov 3, 2021. It is now read-only.

Bug 826293 Pick Contact Activity is broken if the Contact has more than ... #7294

Merged
merged 1 commit into from
Jan 5, 2013

Conversation

jmcanterafonseca
Copy link
Contributor

...one Phone Number

@basiclines
Copy link
Contributor

No idea how to test it :D, can you explain me the steps to follow?

@jmcanterafonseca
Copy link
Contributor Author

add a new contact with two phone numbers, let it be C. open the SMS app, click on '+', then click on the 'person' icon. The contacts app should be opened. In the contacts app, choose the contact C, a select dialog with the two numbers should be open and it should allow you to select one of the phone numbers of your contact. Once you select one it should appear in the SMS app as destination number.


section.valueselector [role="dialog"] li input {
left: 0;
.valueselector {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are we keeping this CSS file only for this bunch of rules?
I suggest to move them to contacts.css

As soon as we perform this change r+ from my side

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

of course!

@jmcanterafonseca
Copy link
Contributor Author

R+, BB+ ... landing!

jmcanterafonseca added a commit that referenced this pull request Jan 5, 2013
Bug 826293 Pick Contact Activity is broken if the Contact has more than ...
@jmcanterafonseca jmcanterafonseca merged commit d6baf70 into mozilla-b2g:master Jan 5, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants