Page MenuHomePhorge

room color overhaul
AbandonedPublic

Authored by bohlender on Feb 17 2021, 5:56 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Apr 7, 5:26 AM
Unknown Object (File)
Mar 7 2024, 1:45 PM
Unknown Object (File)
Feb 23 2024, 9:51 AM
Unknown Object (File)
Jan 28 2024, 11:07 AM
Unknown Object (File)
Jan 21 2024, 1:40 PM
Unknown Object (File)
Jan 15 2024, 7:12 PM
Unknown Object (File)
Dec 31 2023, 1:16 AM
Unknown Object (File)
Dec 28 2023, 6:08 AM
Subscribers

Details

Reviewers
machniak
Group Reviewers
Restricted Project

Diff Detail

Repository
rK kolab
Branch
colors
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 33274
Build 13048: arc lint + arc unit

Event Timeline

bohlender created this revision.
bohlender added a reviewer: Restricted Project.Feb 17 2021, 5:57 PM
machniak subscribed.
  • The grey status icons without the slash, do not indicate the muted state properly. Use red background here or icons with expunction element.
  • Enabled (muted) sound and video buttons have different size, which is not nice.
  • Screen sharing and hand button do not work. Some code still uses .text-danger to get the button state.
  • Browser tests need to be modified (.text-danger class)
  • Shouldn't the chat button also have the switched-on state when the chat widget is visible? The same as the hand or mute buttons.
  • I think there's too many elements in gray scale, the UI looks sad.
  • The chat background is too dark, the textarea does not look great with such a background.
  • If you join the room as a subscriber, the mute buttons are html-disabled, but have the background set, this looks strange. I don't know maybe the buttons (not icons) should be smaller, or just should not indicate switched-on state when disabled.
This revision now requires changes to proceed.Feb 18 2021, 12:40 PM