Bug 534318 - [code mining] Redraw of character has wrong y location for

line header annotation

Change-Id: Ife2631df1fc90d98e5c03e210902940e4c5e1e72
Signed-off-by: azerr <angelo.zerr@gmail.com>

1 files changed, 3 insertions, 3 deletions

diff --git a/org.eclipse.jface.text/src/org/eclipse/jface/text/source/inlined/InlinedAnnotationDrawingStrategy.java b/org.eclipse.jface.text/src/org/eclipse/jface/text/source/inlined/InlinedAnnotationDrawingStrategy.java
index 2ffcc52cc79..fc81f873fcc 100644
--- a/org.eclipse.jface.text/src/org/eclipse/jface/text/source/inlined/InlinedAnnotationDrawingStrategy.java
+++ b/org.eclipse.jface.text/src/org/eclipse/jface/text/source/inlined/InlinedAnnotationDrawingStrategy.java
@@ -93,15 +93,15 @@ class InlinedAnnotationDrawingStrategy implements IDrawingStrategy {
 			int x= bounds.x;
 			int y= bounds.y;
 
+			int height= annotation.getHeight();
 			// Colorize line spacing area with the background of StyledText to avoid having highlighted line color
 			gc.setBackground(textWidget.getBackground());
 			Rectangle client= textWidget.getClientArea();
-			gc.fillRectangle(0, y, client.width, annotation.getHeight());
+			gc.fillRectangle(0, y, client.width, height);
 
 			// Draw the line header annotation
 			annotation.setLocation(x, y);
 			annotation.draw(gc, textWidget, offset, length, color, x, y);
-			int height= annotation.getHeight();
 			if (height != 0) {
 				// The inline annotation replaces one character by taking a place width
 				// GlyphMetrics
@@ -127,7 +127,7 @@ class InlinedAnnotationDrawingStrategy implements IDrawingStrategy {
 				// FIXME: remove this code when we need not redraw the character (see https://bugs.eclipse.org/bugs/show_bug.cgi?id=531769)
 				// START TO REMOVE
 				int charX= x + bounds.width - charWidth;
-				int charY= y + bounds.height - height;
+				int charY= y + bounds.height - textWidget.getLineHeight();
 				if (style != null) {
 					if (style.background != null) {
 						gc.setBackground(style.background);