Skip to content

Conversation

@SevenOutman
Copy link
Contributor

@SevenOutman SevenOutman commented Aug 4, 2023

Fix #186399

Changes

  • Add f1: true option to the creation of "Peek Call Hierarchy" and "Peek Type Hierarchy" actions, which inherits from EditorAction2 class and adds the action to Command Palette.
  • Add name for PeekCallHierarchyAction and PeekTypeHierarchyAction in order to align with other actions defined in src/vs/editor/contrib/gotoSymbol/browser/goToCommands.ts, e.g. PeekDefinitionAction

Additional info

I could already see "Peek Call Hierarchy" action in local Code - OSS with this change. However I still could not see "Peek Type Hierarchy" action because the built-in languages services don't seem to provide a single type hierarchy provider. So I couldn't verify that my edit in PeekTypeHierarchyAction works. Please give me a hint in how to verify it if it's necessary.

image
@jrieken jrieken added this to the August 2023 milestone Aug 7, 2023
@jrieken jrieken merged commit dd4c9da into microsoft:main Aug 7, 2023
@SevenOutman SevenOutman deleted the issue186399 branch August 8, 2023 03:06
@github-actions github-actions bot locked and limited conversation to collaborators Sep 21, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

3 participants