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.

Sign up for GitHub

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

Jump to bottom

[Popover] should have z-index set to prevent content overlapping #332

Closed
MichalPaszowski opened this issue Jun 24, 2022 · 1 comment · Fixed by #398
Closed

[Popover] should have z-index set to prevent content overlapping #332

MichalPaszowski opened this issue Jun 24, 2022 · 1 comment · Fixed by #398
Assignees
@MichalPaszowski
Labels
bug Something isn't working

Comments

@MichalPaszowski
Copy link
Contributor

MichalPaszowski commented Jun 24, 2022 •
edited by sgraczyk
Loading

Describe the bug

One image is worth a thousand words:
Image

Steps To Reproduce

Place popover near element with higher z-index.

Expected behaviour

A popover should be over(hehe) other elements.

Screenshots

Image

Environment

N/A

Additional context

N/A
@MichalPaszowski MichalPaszowski added the bug Something isn't working label Jun 24, 2022
@MichalPaszowski MichalPaszowski self-assigned this Jun 24, 2022
@MichalPaszowski MichalPaszowski moved this to Selected for Development in LiveChat Design System Jun 24, 2022
@MichalPaszowski MichalPaszowski added this to the v1.0.0 milestone Jun 24, 2022
@sgraczyk sgraczyk changed the title Popover component should have z-index set to prevent content overlapping [Popover] should have z-index set to prevent content overlapping Jul 1, 2022
@MichalPaszowski MichalPaszowski moved this from Selected for Development to In Progress in LiveChat Design System Aug 26, 2022
@MichalPaszowski MichalPaszowski mentioned this issue Aug 26, 2022
Merged
6 tasks
@MichalPaszowski MichalPaszowski moved this from In Progress to In Review in LiveChat Design System Aug 26, 2022
@sgraczyk sgraczyk linked a pull request Aug 30, 2022 that will close this issue
#332 z-index for Popover component #398
Merged
6 tasks
MichalPaszowski added a commit that referenced this issue Sep 9, 2022
@MichalPaszowski
#332 z-index for Popover component (#398)
545948b 
* set zindex on popover

* use zindex variable

* follow agent app convention

* import variable
@sgraczyk sgraczyk moved this from In Review to Done in LiveChat Design System Sep 12, 2022
@sgraczyk
Copy link

Released in v1.0.0-alpha.22

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

@MichalPaszowski MichalPaszowski

Labels
bug Something isn't working
Projects
LiveChat Design System
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

#332 z-index for Popover component livechat/design-system
2 participants
@MichalPaszowski @sgraczyk