Bug 14419 - WebView: searchFor: direction: caseSensitive: wrap: method doesn't properly scroll to selected text
Summary: WebView: searchFor: direction: caseSensitive: wrap: method doesn't properly s...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Misc. (show other bugs)
Version: 419.x
Hardware: Mac OS X 10.4
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2007-06-26 12:32 PDT by Wade Cosgrove
Modified: 2007-07-14 16:22 PDT (History)
0 users

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Wade Cosgrove 2007-06-26 12:32:12 PDT 
When using the method searchFor:direction:caseSensitive:wrap: on a WebView that contains a "sub-element scroller" the selection isn't properly scrolled to.

For example, open Safari's RSS reader and search for a word that is below the bottom of the visible area. The text is properly selected, but isn't displayed as the scroller position isn't adjusted to make the newly selectd text visible.

This bug is present when a page contains a scroller that isn't part of the page's body content.
Comment 1 David Kilzer (:ddkilzer) 2007-07-05 21:07:15 PDT 
<rdar://problem/5316264>
Comment 2 David Kilzer (:ddkilzer) 2007-07-13 16:13:16 PDT 
Wade, was this tested in WebKit 419.x (e.g., the one that shipped with Tiger 10.4.x), or a WebKit nightly?  Is this fixed on a WebKit nightly?
Comment 3 Wade Cosgrove 2007-07-13 16:46:52 PDT 
I have confirmed this bug with WebKit version 522.11 which comes with Safari 3.0.x as well as the version that ships with Tiger. This issue does seem to be fixed with the 522+ nightly builds of WebKit.

I had been testing searching in the RSS view in Safari as a test case, and using the nightly build it did scroll down the page properly to select the word I was searching for.
Comment 4 David Kilzer (:ddkilzer) 2007-07-14 16:22:58 PDT 
Closing as RESOLVED/FIXED per Comment #3 (fixed in nightly build).