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

Make all multiline statements collapsible #35792

Closed
VBAndCs opened this issue May 17, 2019 · 2 comments
Closed

Make all multiline statements collapsible #35792

VBAndCs opened this issue May 17, 2019 · 2 comments

Comments

@VBAndCs
Copy link

@VBAndCs VBAndCs commented May 17, 2019

I noticed that this statement is collapsible in the editor:

var multiline = string.Join("\r\n", new string[] {
                       "Class Test",
                        "     Dim x As Integer",
                        "     Dim Y As Integer",
                        "End Class"});

While this is not:

var multiline = string.Join("\r\n", 
                       "Class Test",
                        "     Dim x As Integer",
                        "     Dim Y As Integer",
                        "End Class");

I think any multiline statement (in VB and C#) should be collapsible, whether or not it contains {}.

@jinujoseph

This comment has been minimized.

Copy link
Contributor

@jinujoseph jinujoseph commented Jul 1, 2019

Design Meeting Notes
Today you can make any selection and do Edit - > Outline -> Hide selection Ctrl+H Ctrl+M
We are afraid that by making everything collapsible, it will add too many noises to the UI.

@jinujoseph jinujoseph moved this from In Queue to Next meeting (priority) in IDE: Design review Jul 1, 2019
@jinujoseph jinujoseph moved this from Next meeting (priority) to Complete in IDE: Design review Jul 1, 2019
@CyrusNajmabadi

This comment has been minimized.

Copy link
Contributor

@CyrusNajmabadi CyrusNajmabadi commented Mar 26, 2020

Closing out as per design review conclusion. There are ample nesting locations offered in the IDE already, and we haven't heard about this from any other sources. This will likely make things too noisy.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
IDE: Design review
  
Complete
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.