Developer forums (C::B DEVELOPMENT STRICTLY!) > Development

rev 9137 causes C::B to use common controls 6.0.0

(1/1)

golgepapaz:
Manifest files introduced with the change makes C::B use common controls 6.0.0 (probably)
it looks ugly.To be sure I've checked it with the latest nightly build 9156 http://forums.codeblocks.org/index.php/topic,18034.0.html

How did this go unnoticed or is it just me?

Screenshots
pre 9137



after 9137

MortenMacFly:
...confirmed. I'll look into it. The actual idea was to be natively compatible with Windows 7/8.

MortenMacFly:

--- Quote from: MortenMacFly on June 14, 2013, 10:37:54 am ---...confirmed. I'll look into it. The actual idea was to be natively compatible with Windows 7/8.

--- End quote ---
Just a quick note: Its not as easy as it seems. Usually this really should work - it works just fine in several other applications. So the manifest/resource file is not the issue. I'm afraid one of our hacks (maybe the one to achieve debugging privileges on Windows) screws it. It was wrong all the time, just it becomes visual now.

Biplab:
Following lines are commented in resource.rc file (see line no 65 onwards).

--- Code: ---/*
  #if defined(WX_CPU_AMD64)
    wxMANIFEST_ID RT_MANIFEST "amd64.manifest"
  #elif defined(WX_CPU_IA64)
    wxMANIFEST_ID RT_MANIFEST "ia64.manifest"
  #elif defined(WX_CPU_X86)
    wxMANIFEST_ID RT_MANIFEST "x86.manifest"
  #else
    #warning No platform defined! Unable to embed manifest file.
  #endif
*/
--- End code ---

As a result no manifest file is being embedded in codeblocks.exe file.

MortenMacFly:

--- Quote from: Biplab on June 16, 2013, 02:47:17 pm ---As a result no manifest file is being embedded in codeblocks.exe file.

--- End quote ---
Yes, I've seen this meanwhile (I wasn't looking at the resource file in the first place, just at the manifest files). It was commented because it was wrong. I've fixed it properly in SVN head now.

Navigation

[0] Message Index