Bug 6916

Summary: REGRESSION: URLs with named anchors in a page don't jump to proper location in document
Product: WebKit Reporter: Gibbons Burke <gibbonsb>
Component: New BugsAssignee: Vicki Murley <vicki>
Status: VERIFIED FIXED    
Severity: Major CC: darin, ddkilzer, vicki
Priority: P1 Keywords: Regression
Version: 420+   
Hardware: Mac   
OS: OS X 10.4   
URL: http://www.nola.com/newslogs/tpupdates/index.ssf?/mtlogs/nola_tpupdates/archives/2006_01_28.html#108490
Attachments:
Description Flags
Reduced test case
none
Patch committed to svn none

Gibbons Burke
Reported 2006-01-29 11:46:48 PST
The URL included above is a "Permalink" pointer to a named anchor within the page of a news blog. Instead of scrolling directly to the named anchor tag, the latest version of Safari scrolls to the end of the page. The URL works as expected (correctly) in latest production versions of Safari, Firefox and OmniWeb. The bug has been present for some time in the nightly build versions.
Attachments
Reduced test case (2.73 KB, text/html)
2006-01-29 20:45 PST, David Kilzer (:ddkilzer) 		no flags
Details
Patch committed to svn (49.63 KB, patch)
2006-01-31 19:18 PST, David Kilzer (:ddkilzer) 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
David Kilzer (:ddkilzer)
Comment 1 2006-01-29 20:02:06 PST
Confirmed bug.
David Kilzer (:ddkilzer)
Comment 2 2006-01-29 20:45:47 PST
Created attachment 6095 [details] Reduced test case Reduced test case attached, although a smaller one may be possible. The <font> tag in front of the <a name=""> tag is significant.
Vicki Murley
Comment 3 2006-01-30 11:33:55 PST
<rdar://problem/4426668>
Vicki Murley
Comment 4 2006-01-30 16:39:03 PST
Fixed.
David Kilzer (:ddkilzer)
Comment 5 2006-01-30 16:48:49 PST
Is there a patch available for this? I didn't see anything checked into Subversion (yet), either. How can it be marked RESOLVED/FIXED?
David Kilzer (:ddkilzer)
Comment 6 2006-01-30 21:07:30 PST
Reopening since there is no patch and nothing was committed to Subversion to fix this bug.
Vicki Murley
Comment 7 2006-01-31 10:41:06 PST
How strange. . . I committed this on my work machine yesterday, but I guess something went wrong b/c this change is not in TOT. Maybe there was a conflict in one of the files and the commit didn't go through. Sorry!
Vicki Murley
Comment 8 2006-01-31 13:50:10 PST
I see now why my previous commit failed: "The following files contain tab characters: trunk/WebCore/ChangeLog" I just recommitted.
David Kilzer (:ddkilzer)
Comment 9 2006-01-31 19:18:27 PST
Created attachment 6171 [details] Patch committed to svn
David Kilzer (:ddkilzer)
Comment 10 2006-01-31 20:17:26 PST
Verified fixed as of r12510.
Eric Seidel (no email)
Comment 11 2006-01-31 21:20:59 PST
Removing Regression keyword from bugs already fixed.
David Kilzer (:ddkilzer)
Comment 12 2006-06-11 05:08:18 PDT
(In reply to comment #11) > Removing Regression keyword from bugs already fixed. Regression keyword re-added.
Note You need to log in before you can comment on or make changes to this bug.