Re: [cp-patches] FYI: BasicListUI

[Lillian Angel]

On Fri, 2005-11-11 at 16:09 -0500, Lillian Angel wrote:
> On Thu, 2005-11-10 at 20:35 +0000, Roman Kennke wrote:
> > Hi,
> > 
> > I fixed some buggies in the BasicListUI. The fixed cell height and width
> > should now correctly be recognized. In addition to that I optimized the
> > painting so that only the cells within the clip bounds are painted. This
> > greatly improved performance when scrolling. This is really smooth now.
> > I'll implement a similar optimization for JTable, JTree and the text
> > components soon.
> > 
> > 2005-11-10  Roman Kennke  <address@hidden>
> > 
> >         * javax/swing/plaf/basic/BasicListUI.java
> >         (valueChanged): Repaint list when selection changed.
> >         (updateLayoutState): Reworked to correctly respect fixed cell
> > sizes.
> >         (installListeners): Create component listener before adding it.
> >         (paint): Optimized to only draw the cells in the clip.
> 

Also, attached is the same test case with a simple change. I put the
list inside a JScrollPane. When you scroll, your patch works, but the
painting is not reliable- the list disappears sometimes when scrolling.

Lillian

MyList.java
Description: Text Data