On 10/15/2013 12:33 PM, Christophe wrote:
> Hi, May I suggest to take the opportunity to transform these #define
> into an enum? that would make the code easier to work with. 

I'll take a look at it.
>>  [...]
>>  
>> +    entry = wMenuAddCallback(menu, _("Other maximization options"), NULL, 
>> NULL);
>> +    wMenuEntrySetCascade(menu, entry, makeMaximizeMenu(scr));
>> +
> On personal taste, I would suggest a shorter line "Other maximization" so it 
> does not stand out too much in the menu compared to the other entries.
That sounds good.  I thought it was a little long myself, but couldn't
think of anything shorter. 
> Apart from these, I think it's a great idea to bring more visibility to these 
> capabilities!

Thanks!

Doug


--
To unsubscribe, send mail to [email protected].

Reply via email to