On Mon, Dec 04, 2006 at 04:31:20AM -0000, [EMAIL PROTECTED] wrote:
> +     // NOTE: eidtXY returns the top level inset of nested insets. If you 
> happen

Try to spell correctly even within comments.

>  void Painter::buttonFrame(int x, int y, int w, int h)
>  {
> -     //  Width of a side of the button
> -     int const d = 2;
> -
> -     fillRectangle(x, y, w, d, LColor::top);
> -     fillRectangle(x, y + h - d, w, d, LColor::bottom);
> -
> -     for (int i = 0 ; i < d ; ++i) {
> -             line(x + i, y + i,
> -                  x + i, y + h - 1 - i, LColor::left);
> -             line(x + w - 1 - i, y + i + 1,
> -                  x + w - 1 - i, y + h - 1 - i, LColor::right);
> -     }
> +     line(x, y, x, y + h - 1, LColor::buttonframe);
> +     line(x - 1 + w, y, x - 1 + w, y + h - 1, LColor::buttonframe);
> +     line(x, y - 1, x - 1 + w, y - 1, LColor::buttonframe);
> +     line(x, y + h - 1, x - 1 + w, y + h - 1, LColor::buttonframe);
>  }

I am not saying this is bad, but this is surely changed behaviour.

Andre'

Reply via email to