Skip to content

Conversation

@Elchi3
Copy link
Collaborator

@Elchi3 Elchi3 commented Oct 11, 2024

Followup to #1940
I should have checked the draft folder and also search for "math" in BCD keys. Did that now.

@github-actions github-actions bot added the feature definition Creating or defining new features or groups of features. label Oct 11, 2024
jamesnw
jamesnw previously approved these changes Oct 11, 2024
Copy link
Collaborator

@jamesnw jamesnw left a comment

Choose a reason for hiding this comment

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

This looks good to me, and I agree the CSS properties belong here.

@jamesnw jamesnw dismissed their stale review October 11, 2024 18:29

Realized I still have questions

@jamesnw
Copy link
Collaborator

jamesnw commented Oct 11, 2024

After approval, I realized that the draft spec has a lot of keys that were removed, but not added to the feature, for instance mathml.elements.mspace. Should they be added as well?

@Elchi3
Copy link
Collaborator Author

Elchi3 commented Oct 14, 2024

hm, as I see it mspace is in the mathml.yml feature. I added that in #1940.

Copy link
Collaborator

@jamesnw jamesnw left a comment

Choose a reason for hiding this comment

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

Whoops- not sure what I was looking at where those were missing- sorry! Looks good!

@ddbeck ddbeck merged commit e9c00b4 into web-platform-dx:main Oct 15, 2024
3 checks passed
@Elchi3 Elchi3 deleted the more-math branch October 15, 2024 12:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature definition Creating or defining new features or groups of features.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants