Closed Bug 1022590 Opened 10 years ago Closed 10 years ago

[meta] Desktop client needs incoming call handling

Categories

(Hello (Loop) :: Client, defect)

All
Windows 8
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED INVALID

People

(Reporter: RT, Unassigned)

References

Details

      No description provided.
Depends on: 990678
Depends on: 1000134
Depends on: 1000761
Depends on: 1011472
Summary: [meta] Desktop client needs incoming call notification → [meta] Desktop client needs incoming call handling
Depends on: 1022594
No longer depends on: 1011472
Depends on: 1015486
Depends on: 1011472
Depends on: 1030253
Depends on: 1038742
Depends on: 1045569
Comparing https://people.mozilla.org/~dhenein/labs/loop-mvp-spec/#call-incoming-unknown with current aurora 33 results in several differences:
1. background photo
2. URL of the call
3. small mic and camera buttons
4. date
5. different styles for decline/answer buttons (arrows, mic icon)
6. 'ignore' instead of 'decline'
7. avatar
I'm looking at the dependencies, but I'm not sure which ones are already filed. Romain, could you please give me a little hand?
Flags: needinfo?(rtestard)
(In reply to Paul Silaghi, QA [:pauly] from comment #1)
> Comparing
> https://people.mozilla.org/~dhenein/labs/loop-mvp-spec/#call-incoming-
> unknown with current aurora 33 results in several differences:
> 1. background photo
It is not a background photo but actually a preview of the users webcam as per Darrin's comment on the mockup.
This is tracked on https://bugzilla.mozilla.org/show_bug.cgi?id=1038742
> 2. URL of the call
This is tracked under https://bugzilla.mozilla.org/show_bug.cgi?id=1020449
> 3. small mic and camera buttons
This was required on https://bugzilla.mozilla.org/show_bug.cgi?id=1030253 but not delivered
Incoming call visual notification when not signed in (link call-back only):
* Information whether the caller's media selection is audio only or audio+video
> 4. date
This is tracked under https://bugzilla.mozilla.org/show_bug.cgi?id=1020449
> 5. different styles for decline/answer buttons (arrows, mic icon)
Wrong implementation of the proposed UX - we need a bug.
> 6. 'ignore' instead of 'decline'
I think the UX is wrong. I think we need "Decline" as "Ignore" would infer that the caller is not notified of the decline. "Ignore" may be addressed later at some point but we don't need it now. I NI Darrin to correct the UX
> 7. avatar
Yes, this is missing, we need a bug for this. This will always be a default Avatar when the link clicker initiates a call.

> I'm looking at the dependencies, but I'm not sure which ones are already
> filed. Romain, could you please give me a little hand?
Flags: needinfo?(rtestard) → needinfo?(dhenein)
Thanks, RT.
So, don't we need a follow up for 3 too?
(In reply to Romain Testard [:RT] from comment #2)
> > 3. small mic and camera buttons
> This was required on https://bugzilla.mozilla.org/show_bug.cgi?id=1030253
> but not delivered

Lets just get a separate bug on it. The parts of that bug had been separated down anyway.

> > 5. different styles for decline/answer buttons (arrows, mic icon)
> Wrong implementation of the proposed UX - we need a bug.

In 34 we've already implemented the Decline with Decline & Block drop-down.

Bug 990678 will implement the mic icon/drop down for the answer button.
(In reply to Mark Banner (:standard8) from comment #4)
> In 34 we've already implemented the Decline with Decline & Block drop-down.
Is there a chance to remember the bug number?
(In reply to Paul Silaghi, QA [:pauly] from comment #5)
> (In reply to Mark Banner (:standard8) from comment #4)
> > In 34 we've already implemented the Decline with Decline & Block drop-down.
> Is there a chance to remember the bug number?

Bug 1000134
Depends on: 1047435
Depends on: 1047438
Fixed in MVP spec, as well as removing chevron from Answer button at RT's request.
Flags: needinfo?(dhenein)
No longer depends on: 1038742
No longer depends on: 1047435
No longer depends on: 1047438
I've moved the remaining blockers to the bug 1066014 meta about incoming call handling, to consolidate the metas a bit. Hence, closing this one out.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.