As a user I want easy access to the attachment button by having the button closer to the keyboard so I can quickly hit the button. AC1: I want attachment icon to be not on the keyboard but close to it
604.19 KB, application/pdf
16.56 KB, image/png
16.33 KB, image/png
17.81 KB, image/png
26.46 KB, image/png
44.04 KB, image/png
9.31 KB, application/zip
46 bytes, text/x-github-pull-request
|Details | Review | Splinter Review|
blocking-b2g: --- → backlog
feature-b2g: --- → 2.1
User Story: (updated)
Note that this was how it was designed in v1.1 and was changed somewhen in v1.3 (bug 923023). We'll eventually need to settle. Needs UX but really need VD before being actionable.
This is being worked on from VD. Will provide visuals soon but there's some work still remaining to solve the spacing left for the text input area when saving space for the button. Not clearing NI as reminder. Remember all mockups and specs will be provided in a 2.1 messaging metabug where all changes related to this app should be gathered.
I'm ok with moving the attachment button to composer area. If that's what we are planning to do, I suggest we pull out the "call contact" function and place it on header (when there's only one recipient). Tks ;)
Created attachment 8450049 [details] [2.1 Message] Access to attach button v1.0.pdf Hello, please see attached for the spec. Thanks!
Find attached visuals and specs for implementing the attachment icon new placing. Cheers. V
Manuel, please note we took this in our sprint, this means we think the SMS team can handle this. So feel free to work on another work if you have some.
Assignee: nobody → azasypkin
Status: NEW → ASSIGNED
OS: Mac OS X → Gonk (Firefox OS)
Hardware: x86 → ARM
Hey Fang, Could you please provide us with the right image assets (1@x, @1.5x, @2x, @2.25x) for the new attachment button? Currently we have a white icon, but per spec color has changed + we didn't have @2x version for some reason. Thanks!
Created attachment 8462325 [details] sms_icon_attachment .zip Attached the new attachment icon at 1x 1.5x 2x 2.25x.
(In reply to Fang Shih from comment #13) > Created attachment 8462325 [details] > sms_icon_attachment .zip > > Attached the new attachment icon at 1x 1.5x 2x 2.25x. Thanks Fang!
Setting dependency on bug 1043518 as patch for this bug based on the latest master is impossible to verify in full.
Depends on: 1043518
Comment on attachment 8462753 [details] [review] GitHub pull request URL Hi Steve, Here is the patch with attachment button VR. Could you please review it? Thanks!
Comment on attachment 8462753 [details] [review] GitHub pull request URL Patch looks fine, but the unit test seems broken in composer test after applied it. Could you please investigate the root cause?
Comment on attachment 8462753 [details] [review] GitHub pull request URL (In reply to Steve Chung [:steveck] from comment #18) > Comment on attachment 8462753 [details] [review] > GitHub pull request URL > > Patch looks fine, but the unit test seems broken in composer test after > applied it. Could you please investigate the root cause? Thanks for review! I've added "type=button" to the add attachment button element, so tests should be fine now.
Comment on attachment 8462753 [details] [review] GitHub pull request URL Just one nit, so r=me :)
Attachment #8462753 - Flags: review?(schung) → review+
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
QA Whiteboard: [COM=Gaia::SMS] → [COM=Gaia::SMS][2.1-feature-qa+]
Hi Oleg, I do not see the change of call button on header for single recipient when I tried on Gaia 5fd14b8bc428f87f9b5cf9cc49f9a4f362a970fb Gecko https://hg.mozilla.org/mozilla-central/rev/e6614d8d85f9 BuildID 20140803160202 Version 34.0a1 We should expect the change at this user story since the spec is defined on UX doc, right?
(In reply to ashiue from comment #22) > Hi Oleg, > > I do not see the change of call button on header for single recipient when I > tried on > > Gaia 5fd14b8bc428f87f9b5cf9cc49f9a4f362a970fb > Gecko https://hg.mozilla.org/mozilla-central/rev/e6614d8d85f9 > BuildID 20140803160202 > Version 34.0a1 > > We should expect the change at this user story since the spec is defined on > UX doc, right? Hi Alison, Thanks for noticing! This bug was only about attachment button, but I've just filed a separate bug 1048717 for the "Call" button too. Thanks!
QA Whiteboard: [COM=Gaia::SMS][2.1-feature-qa+] → [COM=Gaia::SMS]
Whiteboard: [tako][p=1] → [tako][p=1][2.1-feature-qa+]
Tested and working Flame 2.1 Gecko-329b2e4 Gaia-d688d4f
Status: RESOLVED → VERIFIED
Flags: in-moztrap?(ashiue) → in-moztrap?
QA Contact: ashiue
Flags: in-moztrap? → in-moztrap?(echang)
QA Contact: echang
[Comms][Messages]Small set https://moztrap.mozilla.org/manage/suites/?pagenumber=1&pagesize=20&sortfield=created_on&sortdirection=desc&filter-name=%5Bcomms%5D&filter-name=messages&filter-name=small
Flags: in-moztrap?(echang) → in-moztrap+
blocking-b2g: backlog → ---
tracking-b2g: --- → backlog
You need to log in before you can comment on or make changes to this bug.