Bug 23072 - REGRESSION (r37371): In the Dictionary application, scroll bar appears inside its web view when resizing its window
Summary: REGRESSION (r37371): In the Dictionary application, scroll bar appears inside...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: DOM (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac OS X 10.5
: P2 Normal
Assignee: Darin Adler
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2009-01-02 00:47 PST by Darin Adler
Modified: 2009-01-02 18:06 PST (History)
1 user (show)

See Also:


Attachments
patch (11.76 KB, patch)
2009-01-02 01:06 PST, Darin Adler
no flags Details | Formatted Diff | Diff
second try (1.97 KB, patch)
2009-01-02 11:40 PST, Darin Adler
zwarich: review-
Details | Formatted Diff | Diff
third try -- this one I was able to test (21.41 KB, patch)
2009-01-02 16:43 PST, Darin Adler
sam: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Darin Adler 2009-01-02 00:47:34 PST
<rdar://problem/6368028>

Cameron figured out this is due to the Dictionary application erroneously setting the scrolling attribute on frames. I will do a workaround so we can work with the old Dictionary application and ask the Dictionary application folks to fix the new version.
Comment 1 Darin Adler 2009-01-02 01:06:08 PST
Created attachment 26363 [details]
patch
Comment 2 Oliver Hunt 2009-01-02 01:21:44 PST
Comment on attachment 26363 [details]
patch

r=me
Comment 3 Darin Adler 2009-01-02 01:27:33 PST
http://trac.webkit.org/changeset/39547
Comment 4 Darin Adler 2009-01-02 11:35:55 PST
Cameron says the fix did not work.
Comment 5 Darin Adler 2009-01-02 11:36:14 PST
Comment on attachment 26363 [details]
patch

Clearing patch since this was landed.
Comment 6 Darin Adler 2009-01-02 11:40:34 PST
Created attachment 26373 [details]
second try
Comment 7 Cameron Zwarich (cpst) 2009-01-02 14:43:48 PST
Comment on attachment 26373 [details]
second try

This patch still has the b/f problem, although I am not sure why. I'll r- it until we figure it out.
Comment 8 Darin Adler 2009-01-02 16:43:21 PST
Created attachment 26385 [details]
third try -- this one I was able to test
Comment 9 Darin Adler 2009-01-02 18:06:22 PST
http://trac.webkit.org/changeset/39566