fix(tools): increase z-index of filters drawer and detail modal to z-… - #2457
fix(tools): increase z-index of filters drawer and detail modal to z-…#2457Akshita-tech17 wants to merge 2 commits into
Conversation
built with Refined Cloudflare Pages Action⚡ Cloudflare Pages Deployment
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2457 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 31 31
Lines 695 695
Branches 215 215
=========================================
Hits 695 695 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Utkarsh-123github
left a comment
There was a problem hiding this comment.
Hi @Akshita-tech17
The changes you made look good.
Could you please fix this small UI issue in the same PR as well? The search bar has a border in light mode, but it’s missing in dark mode. Please add the border for dark mode too.
Thanks!
|
Hi @Utkarsh-123github! Done! I've updated the branch with the fix for the dark mode border. Let me know if everything looks good now. Also, since I am new to open source and contributing, I'm really excited to learn more. Please keep guiding me if there is anything else I can improve. Thanks! |
What kind of change does this PR introduce?
Bugfix (UI Layout Improvement)
Issue Number:
Screenshots/videos:
If relevant, did you update the documentation?
N/A
Summary
This PR resolves an issue where the Tools page's Filter Drawer and Tooling Detail Modal did not overlay other page components properly because of their low z-index (
z-50).To fix this:
z-50toz-[200]in bothToolingDetailModal.tsxandindex.page.tsx.These changes ensure both overlays display on top of all other elements correctly.
Does this PR introduce a breaking change?
No
Checklist
Please ensure the following tasks are completed before submitting this pull request.