Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Extract Method does not appear on the editor context menu #597

Closed
zooba opened this issue Jul 14, 2015 · 2 comments · Fixed by #1226
Closed

Extract Method does not appear on the editor context menu #597

zooba opened this issue Jul 14, 2015 · 2 comments · Fixed by #1226
Assignees
Labels
Milestone

Comments

@zooba
Copy link
Member

zooba commented Jul 14, 2015

(Definitely VS 2015, haven't checked VS 2013 but I doubt it's changed there.)

The Extract Method command is no longer visible on the editor context menu.

It appears under Edit -> Refactor, and the keyboard shortcut still works. Users can use Tools -> Customize to add the item to Editor Context Menus | Code Window - "Extract Method" is found under Edit in "Add Command".

@zooba zooba added this to the 2.2.1 milestone Oct 22, 2015
@zooba zooba modified the milestones: 2.2.1, 2.2.3, 3.0 Jan 13, 2016
@zooba
Copy link
Member Author

zooba commented Jan 15, 2016

Moving to 3.0 - this does not meet the bar for 2.2 updates any more.

@DinoV
Copy link
Contributor

DinoV commented Apr 12, 2016

C# has moved this to Quick Actions

@DinoV DinoV self-assigned this Apr 12, 2016
DinoV pushed a commit to DinoV/PTVS that referenced this issue Apr 15, 2016
DinoV pushed a commit to DinoV/PTVS that referenced this issue Apr 15, 2016
Add a Extract Method suggested action and include a preview of the new
change

Add a PreviewChangesService for creating a preview text view based upon a
set of changes

Fix microsoft#186
 Special case checking for REPL buffers that start with a $ and don't
 provide quick info. Unfortunately we have to do that because the REPL window
 input kind doesn't change until we have a complete input

 Fix microsoft#792
 Clean up our BufferParser/AnalysisEntry/TextView tracking so that we can
 support this.  We no longer attach any properties to text buffers at all.
 Instead we always resolve from textview and/or filename into a
 ProjectAnalyzer and then get the buffer from there.

 We now flow TextView's where they're available into our analyzer.
 There's only really 3 spots where we don't have that full context: Outlining
 regions and proximity info via the old IVsLanguageInfo interfaces.

 Add support for CommonPackage.GetActiveTextView so that we can find the
 correct active text view for diff windows. This is used all over the
 place to find the text view when we're doing various editing commands like
 extract method, find all refs, goto def, etc...

 Update Extensions to add various helpers which deal with the caret
 within a projection buffer.
@DinoV DinoV mentioned this issue Apr 15, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants