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

Bug: Hidden side bar can't opened #641

Closed
kamre opened this issue Jun 27, 2021 · 6 comments
Closed

Bug: Hidden side bar can't opened #641

kamre opened this issue Jun 27, 2021 · 6 comments
Assignees
Labels
Bug Something isn't working

Comments

@kamre
Copy link
Contributor

kamre commented Jun 27, 2021

Summary:
It is not possible to show the side bar when it was hidden with some active board.

Steps to reproduce the behavior:

  1. Activate (select) some board
  2. Click on hide side bar button
  3. Button (hamburger) for showing the side bar is not visible

Expected behavior:
Button for showing the side bar should be visible.

Screenshots:
HideSideBar

Platform:

  • Browser and version: Chrome 91
  • OS and version: Windows

Additional context:
This is a regression from 0.6.7 introduced in pull request #447 by setting z-index of .BoardComponent> .top-head inside webapp/src/components/centerPanel.scss to value 100 while z-index of hidden side bar is only 5.

@kamre kamre added the Bug Something isn't working label Jun 27, 2021
@kamre
Copy link
Contributor Author

kamre commented Jun 27, 2021

Looks like duplicate of #567 that is fixed in main branch.

But assigning bigger and bigger z-index looks a bit suspicious. Is there a place where the usage of z-index for different UI elements is described?

@peteristhegreat
Copy link

Raising the z index does fix it!

.Sidebar.hidden {
    z-index: 101;
}

@chenilim
Copy link
Contributor

@harshilsharma63, could you take a quick look at this and close it if the issue is indeed fixed in #567? Thanks.

@harshilsharma63
Copy link
Member

@chenilim yes this is a duplicate.

@kamre can you try it out the latest main?

As long as we can ensure relative values of z-index stay in the right order it's completely fine.

@kamre
Copy link
Contributor Author

kamre commented Jun 30, 2021

@harshilsharma63 It works correctly on the latest main branch.

I propose to add a check for hide/show sidebar when there is an active board to Cypress test. What do you think about this?

@harshilsharma63
Copy link
Member

@kamre yes we should definitely add a test for it since it's a primary functionality.

Would you be willing to add that test?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants