| Summary: | [rulers] Annotation in vertical ruler not on the wrapped line | ||||||
|---|---|---|---|---|---|---|---|
| Product: | [Eclipse Project] Platform | Reporter: | Dani Megert <daniel_megert> | ||||
| Component: | Text | Assignee: | Andrey Loskutov <loskutov> | ||||
| Status: | RESOLVED FIXED | QA Contact: | |||||
| Severity: | normal | ||||||
| Priority: | P3 | CC: | Lars.Vogel, markus.kell.r | ||||
| Version: | 4.6 | ||||||
| Target Milestone: | 4.6 M4 | ||||||
| Hardware: | All | ||||||
| OS: | All | ||||||
| See Also: |
https://bugs.eclipse.org/bugs/show_bug.cgi?id=481819 https://git.eclipse.org/r/61706 https://git.eclipse.org/c/platform/eclipse.platform.text.git/commit/?id=3ed1845ebd7446875f59a4b229c203bfcf13bff6 |
||||||
| Whiteboard: | |||||||
| Bug Depends on: | 35779 | ||||||
| Bug Blocks: | |||||||
| Attachments: |
|
||||||
|
Description
Dani Megert
Dani, if someone else would be the reporter, I would close it as "won't fix" :-) The bookmarks have no offset information, they always point to the single line, see AddMarkerAction. So are you requesting an enhancement here? (In reply to Andrey Loskutov from comment #1) > Dani, if someone else would be the reporter, I would close it as "won't fix" > :-) > > The bookmarks have no offset information, they always point to the single > line, see AddMarkerAction. This is the ruler action which adds a line marker. You have to use the steps I provided i.e. use Edit > Add Bookmark... And bookmarks are just one example. Same problem for
- problem annotations, e.g. the "value of parameter is not used" problem in a Java editor:
void methodWithAVeryLongName(int unusedParameter) {
System.out.println(1);
}
- quick diff annotations (colored ruler background should span all visual lines of a wrapped document line)
Created attachment 258400 [details]
example of annotations, bookmark, quickdiff etc on wrapped text
Upcoming patch should allow to see something like attached screenshot.
New Gerrit change created: https://git.eclipse.org/r/61706 Dani, Andrey is this still planned for M4? If yes, please move back. Gerrit change https://git.eclipse.org/r/61706 was merged to [master]. Commit: http://git.eclipse.org/c/platform/eclipse.platform.text.git/commit/?id=3ed1845ebd7446875f59a4b229c203bfcf13bff6 |