Skip to content

Comments

Remove documentation about Microsoft Extensions#18750

Closed
queengooborg wants to merge 5 commits intomdn:mainfrom
queengooborg:ms-extensions
Closed

Remove documentation about Microsoft Extensions#18750
queengooborg wants to merge 5 commits intomdn:mainfrom
queengooborg:ms-extensions

Conversation

@queengooborg
Copy link
Collaborator

This PR removes all of the documentation regarding Microsoft proprietary events, properties, etc. from MDN Web Docs. They can be removed per BCD's irrelevance guidelines, since they have been removed from Edge over two years ago, and have not been included in a browser version that has been released within the last five years. Related BCD PR: mdn/browser-compat-data#17120

@queengooborg queengooborg requested review from a team as code owners July 25, 2022 02:59
@queengooborg queengooborg requested review from Rumyra, estelle, jpmedley and schalkneethling and removed request for a team July 25, 2022 02:59
@github-actions github-actions bot added Content:Accessibility Accessibility docs Content:CSS Cascading Style Sheets docs Content:HTML Hypertext Markup Language docs Content:Other Any docs not covered by another "Content:" label Content:WebAPI Web API docs labels Jul 25, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Jul 25, 2022

Preview URLs (14 pages)
Flaws (268)

Note! 3 documents with no flaws that don't need to be listed. 🎉

URL: /en-US/docs/Web/CSS/WebKit_Extensions
Title: WebKit CSS extensions
Flaw count: 92

  • macros:
    • /en-US/docs/Web/CSS/-webkit-animation-trigger does not exist
    • /en-US/docs/Web/CSS/-webkit-app-region does not exist
    • /en-US/docs/Web/CSS/-webkit-aspect-ratio does not exist
    • /en-US/docs/Web/CSS/-webkit-backdrop-filter does not exist
    • /en-US/docs/Web/CSS/-webkit-border-horizontal-spacing does not exist
    • and 87 more flaws omitted

URL: /en-US/docs/Web/CSS/Mozilla_Extensions
Title: Mozilla CSS extensions
Flaw count: 97

  • macros:
    • /en-US/docs/Web/CSS/-moz-binding does not exist
    • /en-US/docs/Web/CSS/-moz-border-bottom-colors does not exist
    • /en-US/docs/Web/CSS/-moz-border-left-colors redirects to /en-US/docs/Web/CSS/Mozilla_Extensions#b
    • /en-US/docs/Web/CSS/-moz-border-right-colors does not exist
    • /en-US/docs/Web/CSS/-moz-border-top-colors does not exist
    • and 90 more flaws omitted
  • broken_links:
    • Can't resolve /en-US/docs/Mozilla/Tech/XUL
    • Can't resolve /en-US/docs/Mozilla/Gecko/Chrome/CSS

URL: /en-US/docs/Web/CSS/Reference
Title: CSS reference
Flaw count: 14

  • macros:
    • wrong xref macro used (consider changing which macro you use)
    • wrong xref macro used (consider changing which macro you use)
    • wrong xref macro used (consider changing which macro you use)
    • wrong xref macro used (consider changing which macro you use)
    • wrong xref macro used (consider changing which macro you use)
    • and 8 more flaws omitted
  • broken_links:
    • Can't resolve /en-US/docs/Glossary/Descriptor_(CSS)

URL: /en-US/docs/Web/CSS/Media_Queries/Using_Media_Queries_for_Accessibility
Title: Using media queries for accessibility
Flaw count: 1

  • broken_links:
    • Can't resolve /en-US/docs/Web/CSS/@media/

URL: /en-US/docs/Web/Events
Title: Event reference
Flaw count: 51

  • broken_links:
    • Can't resolve /en-US/docs/Web/API/Document/wheel_event
    • Can't resolve /en-US/docs/Web/API/Document/animationcancel_event
    • Can't resolve /en-US/docs/Web/API/Document/animationend_event
    • Can't resolve /en-US/docs/Web/API/Document/animationiteration_event
    • Can't resolve /en-US/docs/Web/API/Document/animationstart_event
    • and 46 more flaws omitted

URL: /en-US/docs/Web/API/Element/gesturestart_event
Title: Element: gesturestart event
Flaw count: 2

  • macros:
    • /en-US/docs/Web/API/GestureEvent/rotation does not exist
    • /en-US/docs/Web/API/GestureEvent/scale does not exist

URL: /en-US/docs/Web/API/Element/gesturechange_event
Title: Element: gesturechange event
Flaw count: 2

  • macros:
    • /en-US/docs/Web/API/GestureEvent/rotation does not exist
    • /en-US/docs/Web/API/GestureEvent/scale does not exist

URL: /en-US/docs/Web/API/Element/gestureend_event
Title: Element: gestureend event
Flaw count: 2

  • macros:
    • /en-US/docs/Web/API/GestureEvent/rotation does not exist
    • /en-US/docs/Web/API/GestureEvent/scale does not exist

URL: /en-US/docs/Web/API/PointerEvent
Title: PointerEvent
Flaw count: 2

  • macros:
    • /en-US/docs/Web/API/PointerEvent/getPredictedEvents does not exist
    • /en-US/docs/Web/API/HTMLElement/pointerrawupdate_event does not exist

URL: /en-US/docs/Web/API/TouchEvent
Title: TouchEvent
Flaw count: 2

  • macros:
    • /en-US/docs/Web/API/TouchEvent/rotation does not exist
    • /en-US/docs/Web/API/TouchEvent/scale does not exist

URL: /en-US/docs/Web/API/GestureEvent
Title: GestureEvent
Flaw count: 3

  • macros:
    • /en-US/docs/Web/API/GestureEvent/rotation does not exist
    • /en-US/docs/Web/API/GestureEvent/scale does not exist
    • /en-US/docs/Web/API/GestureEvent/initGestureEvent does not exist
External URLs (16)

URL: /en-US/docs/Web/CSS/WebKit_Extensions
Title: WebKit CSS extensions


URL: /en-US/docs/Web/CSS/Mozilla_Extensions
Title: Mozilla CSS extensions


URL: /en-US/docs/Web/CSS/Reference
Title: CSS reference


URL: /en-US/docs/Web/CSS/Media_Queries/Using_Media_Queries_for_Accessibility
Title: Using media queries for accessibility


URL: /en-US/docs/Web/API/Element/gesturestart_event
Title: Element: gesturestart event


URL: /en-US/docs/Web/API/Element/gesturechange_event
Title: Element: gesturechange event


URL: /en-US/docs/Web/API/Element/gestureend_event
Title: Element: gestureend event


URL: /en-US/docs/Web/API/PointerEvent
Title: PointerEvent


URL: /en-US/docs/Web/API/TouchEvent
Title: TouchEvent


URL: /en-US/docs/Web/API/GestureEvent
Title: GestureEvent

(comment last updated: 2023-02-13 13:54:03)

@jpmedley
Copy link
Collaborator

I must decline the request to review this. I don't know anything about MS extensions.

@queengooborg
Copy link
Collaborator Author

No worries, @jpmedley! This is something that should probably be discussed during an MDN content planning meeting first anyways, before official review!

@queengooborg queengooborg marked this pull request as draft July 25, 2022 17:19
@queengooborg
Copy link
Collaborator Author

queengooborg commented Jul 25, 2022

I've converted this to a draft to make sure it is not merged before I add redirects where applicable, as requested during the MDN content planning meeting earlier today!

Edit: there are no applicable redirects I can create, so I have not created any.

Copy link
Member

@estelle estelle left a comment

Choose a reason for hiding this comment

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

I would like to keep Microsoft CSS Extensions around until IE is completely not supported. It's the only place that has a somewhat complete list and would prefer not to remove the resource until IE really is completely gone.

@queengooborg
Copy link
Collaborator Author

I would like to keep Microsoft CSS Extensions around until IE is completely not supported. It's the only place that has a somewhat complete list and would prefer not to remove the resource until IE really is completely gone.

Unfortunately, I don't think that IE will ever be completely gone, or at least, not until 2029 due to Edge's "IE mode", and is still supported in enterprise editions of Windows 10. Windows 10 itself will be EOL in 2025 for all editions.

I personally feel that documenting Internet Explorer-specific features will encourage developers to continue writing IE-compatible websites, which encourages continued use of IE, despite the fact it contains numerous security holes that will never be patched. At some point, we've got to rip this bandaid off, but I don't think we should wait until 2025 or 2029 to do it.

@Rumyra Rumyra removed the request for review from jpmedley July 27, 2022 11:33
@Elchi3
Copy link
Member

Elchi3 commented Sep 1, 2022

This PR is too big.

I think we might have consensus on a few bits here (I was about to propose to remove the MS gesture event pages). Maybe other features (CSS properties) should stay for a bit still. We need smaller PRs to discuss in more detail (as opposed to "remove everything").

Closing.

@Elchi3 Elchi3 closed this Sep 1, 2022
@queengooborg queengooborg reopened this Sep 1, 2022
@queengooborg queengooborg marked this pull request as draft September 1, 2022 20:54
@queengooborg
Copy link
Collaborator Author

I've reopened the PR but marked it as a draft instead, to show what needs to be done to remove this category.

@teoli2003
Copy link
Contributor

@schalkneethling Can you move this to a discussion?

@queengooborg
Copy link
Collaborator Author

Pull requests can't be converted into discussions, but a new discussion can be created!

@teoli2003
Copy link
Contributor

teoli2003 commented Sep 1, 2022

Let's close this then. It will be easier to create new PRs than to merge main into this every 3 days for the next 6 months.

@queengooborg
Copy link
Collaborator Author

Can we leave it open as a draft for the time being, please?

Copy link
Contributor

@teoli2003 teoli2003 left a comment

Choose a reason for hiding this comment

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

We can keep this as a draft if you want.

I don't think there is an agreement to merge this. We want to remove old IE data, but there is no consensus about when and how.

As @Elchi3 said, likely, some pages here can already be removed, but others not. As long as nobody has looked at each in detail and has demonstrated a consensus, we should not merge this.

The properties, methods, and attributes listed here are heterogenous and have different uses. We decided at the end of last Quarter (when we added the page-type YAML header) to keep them as it was unclear how useful they were. We really hesitated, but we decided not to force the move without a proper discussion, as there was no urgency.

I'm interested in seeing the reasons for removing the different groups of features. Example of points to discuss (there are likely more): are some of these features already deprecated in IE11, and superseded by newer features in that browser? In these cases, we should delete them. For other cases, for features without replacement, I think we should keep them for the time being.

I'm open to discussing all this, and I don't think this PR is appropriate as we don't know how we will proceed.

@teoli2003
Copy link
Contributor

There are many other questions to answer here:

  • Are these pages often visited, or never?
  • Should we remove the internal links to them, but keep them accessible via search engines?

@Elchi3
Copy link
Member

Elchi3 commented Sep 2, 2022

Thanks for doing the work, Vinyl! I'm happy to look at this PR in any state. Just wanted to set expectations that I don't think this PR is mergable as is.

For actual removal it is just better to decide on the individual topics / feature sets. It's just too hard to decide on all of them at once. Also, maybe we want to set redirects and that is easier to do in smaller PRs, too.

Example PR: Blob APIs: #20245

@Rumyra
Copy link
Collaborator

Rumyra commented Sep 2, 2022

Hey folks! We discussed this and we're going to do the following:

  • A discussion is now open here where we can talk about the removal and conclude what action to take there
  • Smaller prs might be a good way to go, but we'll keep this pr in draft until whatever conclusion comes from discussion

@github-actions
Copy link
Contributor

github-actions bot commented Nov 7, 2022

This pull request has merge conflicts that must be resolved before it can be merged.

Conflicts:
	files/en-us/_redirects.txt
	files/en-us/_wikihistory.json
	files/en-us/web/api/cssstyledeclaration/msgetpropertyenabled/index.md
	files/en-us/web/api/cssstyledeclaration/msputpropertyenabled/index.md
	files/en-us/web/api/document/mssitemodejumplistitemremoved_event/index.md
	files/en-us/web/api/document/msthumbnailclick_event/index.md
	files/en-us/web/api/element/msgesturechange_event/index.md
	files/en-us/web/api/element/msgestureend_event/index.md
	files/en-us/web/api/element/msgesturehold_event/index.md
	files/en-us/web/api/element/msgesturestart_event/index.md
	files/en-us/web/api/element/msgesturetap_event/index.md
	files/en-us/web/api/element/msgetregioncontent/index.md
	files/en-us/web/api/element/msinertiastart_event/index.md
	files/en-us/web/api/element/msmanipulationstatechanged_event/index.md
	files/en-us/web/api/element/msregionoverflow/index.md
	files/en-us/web/api/element/mszoomto/index.md
	files/en-us/web/api/htmlaudioelement/msaudiodevicetype/index.md
	files/en-us/web/api/htmlelement/mscandidatewindowhide_event/index.md
	files/en-us/web/api/htmlelement/mscandidatewindowshow_event/index.md
	files/en-us/web/api/htmlelement/mscandidatewindowupdate_event/index.md
	files/en-us/web/api/htmlmediaelement/mscleareffects/index.md
	files/en-us/web/api/htmlmediaelement/msinsertaudioeffect/index.md
	files/en-us/web/api/htmlvideoelement/index.md
	files/en-us/web/api/microsoft_extensions/index.md
	files/en-us/web/api/msgestureevent/index.md
	files/en-us/web/api/msmanipulationevent/index.md
	files/en-us/web/api/msmanipulationevent/initmsmanipulationevent/index.md
	files/en-us/web/api/mssitemodeevent/index.md
	files/en-us/web/api/mswriteprofilermark/index.md
	files/en-us/web/api/navigator/mslaunchuri/index.md
	files/en-us/web/api/touch/msmanipulationviewsenabled/index.md
	files/en-us/web/api/xmlhttprequest/mscaching/index.md
	files/en-us/web/api/xmlhttprequest/mscachingenabled/index.md
	files/en-us/web/css/@media/-ms-high-contrast/index.md
	files/en-us/web/css/media_queries/using_media_queries_for_accessibility/index.md
	files/en-us/web/css/microsoft_extensions/index.md
	files/en-us/web/events/index.md
	files/en-us/web/html/global_attributes/x-ms-acceleratorkey/index.md
	files/en-us/web/html/global_attributes/x-ms-format-detection/index.md
@github-actions
Copy link
Contributor

This pull request has merge conflicts that must be resolved before it can be merged.

@github-actions
Copy link
Contributor

github-actions bot commented Mar 3, 2023

This pull request has merge conflicts that must be resolved before it can be merged.

@queengooborg
Copy link
Collaborator Author

This removal project has now been taken care of in other PRs!

@queengooborg queengooborg deleted the ms-extensions branch March 13, 2023 00:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Content:CSS Cascading Style Sheets docs Content:HTML Hypertext Markup Language docs Content:Other Any docs not covered by another "Content:" label Content:WebAPI Web API docs merge conflicts 🚧 [PR only]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants