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

Render: Fix panel titles should not be focused when rendering #114

Merged
merged 1 commit into from
Mar 19, 2020

Conversation

AgnesToulet
Copy link
Contributor

@AgnesToulet AgnesToulet commented Mar 11, 2020

Add a call to the mouse.move function so the mouse is in the bottom right corner of the viewport instead of the top left corner.
It allows to not have the "mouse:hover" style on titles while rendering panels.

I've tested several calls to the render function and it seems not to have any other impact.

Fixes #111

@AgnesToulet AgnesToulet requested a review from marefr March 11, 2020 09:52
@aknuds1 aknuds1 self-requested a review March 16, 2020 14:32
Copy link
Contributor

@aknuds1 aknuds1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@marefr You agree with this change?

@marefr
Copy link
Member

marefr commented Mar 16, 2020

We just need to test it

@aknuds1
Copy link
Contributor

aknuds1 commented Mar 16, 2020

Any suggestions on how to test it @marefr? You know better than me how to look for eventual bugs from this change.

@marefr
Copy link
Member

marefr commented Mar 16, 2020

@aknuds1 you can test referenced issue by creating a panel which have a panel link and use v1.0.10 to reproduce. Then checkout this branch and build a local plugin version or docker image and verify it resolves the issue. Please let me know if you would like me to do it.

Copy link
Member

@marefr marefr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Was not able to reproduce so maybe problem on older Grafana versions. But it still works as expected so LGTM

@marefr marefr changed the title Move mouse before screenshot Render: Fix panel tooltip content should not be visible when rendering Mar 19, 2020
@marefr marefr changed the title Render: Fix panel tooltip content should not be visible when rendering Render: Fix panel titles should not be focused when rendering Mar 19, 2020
@marefr marefr merged commit a0c939c into grafana:master Mar 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Image render shows popup description without disable popup option
3 participants