Skip to content

Adds baseline status to html elements and attributes#202

Merged
aeschli merged 1 commit intomicrosoft:mainfrom
rviscomi:baseline-html
Apr 23, 2025
Merged

Adds baseline status to html elements and attributes#202
aeschli merged 1 commit intomicrosoft:mainfrom
rviscomi:baseline-html

Conversation

@rviscomi
Copy link
Copy Markdown
Contributor

@rviscomi rviscomi commented Apr 8, 2025

Progress on microsoft/vscode-css-languageservice#427

Similar to microsoft/vscode-css-languageservice#428, this displays baseline info in element and attribute hovercards

image image image image

@aeschli
Copy link
Copy Markdown
Collaborator

aeschli commented Apr 23, 2025

Looks good! Thanks a lot @rviscomi !

@aeschli aeschli marked this pull request as ready for review April 23, 2025 13:49
@aeschli aeschli enabled auto-merge (squash) April 23, 2025 13:49
@aeschli aeschli merged commit 31a1168 into microsoft:main Apr 23, 2025
3 checks passed
@rviscomi rviscomi deleted the baseline-html branch April 23, 2025 14:41
"type": "string",
"description": "Date when the feature became newly supported in all major browsers",
"pattern": "^\\d{4}-\\d{2}-\\d{2}$",
"patternErrorMessage": "Date must be in the format of `YYYY-MM-DD`"
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I've analyzed the browsers.html-data.json file from the @vscode/web-custom-data package and found the elements with values like ≤2020-07-28 instead of just 2020-07-28 in the baseline_low_date and baseline_high_date fields.

@aeschli JFYI.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@FluorescentHallucinogen Thanks for looking into this and catching this. Lets file this directly to against https://github.com/microsoft/vscode-custom-data

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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.

4 participants