Re: [PATCH] comctl32: support drawing themed push buttons. [try 4] [resend]

[Reece Dunn]

2008/11/5 Roderick Colenbrander :
> Hi,
>
> I have the feeling that some of the drawing code is not correct and should use information which is buried inside the theme file. Things like content margins, text positions and things like that. I think it also requires test cases on what has to be done when a property isn't set in the theme.

Having had some experience with writing custom controls for Windows
that support theming, I haven't had to query explicitly for the
various properties. All of this should be handled by the uxtheme logic
in the various API calls.

That said, I suspect the code is missing a GetThemeBackgroundContentRect call.

- Reece