Bug 250555 - [Type Hierarchy] shortcut keys - Shift+Ctrl+H not functional in PHP explorer
Summary: [Type Hierarchy] shortcut keys - Shift+Ctrl+H not functional in PHP explorer
Status: CLOSED DUPLICATE of bug 258803
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: PDT (show other bugs)
Version: unspecified   Edit
Hardware: Other All
: P1 normal (vote)
Target Milestone: ---   Edit
Assignee: Q.S. Wang CLA
QA Contact:
URL:
Whiteboard:
Keywords: contributed
Depends on:
Blocks:
 
Reported: 2008-10-12 06:03 EDT by Gadi Goldbarg CLA
Modified: 2020-05-14 11:08 EDT (History)
4 users (show)

See Also:


Attachments
Define the shortcut key in plugin.xml file of DLTK (1000 bytes, patch)
2009-01-07 22:23 EST, Q.S. Wang CLA
no flags Details | Diff
patch (1.23 KB, patch)
2010-01-28 01:51 EST, Zhongwei Zhao CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Gadi Goldbarg CLA 2008-10-12 06:03:54 EDT
When editor is closed - we can't open the dialog with CTRL+Shift+ H

Expected: When a class and below is selected , ctrl+shift+h should open the 
dialog + populate the search field with selected element
Comment 1 Roy Ganor CLA 2008-11-02 07:44:19 EST
targeting to PDT 2.0 M2

Comment 2 Kalin CLA 2008-12-15 07:57:03 EST
The same behavior when editor is opened, the focus is on PHP Explorer.
When the focus is on the relevant element in PHP Editor - correct behavior ('Open Type in hierarchy' dialog is launched using shortcut CTRL+Shift+H)

Tested on pdt-debugger-all-in-one-2.0.0RC1-linux-glibc23-i386
Comment 3 Q.S. Wang CLA 2009-01-07 22:23:05 EST
Created attachment 121906 [details]
Define the shortcut key in plugin.xml file of DLTK
Comment 4 Q.S. Wang CLA 2009-01-07 22:23:53 EST
For fixing this bug, please apply the patch of bug 250554 at first, then apply the patch here.
Comment 5 Michael Spector CLA 2009-01-08 09:22:22 EST
Applied to DLTK.
Comment 6 Gadi Goldbarg CLA 2009-01-30 11:42:56 EST
Still reproducible.
pdt-all-in-one
2.1.0M4
I200901300153

Build Dependencies: DLTK R1.0 1.0M4 200812290249 

Kalin Yanev
kalin.a@zend.com
Comment 7 Zhongwei Zhao CLA 2010-01-28 01:51:55 EST
Created attachment 157496 [details]
patch

This one is same as bug 250554.


Shift+Ctrl+H should be Alt+Ctrl+H
Comment 8 Q.S. Wang CLA 2010-03-15 03:37:45 EDT
Applied the patch.Thanks for contribution.
Comment 9 Q.S. Wang CLA 2010-03-15 04:31:24 EDT
Reopen it. It should not use the DLTK action directly.
Comment 10 Q.S. Wang CLA 2010-03-15 20:49:29 EDT
The issues here are:
1. DLTK contributes a set of open* actions(retarget actions), however it only contribute the key binding to the editor scope.
2. PHP explorer extends from DLTK script explorer, which gets those open* action on PHP explorer. However since the PHPExplorer needs define it's own context, and DLTK doesn't provides its own windows context, the key binding will not be activated then.  That's why this bug was resolved but re-opened later.

Solution: change the definition of PHP open* actions to non-retarget. Specify the class to a PHP local action, which can be a delegator to the DLTK actions. 
Define local key bindings. 
This should resolve the issue of PHPExplorer.

3. There's issues of contributing the open* actions to navigate group when the editor gets the focus. 
Solution: define the action set association between the open* action set and the PHP editor. 
Define local key bindings and etc. Reuse the action delegators which has been defined in step 2 above.

Using this solution, all the actions are manged and contributed by PHP consistently. 

These solutions are for 250554 and 253193 as well.


Zhongwei, would you want to make a try? Let me know if you have further questions.
Comment 11 Zhongwei Zhao CLA 2010-04-03 06:54:45 EDT
fixed in head,close
Comment 12 Gadi Goldbarg CLA 2010-05-04 10:04:04 EDT
Hi Zhongwei,
I think your suggestion is wrong.
The associated shortcuts should be as follows:
Open Call Hierarchy = Ctrl+Alt++H
Open Type Hierarchy = F4
Open Type in Hierarchy = Ctrl+Shift+H

Look at Key Assist Table by Ctrl+Shift+L for more info.

The subject of the issue is related to 'Open Type in Hierarchy' shortcut functionality.
The problem is still reproducible. Shift+Ctrl+H takes no effect when the focus in on a class in PHP Explorer view.
Tested in EclipsePdt-2.2.0.v20100427
Kalin Yanev
kalin.a@zend.com
Comment 13 Zhongwei Zhao CLA 2010-05-04 20:25:36 EDT
now Shift+Ctrl+H works,but not works as expected,there is another bug report the wrong behavior of Shift+Ctrl+H for PHP Explorer view(but I forget the number,and I talked with Kalin about that bug) which need to commit the patch to dltk,I have told Roy about that.
Comment 14 Zhongwei Zhao CLA 2010-05-04 20:28:27 EDT
The above bug I said is Bug 258803,so I mark this bug fixed,because Shift+Ctrl+H works now
Comment 15 Gadi Goldbarg CLA 2010-05-17 03:59:05 EDT
I close it as a Duplicate of Bug 258803
Zhao, thanks for contribution.

Kalin Yanev
kalin.a@zend.com

*** This bug has been marked as a duplicate of bug 258803 ***