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

[BUG]: Info overlay for web page doesn't inspect the element underneath #1854

Closed
drjoho opened this issue Feb 26, 2024 · 2 comments · Fixed by #1894
Closed

[BUG]: Info overlay for web page doesn't inspect the element underneath #1854

drjoho opened this issue Feb 26, 2024 · 2 comments · Fixed by #1894
Assignees
Labels

Comments

@drjoho
Copy link
Contributor

drjoho commented Feb 26, 2024

Use pointer-events: none

However, do not use for elements OFF screen

@drjoho drjoho self-assigned this Feb 26, 2024
@drjoho drjoho added extension-checker Issues related to the browser extensions priority-2 (med) development labels Feb 26, 2024
@drjoho
Copy link
Contributor Author

drjoho commented Apr 10, 2024

I have fixed the issue of being able to inspect the object behind the overlay using Use pointer-events: none. I do not see why using pointer-events: none on off screen objects since they can't be reached.
Since the overlay can not accept a click now we cannot have an x in the overlay to close it. However, we could have a button in the control panel that closes the current overlay and we could have a setting to turn the overlays off. So I need some help with "design" decisions before proceeding further or stopping where I am now.

@drjoho
Copy link
Contributor Author

drjoho commented Apr 16, 2024

Note the fix for this issues original issue: that inspect could not be used on an overlay and the underlying element could not be used. Is fixed and in master. So I am going to close this issue and open another issue regarding closing the overlay.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants