| Summary: | Visual Editor: Press <Delete> key in the graph viewer has an impact on the source viewer. | ||||||
|---|---|---|---|---|---|---|---|
| Product: | z_Archived | Reporter: | Suwit Srivilairith <suwits> | ||||
| Component: | VE | Assignee: | Gili Mendel <gmendel> | ||||
| Status: | CLOSED DUPLICATE | QA Contact: | |||||
| Severity: | major | ||||||
| Priority: | P3 | CC: | camle | ||||
| Version: | unspecified | ||||||
| Target Milestone: | --- | ||||||
| Hardware: | PC | ||||||
| OS: | Windows XP | ||||||
| Whiteboard: | |||||||
| Attachments: |
|
||||||
|
Description
Suwit Srivilairith
Created attachment 9244 [details] Bug 57551 screen capture The current action handler for the delete global action is getting run. This prevents any control from receiving the delete key. *** This bug has been marked as a duplicate of 58899 *** GEF will be setting doit==false when DEL key is pressed. This will comply to the new 3.0 behavior. However, VE should be re-mapping the global action handler based on which control has focus. At any time the user may either press DEL key, or use the Edit menu and select Delete. The behavior *should* be the same. Gili... I think this is related to the other defect you fixed w.r.t. using the delete key for removing a component in the graph viewer. This problem still occurs for this case (i.e. delete characters in direct edit mode). I had put a work around in Bug 56840 to hack into the (DEL) command .... Rich/Randy note that there is a formal manner to change scope on the fly, it needs to be done, and will use 56840 to complete this properly. In any case the reason the workaround is not working in the direct edit case, is that the GraphicalEdit part loses focus (and hence re-enable the JavaEditor's DEL action). The (GEF) Direct edit canvas itself have to set doit=false so that the DEL key that was sent to it, is not pass on forward. *** This bug has been marked as a duplicate of 59543 *** M1 released, move all resolved to Verified, as prelude to closing. M1 released, move all resolved/verified to Closed. |