-
Notifications
You must be signed in to change notification settings - Fork 30.1k
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
1.13.0 | New Emmet expansion suggestions don't honor emmet.syntaxProfiles setting #28286
Comments
The same with vue,
|
@blukai yea happened to me too 😢 |
Thanks for reporting this. The feature of mapping a non emmet supported language to emmet supported language is not yet implemented in the new emmet model. I have added this to the June plan for emmet. For now, you will have to turn 'html', 'xml', 'xsl', 'jsx', 'js', 'pug', 'slim', 'haml', 'css', 'sass', 'scss', 'less', 'sss', 'stylus' |
With 1809bb2 the This will be out in the coming Wednesday's (21st June) Insiders build |
Can PHP with embedded HTML be added to the supported language list? |
@nanaasumadu Doesnt the below work for you?
|
After enabling the use of This is because the emmet extension is not aware of embedded languages (case of style blocks in vue files or js code in jsx files) and wrong locations (like <? in php) Therefore, its best that we leave the decision to trigger emmet suggestions to language specific extensions who know better. Conclusion:
Thanks to everyone who provided feedback! |
Emmet: Expand Abbreviation has stoped working for php files in the vscode-insiders build 2017-06-26T05:03:31.319Z. It was working fine in the build on 2017-06-23. I still have the following in my settings.json file: "emmet.useNewEmmet": true, |
@nanaasumadu I was pretty happy when I saw your comment on emmet working again for you after re-installing the Insiders. Looks like we are back to square one. The command @jens1o Can you give it a try? |
@ramya-rao-a Yes, it was all working until I install the latest build of vscode-insider yesterday. Is there any way of reverting to the previous build for testing?? I am certain something has changed in the latest build that is causing that. I have tried on two different Macs with the same result. |
@nanaasumadu I don't think we can go back to a previous version of Insiders.
|
Timezones... @ramya-rao-a It works for me: However, it is bound to Tab, but I do not set it to it, nor |
@ramya-rao-a After setting "emmet.useNewEmmet" to "false" and "true" again, Emmet: Expand Abbreviation" works in my php file. As mentioned above, tab completion doesn't work. If I, however, set "emmet.useNewEmmet" to false, tab completion works. Emmet auto complete in php doesn't work now when "emmet.useNewEmmet" is set to true. It was working fine in vscode-insider build on 23/06/17. Any ideas?? |
I think this issue may be what I am experiencing. I didn't realise that emmet suggestions has been disabled for php and responsibility passed on to the language specific extensions. Is there a fallback until the extension authors can make their extensions "emmet" aware to call emmet completion provider instead of the emmet extension itself trying to suggest ? I think if you can provide a user setting such "emmet.useLanguageExtension" which can be set to true or false as fallback will be good. Alternatively, are you able to provide vscode-emmet-helper as a VSIX and provide an option for a user to set which language such as "emmet.autocompleteSuggestion": "php" ? Some of us find the emmet suggestions and completions very useful when working in PHP/HTML mixed environment. |
@jens1o Thanks! We are definitely moving away from binding Tab to Emmet. See https://code.visualstudio.com/updates/v1_13#_emmet-abbreviation-expansion-in-suggestion-list for more on this. @nanaasumadu Let's use #29532 to track the option to allow suggestions to show up when the language is not a pure markup or stylesheet. Meanwhile, please look at #29678 and add your thoughts there. |
@ramya-rao-a
"Emmet: Expand Abbrevation", however works. |
As far as I interpreted the code, this is by design. |
@nanaasumadu Am using #29532 to track the emmet suggestions for languages that are not pure markup or stylesheet |
@ramya-rao-a Sorry about that. I have copied my comments to #29532 for completeness. |
No problem at all |
Steps to Reproduce:
"emmet.useNewEmmet": true,
The text was updated successfully, but these errors were encountered: