Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

Fixes for call UI #5509

Merged
merged 6 commits into from
Dec 21, 2020
Merged

Fixes for call UI #5509

merged 6 commits into from
Dec 21, 2020

Conversation

dbkr
Copy link
Member

@dbkr dbkr commented Dec 18, 2020

Separated by commit

  • Add bottom margin on PiP view
  • Remove avatar blurring & pause icon for held calls
  • Change background of incoming call box to match PiP view
  • Put drop shadow & border radius on PiP view & incoming call box
    rather than the CallContainer they're in (so they each have their own
    drop shadow / rounded corners).
  • Add margin between incoming call box and PiP view
  • Use 'Switch' as resume link if unholding would hold another call
  • Remove the secondary call avatar
  • Change 'paused' to 'on hold'

Avatar no longer blurred

Screenshot 2020-12-18 at 15 51 16

Margin between PiP View & Incoming Call

Screenshot 2020-12-18 at 18 07 18

Secondary avatar removed, 'paused' changed to 'on hold'

Screenshot 2020-12-18 at 19 53 15

'Switch' link in PiP view

Screenshot 2020-12-18 at 19 40 29

'Switch' link in large call view

Screenshot 2020-12-18 at 19 40 37

@nadonomy for approval

dbkr added 3 commits December 18, 2020 18:08
 * Add bottom margin on PiP view
 * Remove avatar blurring & pause icon for held calls
 * Change background of incoming call box to match PiP view
 * Put drop shadow & border radius on PiP view & incoming call box
   rather than the CallContainer they're in (so they each have their own
   drop shadow / rounded corners).
 * Add margin between incoming call box and PiP view
Use 'Switch' if unholding that call would hold another
and change 'paused' to 'on hold'
@dbkr dbkr requested review from a team December 18, 2020 19:51
Copy link
Contributor

@nadonomy nadonomy left a comment

Choose a reason for hiding this comment

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

LGTM

@dbkr dbkr changed the title Fixes for call UI [Release] Fixes for call UI Dec 21, 2020
@dbkr dbkr changed the title [Release] Fixes for call UI Fixes for call UI Dec 21, 2020
Copy link
Collaborator

@jryans jryans left a comment

Choose a reason for hiding this comment

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

Looks good overall! 😄

@@ -84,6 +88,7 @@ limitations under the License.
border-radius: 2000px;
overflow: hidden;
position: relative;
/* Blurred avatar images & pause icon for on-hold removed for now
Copy link
Collaborator

Choose a reason for hiding this comment

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

Would it make more sense to just remove it? Or you expect it to return soon...? It's always available in version control. Anyway, up to you.

Copy link
Member Author

Choose a reason for hiding this comment

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

Yeah true, otherwise it'll just hang around in the css for ages

src/CallHandler.tsx Outdated Show resolved Hide resolved
dbkr and others added 2 commits December 21, 2020 11:21
Co-authored-by: J. Ryan Stinnett <jryans@gmail.com>
@dbkr dbkr merged commit a126c2e into develop Dec 21, 2020
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.

3 participants