-
Notifications
You must be signed in to change notification settings - Fork 30.3k
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
Stylesheet partial parse #46376
Stylesheet partial parse #46376
Changes from 3 commits
787904e
8f796cb
dea5096
55a4ed3
e7dc7f8
e9089d4
f0c6efa
ae81a1d
7b4d979
3536668
a7a1b17
0dddad3
39fb6a8
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -128,6 +128,50 @@ export function parseDocument(document: vscode.TextDocument, showError: boolean | |
return undefined; | ||
} | ||
|
||
export function parsePartialStylesheet(document: vscode.TextDocument, position: vscode.Position): Node | undefined { | ||
|
||
let startPosition = new vscode.Position(0, 0); | ||
let endPosition = new vscode.Position(document.lineCount - 1, document.lineAt(document.lineCount - 1).text.length); | ||
const closeBrace = 125; | ||
const openBrace = 123; | ||
|
||
// Go forward until we found a closing or opening brace | ||
let stream = new DocumentStreamReader(document, position); | ||
stream.eatWhile(char => { return char !== closeBrace && char !== openBrace; }); | ||
stream.eat(closeBrace); | ||
endPosition = stream.pos; | ||
|
||
// Go back until we found an opening brace. If we find a closing one, we first find its opening brace and then we continue. | ||
stream.pos = position; | ||
let openBracesRemaining = 1; | ||
while (openBracesRemaining > 0 && !stream.sof()) { | ||
let ch = stream.backUp(1); | ||
if (ch === openBrace) { | ||
openBracesRemaining--; | ||
} else if (ch === closeBrace) { | ||
if (document.languageId !== 'css') { | ||
openBracesRemaining++; | ||
} | ||
if (position.line - stream.pos.line > 1000) { | ||
return parseStylesheet(new DocumentStreamReader(document, new vscode.Position(0, 0))); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This is missing the range to be passed There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Also, move this code to the beginning of the while loop. No reason to have it inside the if/else correct? |
||
} | ||
} | ||
} | ||
// We are at an opening brace. We need to include its selector | ||
while (stream.pos.character > 0) { | ||
let ch = stream.backUp(1); | ||
if (ch === closeBrace || ch === openBrace) { | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. we don't really need the whole selector, we can stop when we find a non whitespace as well |
||
stream.next(); | ||
break; | ||
} | ||
} | ||
startPosition = stream.pos; | ||
try { | ||
return parseStylesheet(new DocumentStreamReader(document, startPosition, new vscode.Range(startPosition, endPosition))); | ||
} catch (e) { | ||
} | ||
} | ||
|
||
/** | ||
* Returns node corresponding to given position in the given root node | ||
*/ | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If this is css, then we can set startPosition and return from here