Re: [PATCH v2] Calc: Add a simple implementation.

[Alexandre Julliard]

Hua Meng <161220092(a)smail.nju.edu.cn> writes:
> +    /* create Buttons */
> +    for(i = 0; i < ButtonNum; i++)
> +    {
> +        x = 15 + (i % 3) * 70;
> +        y = 120 + (i / 3) * 90;
> +        nWidth = 70;
> +        nHeight = 90;
> +
> +        /* set token buttons */
> +        if(i >= 12)
> +        {
> +            nHeight = 72;
> +            x = 225;
> +            y = 120 + (i % 12) * 72;
> +        }

You can't hardcode values like this, it should depend on the font
size. It would be a lot easier to start from a dialog template instead
of creating everything by hand.

-- 
Alexandre Julliard
julliard(a)winehq.org