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
Add support for @document, @-moz-document #137
Conversation
Also add missing tests for VIEWPORT_SYM
+1 |
Add support for @document, @-moz-document
Thanks! |
No problem, happy to help out! Seems like you and the other maintainers have a lot on your plates. |
functions: functions, | ||
prefix: prefix, | ||
line: token.startLine, | ||
col: token.startCol |
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.
line
and col
are wrong: this.fire(...)
should be prepended with token = tokenStream.token();
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.
Nice catch! Want to submit a PR with the fix?
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.
Only if you don't want to.
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.
I figure you caught my mistake, so you deserve the credit for the fix.
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.
Nah, I'm afraid the PR will be stalled for a year, I have a few of the kind, so please, you submit it.
Closes #53.