1015883 - [Dialer][Call Screen][Call Waiting] Apply the visual refresh to the CDMA call waiting scenario

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::Dialer, defect)

Description

[Germán Toro del Valle (:gtorodelvalle)]

The 2.0 visual refresh should be applied to the CDMA call waiting scenario since it is a special case allowing the user to attend one of the 2 ongoing calls.

Since it is a CDMA network functionality, I'll upload a couple of screenshots about how they look for the time being in desktop Firefox Nightly (thanks to Gabriele for the hack). Once there is a spec available and implemented, I guess we should ask someone in a CDMA network to try it for us :-)

I created this new bug not to block bug 951606 so we can land it ASAP. We would work in this bug as a follow-up.

Comment 1

[Germán Toro del Valle (:gtorodelvalle)]

Attachment: desktop-callscreen-cdma-callwaiting-incoming.png

Comment 2

[Germán Toro del Valle (:gtorodelvalle)]

Attachment: desktop-callscreen-cdma-callwaiting-ongoing.png

Comment 3

[Germán Toro del Valle (:gtorodelvalle)]

Hi Vicky, do you know if there is a visual spec for this scenario? I only found the attachments included in bug 928700. Thanks!

Comment 4

[Victoria Gerchinhoren [:vicky]]

(In reply to Germán Toro del Valle from comment #3)
> Hi Vicky, do you know if there is a visual spec for this scenario? I only
> found the attachments included in bug 928700. Thanks!

Germán, this is slightly different from the regular 2nd incomming call scenario but very similar, right? Since I am in WW right now, Pau will provide the adaptation of this scenario. PLease ping him if there's anything that should be different from this screens. (Since I never saw interactions specs for this case)
Thanks!

Comment 5

[Pau Masiá [:Pau]]

Hi Carrie! Looking at these mockups I have seen that there appears no "decline with message" button. Is this right? I've got to provide a proper visual mockup with its corresponding spec in order to implement correctly and I would need that info, please.

Thanks!

Comment 6

[Carrie Wang [New account: [e-mail]]]

Attachment: CDMA_call waiting_20131211.pdf

Hi, UX spec attached here. Thanks!

Comment 7

[Victoria Gerchinhoren [:vicky]]

(In reply to Carrie Wang [:carrie] from comment #6)
> Created attachment 8429925 [details]
> CDMA_call waiting_20131211.pdf
> 
> Hi, UX spec attached here. Thanks!

Hey Carrie,

This spec is outdated and does not contain the new feature of "decline with message" that was later on incorporated. Can you please answer Pau with and updated spec so he can work on this screen? This is pretty urgent as it is blocking implementation.

Thanks!

Comment 8

[Carrie Wang [New account: [e-mail]]]

Hi Vicky, 

OK, we have another UX who works on CDMA part, so there might be a gap on spec transfer. In addition, the MADAI feature didn't update to CDMA spec. So, I've added it and uploaded here again. I think this is the correct one.

Comment 9

[Carrie Wang [New account: [e-mail]]]

Attachment: CDMA_conference call v1.2.pdf

Comment 10

[Carrie Wang [New account: [e-mail]]]

Please refer to p.7.

Comment 11

[Stephany Wilkes]

Sprint 3 ends Friday and this bug is unassigned. Please let me know who is implementing this. Thank you!

Comment 12

[Joe Cheng [:jcheng] (please needinfo)]

ni? Etienne/Rik

Comment 13

[Anthony Ricaud (:rik)]

No one. It wasn't ready for devs when we did our sprint planning and we're already swamped with work.

Comment 14

[Stephany Wilkes]

Joe or Wesley, please let me know if Rik's comment #13 means this will not make it or if there is something we can do here. Thanks!

Comment 15

[Victoria Gerchinhoren [:vicky]]

Just for the sake of clarity, you don't need extra visuals here as all components have already been designed and the only thing different to do is extending the "put on hold" button to be full width. And we can provide a nicer 2 arrow icon than the used in the current build. 

NI to Pau so he can provide the icon asset.

Comment 16

[Anthony Ricaud (:rik)]

For more context, we'll have to fix this before 2.0 ships. But unless some developer finds some free cycles (which I doubt), it will be after the "Feature Landing" period.

Also, CDMA development is harder because a lot of developers are not in countries with CDMA networks.

Comment 17

[Gabriele Svelto [:gsvelto] (PTO)]

I might be able to work on this, not sure if I can make it by Friday though.

Comment 18

[Pau Masiá [:Pau]]

Attachment: CDMA Assets

This is the icon requested.

Comment 19

[Pau Masiá [:Pau]]

Attachment: CDMA Assets

Comment 20

[Pau Masiá [:Pau]]

Attachment: Visual Mockup. CDMA. Switch calls idle

Mockup of how this new icon should look like in context.

Comment 21

[Pau Masiá [:Pau]]

Attachment: Visual Mockup. CDMA. Switch calls active

Comment 22

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

Edit assignee to Gabriele as he's working on it.
We didn't consider CDMA UI early enough in planning so the task came a little late. 
Very likely it won't be landed before FL.
Nominate to 2.0? to triage.

Comment 23

[Jason Smith [:jsmith]]

Joe - Triage believes this should be tagged feature-b2g flag, not the blocker flag, as this is a missed feature needed for 2.0. Can you set the feature-b2g flag here?

Comment 24

[Jason Smith [:jsmith]]

Wesley - Can you help with comment 23?

Comment 25

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

Didn't hear strong request from partner for this. Let's defer the feature into 2.1.

Comment 26

[Stephany Wilkes]

Flagging Vicky since she's been managing Comms VR, to ensure she is aware of this decision.

Comment 27

[Victoria Gerchinhoren [:vicky]]

Aware.

Comment 28

[Stephany Wilkes]

Noted as no longer blocking 2.0.

Comment 29

[Gabriele Svelto [:gsvelto] (PTO)]

Attachment: [PATCH] Applied the visual refresh to the CDMA call waiting scenario

This patch adjusts the CDMA call waiting scenario to match the visual refresh. Since the new appearance is simpler and less quirky than the old one I've taken the chance to clean up the CSS code and DOM:

- I've removed the shadow element which was used for a thin drop shadow in CDMA mode, it was a strange quirk we had only in that mode and I'm glad to see it go, the associated image has been removed.

- The switch-calls-button used to live in a special div (switch-calls) to the right of the numberWrapper for a number of reasons. This was also quirky and forced us to take it into account in a number of places in the CSS rules. Now the switch-calls div is gone and the button has been moved into the numberWrapper, this greatly simplifies the CSS code involved as well as making it more robust (it was brittle as hell before as it had very specific sizes and position).

- The assets for the button have been updated and the old ones removed. Note that the old assets were missing the high resolution icons so this is also very good.

- No changes were done to the unit-tests because the logic used to set the styles is unchanged (this was one of the few parts that was actually well done and robust to start with).

Pull request coming soon.

Comment 30

[Gabriele Svelto [:gsvelto] (PTO)]

Attachment: [PULL REQUEST] Applied the visual refresh to the CDMA call waiting scenario

Comment 31

[Anthony Ricaud (:rik)]

Gabriele: Can you join a screenshot and ask ui-review to Carol?

Comment 32

[Gabriele Svelto [:gsvelto] (PTO)]

Attachment: CDMA call waiting - incoming call scenario screenshot

Comment 33

[Gabriele Svelto [:gsvelto] (PTO)]

Attachment: CDMA call waiting - both calls connected scenario screenshot

Here are screenshots of both scenarios.

Comment 34

[Anthony Ricaud (:rik)]

Comment on attachment 8464028 [details] [review]
[PULL REQUEST] Applied the visual refresh to the CDMA call waiting scenario

Very well done in general.

Instead of doing the UI with floats, I'd rather use flexbox. Can you try this approach? If it requires too much work, we can keep the floats.

Comment 35

[Gabriele Svelto [:gsvelto] (PTO)]

Attachment: [PATCH v2] Applied the visual refresh to the CDMA call waiting scenario

New, flexbox-infused patch, it was only a two-lines change fortunately. The PR was also updated.

Comment 36

[Gabriele Svelto [:gsvelto] (PTO)]

Attachment: [PATCH v3] Applied the visual refresh to the CDMA call waiting scenario

This time for real ;-)

Comment 37

[Gabriele Svelto [:gsvelto] (PTO)]

Comment on attachment 8464690 [details] [diff] [review]
[PATCH v3] Applied the visual refresh to the CDMA call waiting scenario

Screenshots are in attachment 8464668 [details] and attachment 8464669 [details].

Comment 38

[Gabriele Svelto [:gsvelto] (PTO)]

Pushed to gaia/master 7a5372714c8256fbe28c6be33423e8cff5839220

Comment 39

[Anthony Ricaud (:rik)]

[Blocking Requested - why for this release]: If we intend to ship 2.0 on CDMA devices, I think we should take this patch. I'm not sure 2.0 is intended for CDMA devices though.
Before: attachment 8428606 [details]
After: attachment 8464669 [details]

Comment 40

[Jason Smith [:jsmith]]

Wilfred - Can you weigh in from the product perspective if this CDMA visual refresh is still needed for 2.0 or not? We're past FC at this point, so triage would like to avoid introducing risk into the product at this time preferably.

Comment 41

[Wilfred Mathanaraj [:WDM]]

no devices that I am aware of planned for 2.0 on CDMA - but the work is there if any partner wants to take it for their specific release.

Comment 42

[Loli (:lolimartinezcr)]

I need some QA contact that test this bug. We can't test it because we haven't got SIM card with CDMA