Bugzilla – Full Text Bug Listing
| Summary: | [Tree] ENTER does not notify widget default selected | ||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | [RT] RAP | Reporter: | Elias Volanakis <elias> | ||||||||
| Component: | RWT | Assignee: | Project Inbox <rap-inbox> | ||||||||
| Status: | RESOLVED FIXED | QA Contact: | |||||||||
| Severity: | normal | ||||||||||
| Priority: | P3 | CC: | ivan, nobody | ||||||||
| Version: | 1.2 | ||||||||||
| Target Milestone: | 1.3 M1 | ||||||||||
| Hardware: | All | ||||||||||
| OS: | All | ||||||||||
| Whiteboard: | |||||||||||
| Attachments: |
|
||||||||||
Created attachment 139016 [details]
Patch that enable widgetDefaultSelected event on ENTER key.
Small change in the Controls Demo TreeTab is included too.
Thanks Ivan! Works well. The patch causes the tree to show empty nodes occasionally. Jordan, is it possible to provide a snippet? Actually this issue should be fixed. I am trying to reproduce another bug that has obviously nothing to do with this one. Jordan,
not sure what you mean with the previous comment.
> The patch causes the tree to show empty nodes occasionally.
Is this still an issue? Please clarify.
Thanks,
Elias.
Morning Elias, From my point of view this bug is fixed. The empty nodes are possibly another bug, that I am trying to reproduce right now, but the ENTER key works fine, so please resolve here. PS: It is not that I havent found anything... :-) Have a look under https://bugs.eclipse.org/bugs/show_bug.cgi?id=281534 Greetings Jordan One note: On Windows the ENTER key does not expand/collapse the tree folder. On Linux it does. Let's stay with Windows. Fire a widgetDefaultSelected but don't collapse/expand the tree item. Once the default-selection event is fired, it is still possible to expand/collapse the item from the application code. Created attachment 140213 [details]
Updated patch to respect Windows behavior about expand/collapse on ENTER key.
The recent patch fixes the 'expand' state but disables the notification of SelectionListener.widgetDefaultSelected (which was the mean reason for this issue). Please provide an update. Elias, I've just check it and the SelectionListener.widgetDefaultSelected is fired as expected with latest patch. Please provide more information. Ok, it was my fault the first time. I reverted to the original file and reapplied the patch. It works! :-) Changes are in CVS HEAD. |
Created attachment 138948 [details] Snippet to reproduce bug Issue: Hitting ENTER on a TreeItem will not notify SelectionListener.widgetDefaultSelected(e). Note that this works normally on the TableItem. Looks like a bug to me. Use attached snippet to reproduce.