Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

HTML figcaption tag should be suggested when using the full word instead of only "figc" #114923

Closed
Youssef1313 opened this issue Jan 25, 2021 · 6 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug emmet Emmet related issues good first issue Issues identified as good for first-time contributors insiders-released Patch has been released in VS Code Insiders verified Verification succeeded
Milestone

Comments

@Youssef1313
Copy link
Member

Youssef1313 commented Jan 25, 2021

  • VSCode Version: 1.52.1 commit ea3859d
  • OS Version: Windows 10 1909

Steps to Reproduce:

  1. In an html file, write figure and press TAB. You'll see the following behavior, which is correct.

    image

    image

  2. Try the same with figcaption, it doesn't work.

    image

  3. Try again with only figc, it works.

    image

  4. The same issue with time tag, but VSCode's Emmet completely doesn't recognize time.

Expected behavior:

The suggestion should show using the full tag figcaption.

@Youssef1313
Copy link
Member Author

cc: @rzhao271

@rzhao271 rzhao271 assigned rzhao271 and unassigned aeschli Jan 25, 2021
@rzhao271 rzhao271 added bug Issue identified by VS Code Team member as probable bug emmet Emmet related issues labels Jan 25, 2021
@rzhao271 rzhao271 added this to the Backlog milestone Jan 25, 2021
@AnuBhar
Copy link

AnuBhar commented Feb 5, 2021

I am starting to look into this issue. Have been able to repro it. figcaption does not have any suggestions, only figc does.

@rzhao271 rzhao271 added the good first issue Issues identified as good for first-time contributors label Feb 5, 2021
@rzhao271
Copy link
Contributor

rzhao271 commented Feb 5, 2021

@AnuBhar

Code pointer:
https://github.com/microsoft/vscode-emmet-helper/blob/main/src/emmetHelper.ts#L523

We need to add figcaption to commonlyUsedTags. Otherwise the expanded abbreviation gets marked as noise and is thrown out.

@Youssef1313
Copy link
Member Author

@rzhao271 That should go in https://github.com/microsoft/vscode-emmet-helper/blob/main/src/data.ts?
I don't see "figc" there, so confused a little bit how "figc" works today.

@rzhao271
Copy link
Contributor

rzhao271 commented Feb 5, 2021

@Youssef1313 'figc' gets recognized as a snippet here instead
https://github.com/emmetio/emmet/blob/master/snippets/html.json#L110
It's part of markupSnippetKeys rather than commonlyUsedTags.

@rzhao271 rzhao271 added hackathon Label reserved for specific events and removed hackathon Label reserved for specific events labels Feb 8, 2021
@rzhao271 rzhao271 modified the milestones: Backlog, February 2021 Feb 8, 2021
@rzhao271
Copy link
Contributor

\closedWith 7661c8e

@aeschli aeschli added the verified Verification succeeded label Feb 25, 2021
@github-actions github-actions bot locked and limited conversation to collaborators Mar 29, 2021
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
bug Issue identified by VS Code Team member as probable bug emmet Emmet related issues good first issue Issues identified as good for first-time contributors insiders-released Patch has been released in VS Code Insiders verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

5 participants
@aeschli @rzhao271 @AnuBhar @Youssef1313 and others