Summary: | RenderThemeWin should match Firefox button metrics | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Ojan Vafai <ojan> | ||||||
Component: | Layout and Rendering | Assignee: | Nobody <webkit-unassigned> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | adele, hyatt | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Mac | ||||||||
OS: | Windows XP | ||||||||
Attachments: |
|
Description
Ojan Vafai
2008-12-05 12:32:41 PST
Created attachment 25799 [details]
Sets the padding in RenderThemeWin
Comment on attachment 25799 [details]
Sets the padding in RenderThemeWin
This looks fine to me, but Hyatt or Adele should offer an opinion before we accept this.
Comment on attachment 25799 [details]
Sets the padding in RenderThemeWin
I would query the button style for appearance:none, and if it is none, don't apply this internal padding. Once you're not a "Windows" button, I don't think the theme should have any influence any more.
Created attachment 25892 [details]
Sets padding only if style->appearance() != NoControlPart
Comment on attachment 25892 [details]
Sets padding only if style->appearance() != NoControlPart
r=me
|