Re: Range patchlet
- From: Bill Haneman <bill haneman sun com>
- To: Owen Taylor <otaylor redhat com>
- Cc: Tim Janik <timj gtk org>, Gtk+ Developers <gtk-devel-list gnome org>
- Subject: Re: Range patchlet
- Date: Thu, 24 Jan 2002 16:19:51 +0000
Owen Taylor wrote:
>
> Bill Haneman <bill haneman sun com> writes:
>
> > Owen Taylor wrote:
> > >
> > > Tim Janik <timj gtk org> writes:
> > >
> > > > On Thu, 24 Jan 2002, Bill Haneman wrote:
> > > >
> > > > > Hi:
> > > > >
> > > > > Here is a two-line patch that fixes a small bug in the gtkrange focus
> > > > > drawing - note that without this patch, the focus indication is only
> > > > > visible on the left and top edges, and is clipped out of the right and
> > > > > bottom edges. With this patch the focus indication is contained
> > > > > correctly inside the slider rectangle.
> > > > >
> > > > > OK to commit ?
> > > >
> > > > yep thanks, this is the standard off-by-one error with width/height.
> > >
> > > Actually, when support for wide focus indicators was added, we changed the
> > > interpretation of the rectangle passed to paint_focus; from Changes-2.0:
> > >
> > > The rectangle passed in is the bounding box, instead of
> > > the rectangle used in the <function>gdk_draw_rectangle()</function> call,
> > > so it is no longer necessary to subtract 1 from the width and height.
> >
> > Hmm, that may be so, but the patch still fixes the visible bug. Perhaps
> > a better fix would modify the are rectangle? Note that the rectangle is
> > obtained from a gdk_rectangle_intersect() call on range_rect (and
> > expose).
> >
> > So maybe the real offbyone error is somewhere else...
> >
> > (however the patch is in already, so a further fix would have no visible
> > impact I think).
>
> You are confusing me further:
>
> - I don't see any problems, _without_ this patch.
The default range focus line does not go all the way around the
range trough, as I documented in the email to the list.
> - This patch _is not_ in...
I checked it in on Tim's OK awhile ago.
-Bill
> Regards,
> Owen
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]