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

Tooltip over variable decl has wrong style and content #3940

Closed
JakubValtar opened this Issue Oct 1, 2015 · 3 comments

Comments

Projects
None yet
2 participants
@JakubValtar
Contributor

JakubValtar commented Oct 1, 2015

Tooltip should display the warning instead of a variable name.

The style is not consistent with error tooltips (text size is bigger and possibly it has wrong color):

untitled

untitled2

@benfry

This comment has been minimized.

Show comment
Hide comment
@benfry

benfry Oct 1, 2015

Member

Odd; that's just a default tooltip (not running our code at all), but I can't find where setToolTipText() is being called to make this happen.

Member

benfry commented Oct 1, 2015

Odd; that's just a default tooltip (not running our code at all), but I can't find where setToolTipText() is being called to make this happen.

@benfry benfry added this to the 3.0.1 milestone Oct 1, 2015

@JakubValtar

This comment has been minimized.

Show comment
Hide comment
@JakubValtar

JakubValtar Oct 1, 2015

Contributor

I think we are setting it to null if we don't want it to show, so maybe we need that if there is no error or warning.

Contributor

JakubValtar commented Oct 1, 2015

I think we are setting it to null if we don't want it to show, so maybe we need that if there is no error or warning.

@benfry

This comment has been minimized.

Show comment
Hide comment
@benfry

benfry Oct 1, 2015

Member

But in that particular case, there's a warning, so I don't know why that isn't what's shown, and with it, the correct style.

Member

benfry commented Oct 1, 2015

But in that particular case, there's a warning, so I don't know why that isn't what's shown, and with it, the correct style.

@benfry benfry removed this from the 3.0.1 milestone Oct 23, 2015

@benfry benfry closed this in #4113 Feb 13, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment