Build ID: just pulled Steps to Reproduce: (1) Go to any tree in the app. (2) Select one item in the tree. (3) Press the left arrow key. Result: The item is deselected. Future me :P
Reporters future is secure, will future the bug instead ;-)
Target Milestone: --- → Future
Fair enough! ;) Anyways, I have a simple fix for this. It was finding the tree itself as the parentNode and selecting it, when it should just be finding the parent folder. reassigning to me
Assignee: hyatt → blakeross
Created attachment 14362 [details] [diff] [review] better patch - old one worked fine, this just bullet proofs it
Overparenthesization aside, email@example.com. /be
What can I say, I like parenthesizing. There's no such thing as too much parenthesization! fix checked in
Status: NEW → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED
Blake, a= is not the same thing as r=by the module owner. You need both. In the future, give me the opportunity to review. Thanks!
Hyatt: sorry. I had written you about reviewing it, then bryner offered to so he did. Next time I'll wait for your r=.
That's ok. bryner's review is good enough for me... just mention it next time... ;)
vrfy fixed in just-pulled build on win98
Status: RESOLVED → VERIFIED
Component: XP Toolkit/Widgets: Trees → XUL
QA Contact: jrgmorrison → xptoolkit.widgets
You need to log in before you can comment on or make changes to this bug.