Skip to content

Conversation

jjsarton
Copy link

The html file has now button instead of div, ...
At one place the replacement what not possible (volume with vol on/off + slider)
The aspect and function for the web page with button don't change.


.collapse-video {
left: 30px;
left: calc(-100vw);
Copy link
Owner

Choose a reason for hiding this comment

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

Can you please explain what that does?

Copy link

Choose a reason for hiding this comment

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

This ensures that the chat icon will remain on the left even if the viewport width changes. E.g., if you have a tiling window manager and open a new window, without it firefox will display the icon on the right of the window.

Given the changes from this PR to what it is now, you would use left: calc(-100vw + 10px); instead of left: inherit; (now that the parent has left: 10px;.

overflow-y: hidden;

}
.sidenav.invisible {
Copy link
Owner

Choose a reason for hiding this comment

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

We already have .invisible, so isn't this redundant?

cursor: pointer;
overflow: hidden;
white-space: pre;
display: block;
Copy link
Owner

Choose a reason for hiding this comment

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

What does this do?


function openNav() {
document.getElementById("sidebarnav").style.width = "250px";
//document.getElementById("sidebarnav").style.width = "250px";
Copy link
Owner

Choose a reason for hiding this comment

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

Please don't leave commented-out code. Why is this being removed?


function closeNav() {
document.getElementById("sidebarnav").style.width = "0";
//document.getElementById("sidebarnav").style.width = "0";
Copy link
Owner

Choose a reason for hiding this comment

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

Please don't leave commented-out code.

// return;
// } else {
// openNav();
// }
Copy link
Owner

Choose a reason for hiding this comment

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

Please don't leave commented-out code.

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.

3 participants