On Tue, 2006-04-25 at 10:20 -0400, Lillian Angel wrote: > On Tue, 2006-04-25 at 10:19 -0400, Lillian Angel wrote: > > I noticed that the control icons are painted behind the lines drawn for > > the tree. I ran into this problem when working on BasicTreeUI. You need > > to make sure all the lines are painted before painting the control > > icons. It does not work if you try to paint both simulataneously. > >
Sorry for sending another email :( Here are all the problems I see with the swing tree demo: - Every time you click new element, the tree should be repainted instantly to see the new element. This happens when you add an element to an expanded directory. - The selection rectangle around the node is too small - The key listener does not work properly. I.e. expanding/collapsing nodes with the left and right keys. - The editing does not work well. The cursor does not appear when editing is activated (only on triple click this is broken). If you click on another node to cancel editing, the changes are made to that node you click on and the original name for that node is changed. - Location of the editing cell is a bit off. - The padding around the cells/icons/lines is non-existant. - control icons are painted behind the lines. - Sometimes the anchor is not shown on the node selected. Your work on JTree/BasicTreeUI has been really good. I am sure you are still working on this, but I thought you would be interested in knowing the remaining aesthetic issues. Lillian > > > Let me know if you need further clarification, > > Lillian > > > > > > On Sun, 2006-04-23 at 20:17 +0200, Audrius Meskauskas wrote: > > > It was discovered that JTree multiple element selection is currently not > > > working as expected. This path fixes the tree ability to select multiple > > > nodes, if permitted. > > > > > > 2006-04-23 Audrius Meskauskas <[EMAIL PROTECTED]> > > > > > > * examples/gnu/classpath/examples/swing/TreeDemo.java: > > > (createContent): Added check box to swith between single and > > > multiple selection. > > > * javax/swing/JTree.java (leadSelectionPath): Removed. > > > (addSelectionInterval): Explained. (getLeadSelectionPath): > > > Request the path from model. (getPathsBetweenRows): Explained. > > > (setLeadSelectionPath): Set the path in model. > > > * javax/swing/plaf/basic/BasicTreeUI.java > > > (TreeIncrementAction.actionPerformed, isMultiSelectionEvent, > > > isToggleSelectionEvent, selectPath, selectPathForEvent): Rewritten. > > > (MouseHandler.mousePressed): Call selectPathForEvent.
