Update UI styles for multirecipient based on https://www.dropbox.com/sh/8jkb0osnvfda96e/NXt3gyd-oz/_Specs#f:SMS_v16_compose_measurements.png
Required for core MMS work, blocking+.
blocking-b2g: --- → leo+
Created attachment 750322 [details] Quick link to pull request
Created attachment 750326 [details] Quick link to pull request Hi, For this fix I moved the contact link into the message subHeader. This fix also required updating some dimensions so that the email bubbles display according to the mock. Please have a look and let me know if things need to be changed. Thanks, Mark
Comment on attachment 750326 [details] Quick link to pull request Hi Rick, this patch looks good to me now. If you have any concern, please comment on github or here.
Hi Victoria, The image Borja attached seems a little bit different from the visual in drapbox : https://www.dropbox.com/sh/8jkb0osnvfda96e/TkcQuTbhHj#f:01.SMS.MMS_0006_Compose_Longmessage.png The background color of the recipient container seems different. Could you confirm the color of the recipient area? Thanks.
Hi Steve, The good one is the one you're linking to, the white color. We kept the other so to use the specs spacing wise, but please reffer to the one you put above. If you have further questions please do not hesitate. My skype: @vixvixy Thanks!
Comment on attachment 750326 [details] Quick link to pull request Mark, since the current visual might have slightly difference from original one, please modify your patch and ask for review again. Thanks.
Attachment #750326 - Flags: review+
Attachment #750326 - Flags: feedback?(waldron.rick) → review?(schung)
Landed in master: 54118408dacaedf4b06c393d5b9a500450c5e7f2
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.