Skip to content

Comments

Remove all Microsoft proprietary features from BCD#17120

Merged
queengooborg merged 1 commit intomdn:mainfrom
queengooborg:ms-proprietary
Aug 10, 2022
Merged

Remove all Microsoft proprietary features from BCD#17120
queengooborg merged 1 commit intomdn:mainfrom
queengooborg:ms-proprietary

Conversation

@queengooborg
Copy link
Contributor

@queengooborg queengooborg commented Jul 25, 2022

This PR removes all of the Microsoft proprietary events, properties, etc. from BCD. They can be removed per our guidelines, as they have been removed from Edge over two years ago and have not been included in a browser version released within the last five years.

@github-actions github-actions bot added data:api Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API data:css Compat data for CSS features. https://developer.mozilla.org/docs/Web/CSS labels Jul 25, 2022
@queengooborg queengooborg mentioned this pull request Jul 25, 2022
Closed
@queengooborg queengooborg added the needs content update This PR needs a corresponding update to mdn/content to update the documentation label Jul 25, 2022
foolip
foolip reviewed Jul 25, 2022
View reviewed changes
Copy link
Contributor

@foolip foolip left a comment

Choose a reason for hiding this comment

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

Are none of the CSS properties variants of standard properties so that we should add them with prefix or alt. name to those?

@queengooborg
Copy link
Contributor Author

queengooborg commented Jul 25, 2022 •
edited
Loading

The -ms- prefixed CSS properties that remain in BCD now are all proprietary. In regards to the Grid properties, Microsoft's implementation of CSS Grids is based upon an old version of the specification, and is mostly incompatible with the CSS grid layout we know today. Rachel has written a great article about this.

@queengooborg
Copy link
Contributor Author

queengooborg commented Aug 10, 2022

Although these should also be removed from the content side, I'm going to go ahead and merge this PR a little early since there is still debate on whether to keep them documented or not -- but if they are kept in MDN content, it would strictly be for the purposes of updating old websites and should not be encouraged for new development.

@queengooborg queengooborg merged commit 0a6e624 into mdn:main Aug 10, 2022
@queengooborg queengooborg deleted the ms-proprietary branch August 10, 2022 05:13
@queengooborg queengooborg removed the needs content update This PR needs a corresponding update to mdn/content to update the documentation label Jul 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@foolip foolip foolip approved these changes

Assignees

No one assigned

Labels

data:api Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API data:css Compat data for CSS features. https://developer.mozilla.org/docs/Web/CSS

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@queengooborg @foolip