Re: [cp-patches] The latest two patches on javax/swing/plaf/basic/BasicListUI.java break the List World demo: One line should be reverted

2006-01-30 Thread Roman Kennke
Hi Audrius, Am Montag, den 30.01.2006, 09:33 +0100 schrieb Audrius Meskauskas: > The new JList component does not appear because maybeUpdateLayoutState > does not update the layout state if the list.isValid() returns false. > > I would suggest avoid applying patches, breaking some already existi

Re: [cp-patches] The latest two patches on javax/swing/plaf/basic/BasicListUI.java break the List World demo: One line should be reverted

2006-01-30 Thread Roman Kennke
Hi Audrius, Am Montag, den 30.01.2006, 09:33 +0100 schrieb Audrius Meskauskas: > The new JList component does not appear because maybeUpdateLayoutState > does not update the layout state if the list.isValid() returns false. > > I would suggest avoid applying patches, breaking some already existi

Re: [cp-patches] The latest two patches on javax/swing/plaf/basic/BasicListUI.java break the List World demo: One line should be reverted

2006-01-30 Thread Audrius Meskauskas
The new JList component does not appear because maybeUpdateLayoutState does not update the layout state if the list.isValid() returns false. I would suggest avoid applying patches, breaking some already existing functionality. Even if such patches are correct by themselves, the methods that ha

[cp-patches] The latest two patches on javax/swing/plaf/basic/BasicListUI.java break the List World demo

2006-01-29 Thread Audrius Meskauskas
Hi, Roman, the latest two patches on BasicListUI break the list word demo in the Swing activity board (adding new element (by pressing the button) no longer adds the elements to the end of the list. It is possible that: 1. We may need to revert the patches or 2. Probably patches are correct a