Hi Jorge,

On Tue, Feb 12, 2013 at 12:06:12AM -0600, jorge ivan poot diaz wrote:
> Here is the changes I made, I declare the button according to the tutorial, 
> but
> I have not the expected results.
> 
> http://ooo.pastebin.ca/2313036
> http://ooo.pastebin.ca/2313037
> 
> The changes I have done well, as each code I put it where it belongs.

But it's not simply "putting code". You need to compile the code, and
you''ll the errors:

main/cui/source/dialogs/about.cxx: In constructor
'AboutDialog::AboutDialog(Window*, const ResId&)':
main/cui/source/dialogs/about.cxx:285:33: error: 'ABOUT_BTN_OK' was not
declared in this scope

main/cui/source/dialogs/about.cxx: In member function 'void
AboutDialog::LayoutControls(Size&)':
main/cui/source/dialogs/about.cxx:446:30: error: 'aOutSiz' was not
declared in this scope


The errors speak for themselves:

aOKSureButton( this, ResId( ABOUT_BTN_OK, *rId.GetResMgr() ) ),

here, ABOUT_BTN_OK is not defined.
In the new code, the define is RID_CUI_ABOUT_BTN_OK


aOKSurePnt.X() = 135 + ( aOutSiz.Width() - aOKSureSiz.Width() ) / 2;

there is no variable named aOutSiz in the current code.
Besides, this hard coded layout won't work with the new code (all the
layout is calculated relative to the two pictures, the logo (the orb) on
the left and the header on the top.


Regards
-- 
Ariel Constenla-Haile
La Plata, Argentina

Attachment: pgp8RH8eudWBc.pgp
Description: PGP signature

Reply via email to