Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[201] [Jitsi Facelift] Update connection related styles from the new design #6

Conversation

Jazastry
Copy link

Treelo Card

Description

In this PR we are adding an updated participant low connection information display.

  • makes connection indicator to resize according to the connection status - worse status bigger icon
  • makes participants video layout to have a red shadow if the connection is low
  • makes poor connection notification to show if the connection of participants lower, if the connection improves the notification is removed without user interaction
  • fixes participants video layout hover styling
  • moves _participants-right-strip.scss to _participants.scss as this module is responsible not only for the right strip styling

Selection_001
Selection_005
Selection_006
Selection_007
Selection_008
Selection_009

- makes connection indicator to resize according to the connection
  status - worse status bigger icon
- makes participants videolayout to have red shadow if connection is low
- makes poor connection notification to show if the connection of
  connectiond participants lowers, if connection improves the
  notification is removed without user interaction
- fixes paticipants videolayout hover styling
- moves _participants-right-strip.scss to _participants.scss as this
  module is responsible not only for the right strip styling
@kachar
Copy link
Member

kachar commented Dec 10, 2020

@Jazastry Can you please update changelog.md to add your changes, thanks!

@kachar kachar merged commit bb15777 into threeveta-web-master Dec 11, 2020
@kachar kachar deleted the 201-jitsi-facelift-update-connection-related-styles-from-the-new-design branch December 11, 2020 09:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants