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

Show tooltip on hovering over the error #3119

Merged
merged 1 commit into from Apr 2, 2015

Conversation

Projects
None yet
2 participants
@omerjerk
Contributor

omerjerk commented Feb 22, 2015

Signed-off-by: Umair Khan omerjerk@gmail.com

I think hovering on the error should display the error. Just as in eclipse/idea.

@omerjerk

This comment has been minimized.

Show comment
Hide comment
@omerjerk

omerjerk Feb 22, 2015

Contributor

Screenshot :
Screenshot

Contributor

omerjerk commented Feb 22, 2015

Screenshot :
Screenshot

@benfry

This comment has been minimized.

Show comment
Hide comment
@benfry

benfry Apr 1, 2015

Member

Would you mind redoing this PR with the current source?

Member

benfry commented Apr 1, 2015

Would you mind redoing this PR with the current source?

@omerjerk

This comment has been minimized.

Show comment
Hide comment
@omerjerk

omerjerk Apr 2, 2015

Contributor

Rebased over the latest source.

Contributor

omerjerk commented Apr 2, 2015

Rebased over the latest source.

@benfry

This comment has been minimized.

Show comment
Hide comment
@benfry

benfry Apr 2, 2015

Member

Hm, still no auto-merge. Perhaps the changes I've made in the last few hours have broken it already?

Member

benfry commented Apr 2, 2015

Hm, still no auto-merge. Perhaps the changes I've made in the last few hours have broken it already?

@omerjerk

This comment has been minimized.

Show comment
Hide comment
@omerjerk

omerjerk Apr 2, 2015

Contributor

Let me rebase again. :)

Contributor

omerjerk commented Apr 2, 2015

Let me rebase again. :)

Show tooltip on hovering over the error
Signed-off-by: Umair Khan <omerjerk@gmail.com>
Conflicts:
	java/src/processing/mode/java/pdex/JavaTextAreaPainter.java

Conflicts:
	java/src/processing/mode/java/pdex/JavaTextAreaPainter.java

benfry added a commit that referenced this pull request Apr 2, 2015

Merge pull request #3119 from omerjerk/feature-error_tooltip
Show tooltip on hovering over the error

@benfry benfry merged commit 30797af into processing:master Apr 2, 2015

@benfry

This comment has been minimized.

Show comment
Hide comment
@benfry

benfry Apr 2, 2015

Member

All set, thanks!

Member

benfry commented Apr 2, 2015

All set, thanks!

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