-
Notifications
You must be signed in to change notification settings - Fork 113
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
Fix: Modified z-index for draggable components, navbar and tabsBar in both expanded and collapsed states of tabsBar #261
base: main
Are you sure you want to change the base?
Conversation
…anded and collapsed states.
✅ Deploy Preview for circuitverse ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Hey @Arnabdaz, Please take a look at this. I have fixed the z-indexes following the requirements you specified. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @Arnabdaz I have updated the code and I believe we will be getting the expected behaviour. Video for referencepr_2.mov |
@Arnabdaz @Prerna-0202 it would be great if you could take a look at this. |
i have got some ideas should i start working on this issue @Chandrachur67 @Arnabdaz |
Fixes #260
Describe the changes you have made in this PR -
Modified the z-index values to be assigned to draggable elements during the "mousedown" event (set to previous values similar to circuitverse.org/simulator)
Modified code for the event listener of the tabsBar expand and collapse button
Screenshots of the changes (If any) -
Before
Once draggable components are moved to overlap with the top section, it loses the functionality of moving further
After
tabsBar not expanded state => draggable panels on top of everything.
tabsBar expanded state => navbar and tabsBar on top of draggable components for better accessibility of tabsBar
Video demonstrating the change
pr_video.mov
Note: Please check Allow edits from maintainers. if you would like us to assist in the PR.