From a073611bbc9d1f5381ecbd6ec8cc28f03d178a79 Mon Sep 17 00:00:00 2001
From: Stefano Ceccherini <stefano.ceccherini@gmail.com>
Date: Mon, 6 Mar 2006 10:45:18 +0000
Subject: [PATCH] BTextView::OffsetAt() wasn't working with TAB charachters.
 Fixes bug 249

git-svn-id: file:///srv/svn/repos/haiku/haiku/trunk@16597 a95241bf-73f2-0310-859d-f6bbb57e9c96
---
 src/kits/interface/BTextView/TextView.cpp | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/src/kits/interface/BTextView/TextView.cpp b/src/kits/interface/BTextView/TextView.cpp
index 7d1a95b590..78cbacfd7b 100644
--- a/src/kits/interface/BTextView/TextView.cpp
+++ b/src/kits/interface/BTextView/TextView.cpp
@@ -1778,9 +1778,8 @@ BTextView::OffsetAt(BPoint point) const
 	point.x -= fTextRect.left;
 	point.x = max_c(point.x, 0.0);
 
-	// TODO: The following code chokes on TABs.
-	// we should do something like the part after the #else, but
-	// multi-byte characters safe.
+	// TODO: The following isn't efficient like the part after the #else,
+	// but it's multi-byte characters safe.
 #if 1
 	int32 offset = line->offset;
 	int32 limit = (line + 1)->offset;
@@ -1789,7 +1788,11 @@ BTextView::OffsetAt(BPoint point) const
 	do {
 		saveOffset = offset;
 		int32 nextInitial = NextInitialByte(offset);
-		float width = StyledWidth(saveOffset, nextInitial - saveOffset);
+		float width = 0;
+		if (ByteAt(offset) == B_TAB)
+			width = ActualTabWidth(offset);
+		else
+			width = StyledWidth(saveOffset, nextInitial - saveOffset);
 		if (x + width > point.x) {
 			if (fabs(x + width - point.x) < fabs(x - point.x))
 				offset = nextInitial;