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

z-index issue fix #103

Merged
merged 2 commits into from
Mar 1, 2024
Merged

z-index issue fix #103

merged 2 commits into from
Mar 1, 2024

Conversation

bram2202
Copy link
Contributor

@bram2202 bram2202 commented May 12, 2023

  • Remove z-index

Don't see the point of the z-index here.
Removed did not notice any other issues

closes #102

@bram2202
Copy link
Contributor Author

Missed some z-index, removed them.

Also updated the js file in dist because I don't have the build config

@darkstoneathome
Copy link

@r-renato Could you approve the PR?

@jlaska
Copy link

jlaska commented Jun 14, 2023

@r-renato Is anything needed to increase confidence in this PR?

@gfay63
Copy link

gfay63 commented Jun 15, 2023

THe fork from @bram2202 solved it for me as well. Please approve. And thanks again for the Integration! :)

@fubz
Copy link

fubz commented Feb 29, 2024

@r-renato Help! Please push this fix?

@r-renato r-renato merged commit 149b55e into r-renato:master Mar 1, 2024
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.

2023.5 update causes strange effect
6 participants