Page 1 of 1

Missing definitions

Posted: Fri Dec 01, 2017 12:32 am
by xenic
There is this item in SDK:Include/include_h/gadgets/layout.h:

#define LAYOUT_LabelColumn (LAYOUT_Dummy+34)
/* (ULONG) LABELTEXT_LEFT or LABELTEXT_RIGHT, the side where object
labels are placed in a vertical group. */

I can't find LABELTEXT_LEFT or LABELTEXT_RIGHT defined anywhere in the SDK. Can someone tell me what the values should be and add them to the SDK?

Re: Missing definitions

Posted: Fri Dec 01, 2017 10:49 am
by tonyw
Can't find those macro names anywhere so I suspect they are typos.

Have you tried using these values (from the end of layout.h) :

#define LALIGN_LEFT 0
#define LALIGN_RIGHT 1
#define LALIGN_CENTER 2

Re: Missing definitions

Posted: Fri Dec 01, 2017 11:23 am
by salass00
xenic wrote:There is this item in SDK:Include/include_h/gadgets/layout.h:

#define LAYOUT_LabelColumn (LAYOUT_Dummy+34)
/* (ULONG) LABELTEXT_LEFT or LABELTEXT_RIGHT, the side where object
labels are placed in a vertical group. */

I can't find LABELTEXT_LEFT or LABELTEXT_RIGHT defined anywhere in the SDK. Can someone tell me what the values should be and add them to the SDK?
The correct names are PLACETEXT_LEFT and PLACETEXT_RIGHT and they are mentioned in the autodoc (the defines themselves are in <libraries/gadtools.h>).

Re: Missing definitions

Posted: Sun Dec 03, 2017 7:46 pm
by xenic
salass00 wrote: The correct names are PLACETEXT_LEFT and PLACETEXT_RIGHT and they are mentioned in the autodoc (the defines themselves are in <libraries/gadtools.h>).
Good to know. However, my primary reason for reporting problems here is to get them corrected in the SDK. The note in the LAYOUT_LabelColumn item in SDK:Include/include_h/gadgets/layout.h needs to be corrected to match the autodoc. Will you or Tony correct it?

Re: Missing definitions

Posted: Mon Dec 04, 2017 11:00 am
by salass00
@xenic

I corrected the header file comment just after posting and committed it to the source repo.

Re: Missing definitions

Posted: Tue Dec 05, 2017 2:23 am
by xenic
salass00 wrote:@xenic

I corrected the header file comment just after posting and committed it to the source repo.
Thanks.