Skip to content

Adding outlining for comments to allow collapsing in VS#2612

Merged
ahmad-farid merged 19 commits into
microsoft:masterfrom
ahmad-farid:OCNoTest2
Apr 10, 2015
Merged

Adding outlining for comments to allow collapsing in VS#2612
ahmad-farid merged 19 commits into
microsoft:masterfrom
ahmad-farid:OCNoTest2

Conversation

@ahmad-farid
Copy link
Copy Markdown

#698 Collapsable Comments in VS

While we're walking the file looking for collapsible nodes, check to see if a node has leading comments.
-- If block comments (/*) are available, create a new collapsible item through its span and add it to the elements to be collapsed.
-- For single line comments, create a collapsible item if two or more exist from the start of the first to the end of the last.

@ahmad-farid ahmad-farid added the Visual Studio Integration with Visual Studio label Apr 3, 2015
@msftclas
Copy link
Copy Markdown

msftclas commented Apr 3, 2015

Hi @ahmad-farid, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!


It looks like you're a Microsoft contributor (Ahmad Farid). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

Comment thread src/compiler/scanner.ts Outdated
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

use shorthand properties for hasTrailingNewLine and kind.

@CyrusNajmabadi
Copy link
Copy Markdown
Contributor

Tests?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

only do this if this is a multiline comment. add a named-Comment for 'false'.

Only collapse multiline comment if it spans multiple lines.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Visual Studio Integration with Visual Studio

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants