Bug 6043

Summary: Incorrect selection highlighting in pre-formatted text with tabs
Product: WebKit Reporter: mitz
Component: Layout and RenderingAssignee: Nobody <webkit-unassigned>
Status: VERIFIED FIXED    
Severity: Normal    
Priority: P2    
Version: 420+   
Hardware: Mac   
OS: OS X 10.4   
URL: http://perldoc.perl.org/perl.html
Attachments:
Description Flags
reduced testcase
none
Correct parameters in calls to RenderText::width()
none
Correct parameters in calls to RenderText::width() darin: review+

mitz
Reported 2005-12-11 08:53:38 PST
Summary: the selection highlight is painted incorrectly (offset to the right) when you select text on the right hand side of the man pages lists, e.g. try to select "references" in "Perl references short introduction". It does not happen on the two lines that don't start with a link (perl and perltoc).
Attachments
reduced testcase (225 bytes, text/html)
2005-12-11 10:25 PST, mitz 		no flags
Details
Correct parameters in calls to RenderText::width() (6.19 KB, patch)
2005-12-11 14:03 PST, mitz 		no flags
DetailsFormatted DiffDiff
Correct parameters in calls to RenderText::width() (7.87 KB, patch)
2005-12-11 23:27 PST, mitz 		darin: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
mitz
Comment 1 2005-12-11 10:25:32 PST
Created attachment 5041 [details] reduced testcase
mitz
Comment 2 2005-12-11 14:03:00 PST
Created attachment 5043 [details] Correct parameters in calls to RenderText::width() The fix for rdar://problem/3792529 is porbably responsible for at least one first-line regression that this patch fixes.
mitz
Comment 3 2005-12-11 14:07:27 PST
Comment on attachment 5043 [details] Correct parameters in calls to RenderText::width() Looks like there's more to it than this.
mitz
Comment 4 2005-12-11 23:27:43 PST
Created attachment 5048 [details] Correct parameters in calls to RenderText::width()
Darin Adler
Comment 5 2005-12-12 08:56:53 PST
Comment on attachment 5048 [details] Correct parameters in calls to RenderText::width() r=me Annoying that the compiler didn't catch this one. Shows that default values for parameters can cause problems.
Note You need to log in before you can comment on or make changes to this bug.