Skip to content

Usability: Change hover to click in html report #1053

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

Closed
uncaught opened this issue Dec 11, 2024 · 1 comment
Closed

Usability: Change hover to click in html report #1053

uncaught opened this issue Dec 11, 2024 · 1 comment

Comments

@uncaught
Copy link
Contributor

Q A
php-code-coverage version 11.0.z

I like the html report a lot, but it is not usable for us with javascript enabled. The popover displaying the covering lines is unreadable because it is so huge and is not scrollable. And also scrolling through the page itself with the mouse inside is very annoying due to the constant popup blocking the view.

I would like to ask if we could change the mouseenter/mouseleave events in src/Report/Html/Renderer/Template/js/file.js to toggle with a click event instead. Then I hope the popover will stay open, allows to scroll it, and doesn't come in the way when browsing a page.

I would create a PR if this was acceptable.

@sebastianbergmann
Copy link
Owner

I would create a PR if this was acceptable

Yes, please.

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