Skip to content

[Live-Site] - make sidebar collapsible#720

Open
Pratiyushkumar wants to merge 2 commits intodevelopfrom
feature/collapsibleBar
Open

[Live-Site] - make sidebar collapsible#720
Pratiyushkumar wants to merge 2 commits intodevelopfrom
feature/collapsibleBar

Conversation

@Pratiyushkumar
Copy link
Contributor

@Pratiyushkumar Pratiyushkumar commented Oct 28, 2023

Issue

What is the change?

Currently, in desktop mode, the sidebar which contains a list of participants who join the live streaming, sidebar should be collapsible so that the video player screen can be enlarged

Is Development Tested?

  • Yes
  • No

Before / After Change Screenshots

bandicam.2023-10-28.20-47-46-331.mp4

@Pratiyushkumar Pratiyushkumar self-assigned this Oct 28, 2023
@Pratiyushkumar Pratiyushkumar changed the title sidebar is collapsing on click of expand button [Live-Site] - make sidebar collapsible Oct 28, 2023
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Oct 29, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 29f5151
Status: ✅  Deploy successful!
Preview URL: https://800e1809.www-rds.pages.dev
Branch Preview URL: https://feature-collapsiblebar.www-rds.pages.dev

View logs

@satyam73
Copy link
Contributor

image

Live staging site

image

Your changes

In this there's space coming from right to the sidebar, please fix this.

@satyam73
Copy link
Contributor

Also for mobile screen we can hide the full screen toggle button, since it's not needed

Copy link
Contributor

@satyam73 satyam73 left a comment

Choose a reason for hiding this comment

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

Can you please write the tests for the classes getting update on toggle?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants