| Summary: | [code mining] CodeMining should support line header/content annotation type both | ||||||
|---|---|---|---|---|---|---|---|
| Product: | [Eclipse Project] Platform | Reporter: | Angelo ZERR <azerr> | ||||
| Component: | Text | Assignee: | Platform-Text-Inbox <platform-text-inbox> | ||||
| Status: | RESOLVED FIXED | QA Contact: | |||||
| Severity: | enhancement | ||||||
| Priority: | P3 | CC: | daniel_megert, gautier.desaintmartinlacaze, mistria, noopur_gupta | ||||
| Version: | 4.8 | ||||||
| Target Milestone: | 4.8 M6 | ||||||
| Hardware: | All | ||||||
| OS: | All | ||||||
| See Also: |
https://git.eclipse.org/r/116631 https://git.eclipse.org/c/platform/eclipse.platform.text.git/commit/?id=907bc9caff4b9a164cb7a02c3ec3cadce90578c1 https://git.eclipse.org/r/116770 https://git.eclipse.org/c/platform/eclipse.platform.text.git/commit/?id=b301c7cc39babb8889ad7b83efc279d421882ef2 |
||||||
| Whiteboard: | |||||||
| Bug Depends on: | |||||||
| Bug Blocks: | 529011 | ||||||
| Attachments: |
|
||||||
|
Description
Angelo ZERR
Please set a target milestone again when you really plan to work on it. @Dani I wanted to work on this issue, but since CodeMining with large file is unusable, I'm trying to improve StyledText (see https://bugs.eclipse.org/bugs/show_bug.cgi?id=530019 and https://bugs.eclipse.org/bugs/show_bug.cgi?id=529570) But it's a really an hard task -( As soon as StyledText will be improved, I will work on this issue, but for the moment I take my main time to try to improve StyledText. New Gerrit change created: https://git.eclipse.org/r/116631 Created attachment 272519 [details]
Java CodeMining Parameter Demo
Here a demo with JDT to support parameter annotation by using the "line content" codemining support. You can see in this demo "line content"/"line header" codemining support.
You can in the demo that parameter name have white background when lien is selected although it should be blue. I have tried to use StyledText#getLineBackground(int i) but it doesn't work (returns null), but it seems because this backgroudn is managed with a LineBackroundListener (CursorLinePainter). Have you some idea how to fix that? Thanks! Gerrit change https://git.eclipse.org/r/116631 was merged to [master]. Commit: http://git.eclipse.org/c/platform/eclipse.platform.text.git/commit/?id=907bc9caff4b9a164cb7a02c3ec3cadce90578c1 Thanks a lot Angelo! New Gerrit change created: https://git.eclipse.org/r/116770 Gerrit change https://git.eclipse.org/r/116770 was merged to [master]. Commit: http://git.eclipse.org/c/platform/eclipse.platform.text.git/commit/?id=b301c7cc39babb8889ad7b83efc279d421882ef2 (In reply to Eclipse Genie from comment #9) > Gerrit change https://git.eclipse.org/r/116770 was merged to [master]. > Commit: > http://git.eclipse.org/c/platform/eclipse.platform.text.git/commit/?id=b301c7cc39babb8889ad7b83efc279d421882ef2 Fixed the API tools errors due to missing @since tags. |