Bug 17485 - Margins/padding expand on hover for some headlines on The Huffington Post
Summary: Margins/padding expand on hover for some headlines on The Huffington Post
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac OS X 10.5
: P2 Normal
Assignee: Dave Hyatt
URL: http://www.huffingtonpost.com/
Keywords: InRadar
: 17550 (view as bug list)
Depends on:
Blocks:
 
Reported: 2008-02-21 19:11 PST by Alice Liu
Modified: 2008-02-26 15:18 PST (History)
2 users (show)

See Also:


Attachments
Patch that fixes several bugs with this stuff (and improves our margin support on inline flows) (6.91 KB, patch)
2008-02-22 00:01 PST, Dave Hyatt
no flags Details | Formatted Diff | Diff
Patch that fixes the bug. (10.62 KB, patch)
2008-02-26 14:41 PST, Dave Hyatt
mitz: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Alice Liu 2008-02-21 19:11:48 PST
* SUMMARY
Margins and/or padding expand when the cursor is over, presumably during on hover, some headlines on the front page of The Huffington Post.

* STEPS TO REPRODUCE
1. Go to http://www.huffingtonpost.com/ (or, better yet, open the attached web archive)
2. Hover your cursor over the headline "The Man To Beat."

* RESULTS
Vertical margins and/or padding is expanded around the headline. When you move your cursor away, it collapses back to "normal."
Comment 1 Alice Liu 2008-02-21 19:12:05 PST
<rdar://problem/5759032>
Comment 2 Dave Hyatt 2008-02-22 00:01:41 PST
Created attachment 19275 [details]
Patch that fixes several bugs with this stuff (and improves our margin support on inline flows)

Work in progress.
Comment 3 Mark Rowe (bdash) 2008-02-26 13:20:59 PST
*** Bug 17550 has been marked as a duplicate of this bug. ***
Comment 4 Dave Hyatt 2008-02-26 14:41:53 PST
Created attachment 19385 [details]
Patch that fixes the bug.
Comment 5 Dave Hyatt 2008-02-26 14:44:10 PST
Ignore the Element.cpp change.  That's unrelated.
Comment 6 mitz 2008-02-26 15:13:19 PST
Comment on attachment 19385 [details]
Patch that fixes the bug.

r=me
Comment 7 Dave Hyatt 2008-02-26 15:18:06 PST
Fixed in r30603.