Community
Participate
Working Groups
Build Identifier: I20100602-1502 If we drag something from Snippet or Palette view to Source editor, focus does not change to source editor. Reproducible: Always Steps to Reproduce: 1.Drag a widget from palette to source editor. 2.Focus remains at Palette view, Focus should have changed to Editor.
Created attachment 173241 [details] Setting focus after drop This patch sets the focus to the target editor after drop action.
Thanks, Sarika. This patch looks good to me. I think this can alleviate some user frustration when it comes to adding DND content to our editors. Nitin, would you mind taking a look at this for 3.2.1?
How well does this work both in editors that are just the StructuredTextEditor and cases where it's a Source page? Since it activates the editor, what about cases where the file's out of sync with the workspace?
Created attachment 173315 [details] patch Checks for parent editor parts for cases like the XML editor.
Approved. Please write-up the PMC review details.
* Explain why you believe this is a stop-ship defect. Or, if it is a "hotbug" (requested by an adopter) please document it as such. Needed for an adopter as a usability issue. Context is never switched back over to the editor when something is dragged onto it, which can prevent immediate typing without the user manually focusing the editor again. * Is there a work-around? If so, why do you believe the work-around is insufficient? No. * How has the fix been tested? Is there a test case attached to the bugzilla record? Has a JUnit Test been added? The existing unit tests have been run along with ad hoc testing of drag and drop features on various SSE-based editors. Give a brief technical overview. Who has reviewed this fix? Once an item has been dropped onto the editor, the drop target adapter for Structured Source Editors will now activate the editor part that the item was dropped on. This will allow the user to immediately begin typing into the source of the editor again. It's a small update to our implementation of the drop target adapter. Nitin and I have both reviewed the patch. * What is the risk associated with this fix? Very low.
Released to Maintenance and HEAD.
