On Wed, Dec 5, 2012 at 11:13 AM, Rick McGuire <object.r...@gmail.com> wrote:
> I can't say I'm particularly fond of adding a new class just to implement
> this feature, particularly since the user could just specify the length
> using a call to the max() bif.
There is no way to use the max() bif here. The width and height of the
dialog is dependent on the strings used for labels of the radio buttons,
and the placement of the controls. It is not a simple length of a
character string length. Every string has to be converted to dialog units
dependent on the specific font and font size use for the dialog.
Calculating the needed dialog width to accommodate a specific title
without clipping it is somewhat complex.
The whole point of the class is to present a simple dialog where the
programmer does not have to make all these complicated calculations.
The current implementation does an ok job. But, it does not take into
account the title for the caption bar. If the title is long, the caption
can be clipped. That is the original problem that /hex would like fixed.
Originally he opened up a bug for this, and I said it's not really a bug,
it is working as it was intended to.
The current implementation also does not take into account that the
programmer can provide his own length for the radio buttons, and if he
does, the dialog's width is not calculated correctly. To me that is a bug.
The implementation I just finished is much more professional looking, it
takes into account a number of factors, including the caption bar title,
that the original implementation did not.
If we said the original implementation is buggy and call this a bug fix,
then the class and argument list could remain the same. That was /hex's
original desire.
> I'm not particularly fond of run-away argument lists either. Perhaps you
> could just make the length an optional argument.
Well that's the suggestion, add one optional argument. It just can't be a
length argument. It has to be use the old calculations or use the new
calculations.
--
Mark Miesfeld
------------------------------------------------------------------------------
LogMeIn Rescue: Anywhere, Anytime Remote support for IT. Free Trial
Remotely access PCs and mobile devices and provide instant support
Improve your efficiency, and focus on delivering more value-add services
Discover what IT Professionals Know. Rescue delivers
http://p.sf.net/sfu/logmein_12329d2d
_______________________________________________
Oorexx-users mailing list
Oorexx-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/oorexx-users
