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

fix: mark successive dots as noise and invalidate jinja syntax abbreviations #80

Merged

Conversation

iifawzi
Copy link
Contributor

@iifawzi iifawzi commented May 31, 2023

Hi, this PR should fix microsoft/vscode#179422
let me know if it could be done in any better way

Signed-off-by: Fawzi Abdulfattah <iifawzie@gmail.com>
Signed-off-by: Fawzi Abdulfattah <iifawzie@gmail.com>
…date

Signed-off-by: Fawzi Abdulfattah <iifawzie@gmail.com>
@iifawzi iifawzi force-pushed the fix/jinga-successive--abbreviations branch from 39c430e to 3bcd26e Compare May 31, 2023 21:44
@iifawzi
Copy link
Contributor Author

iifawzi commented Jun 7, 2023

Hi @rzhao271, would appreciate if you can take a look whenever you have a time

Copy link
Contributor

@rzhao271 rzhao271 left a comment

Choose a reason for hiding this comment

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

Thanks for the PR! I have added some comments

src/emmetHelper.ts Outdated Show resolved Hide resolved
src/emmetHelper.ts Outdated Show resolved Hide resolved
src/test/expand.test.ts Show resolved Hide resolved
… successive dots

Signed-off-by: Fawzi Abdulfattah <iifawzie@gmail.com>
Signed-off-by: Fawzi Abdulfattah <iifawzie@gmail.com>
Signed-off-by: Fawzi Abdulfattah <iifawzie@gmail.com>
@iifawzi
Copy link
Contributor Author

iifawzi commented Jun 7, 2023

Pushed the changes https://github.com/iifawzi/vscode-emmet-helper/tree/fix/jinga-successive--abbreviations, but for some reason they didn't reflect here. That's weird, I've never encountered this.

EDIT: ok it seems like there's an outage, https://www.githubstatus.com/

Signed-off-by: Fawzi Abdulfattah <iifawzie@gmail.com>
@iifawzi iifawzi force-pushed the fix/jinga-successive--abbreviations branch from be23333 to cb173b7 Compare June 7, 2023 17:56
@rzhao271 rzhao271 self-assigned this Jun 7, 2023
@rzhao271 rzhao271 added this to the June 2023 milestone Jun 7, 2023
@rzhao271 rzhao271 merged commit 64ecc72 into microsoft:main Jun 7, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Emmet suggestions produces incorrect Jinja syntax
3 participants