HemmeligOrg / Hemmelig.app

Keep your sensitive information out of chat logs, emails, and more with encrypted secrets.
https://hemmelig.app
MIT License
697 stars 61 forks source link

fix: top view visibility in lifetime select component #230

Closed m4-droid closed 1 year ago

m4-droid commented 1 year ago

fixes #229 Increased the z-index for the lifetime select component to be more than the Quill component's z-index (1000)

sonarcloud[bot] commented 1 year ago

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

m4-droid commented 1 year ago

An alternative approach could involve reducing the Quill component z-index to <300 since default z-index of ttl select component is 300, if you another approach in mind, please let me know.

bjarneo commented 1 year ago

Thank you @m4-droid! Hm, this works, of course maybe it would have been better adjusting quill, but, I do not think it matters in this case as we do not have many elements with different z index.