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

Improve plugin errors tooltip #2022

Merged
merged 1 commit into from
Nov 25, 2016
Merged

Improve plugin errors tooltip #2022

merged 1 commit into from
Nov 25, 2016

Conversation

fbarl
Copy link
Contributor

@fbarl fbarl commented Nov 21, 2016

Fixes #1905 - Plugin errors are difficult to view

@rade
Copy link
Member

rade commented Nov 21, 2016

We tend to have more descriptive commit messages, rather than just referencing an issue number.

@fbarl fbarl force-pushed the fix-plugins-tooltip branch 2 times, most recently from e773a68 to fbb8bbc Compare November 23, 2016 15:07
@fbarl fbarl changed the title [WIP] Plugin errors are difficult to view Improve plugin errors tooltip Nov 23, 2016
@fbarl
Copy link
Contributor Author

fbarl commented Nov 23, 2016

Triggered both on click and on hover:

image1
image2

I also added some opacity to the footer background, so that the it's readable when it floats above the nodes texts.

@foot
Copy link
Contributor

foot commented Nov 24, 2016

Nice one.

  • I'd use effect="solid" so it doesn't chase the cursor around.

Please change the commit message too!

@@ -187,11 +188,14 @@ h2 {
}

.footer {
padding: @footer-padding;
margin: -@footer-padding;

This comment was marked as abuse.

@foot
Copy link
Contributor

foot commented Nov 24, 2016

Other than that LGTM

@foot foot assigned fbarl and unassigned foot Nov 24, 2016
@fbarl fbarl force-pushed the fix-plugins-tooltip branch from 2130f49 to c110e32 Compare November 25, 2016 16:41
@fbarl fbarl merged commit 3083871 into master Nov 25, 2016
@fbarl fbarl deleted the fix-plugins-tooltip branch January 17, 2017 10:02
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.

3 participants