Skip to content

Remove obsolete macros and deprecated versions - #4467

Merged
chrisdavidmills merged 1 commit into
mdn:mainfrom
hamishwillee:obsolete_removal1
Apr 26, 2021
Merged

Remove obsolete macros and deprecated versions#4467
chrisdavidmills merged 1 commit into
mdn:mainfrom
hamishwillee:obsolete_removal1

Conversation

@hamishwillee

Copy link
Copy Markdown
Collaborator

According to @chrisdavidmills MDN does not used the argument in deprecated macro and should not use the obsolete macros (header or inline) - see #4226 (comment)

What this does is remove mention of the version argument from deprecated, and mention of the obsolete macro. I have done a very small subset of the cases too. This gives people a chance to complain before I do a PR with all the remaining fixes.

FYI @chrisdavidmills @sideshowbarker @Elchi3

@hamishwillee
hamishwillee requested review from a team and chrisdavidmills and removed request for a team April 26, 2021 02:32
<p><code><a href="https://github.com/mdn/yari/tree/master/kumascript/macros/deprecated_inline.ejs">deprecated_inline</a></code> inserts an in-line deprecated mark to discourage the use of an API that is officially deprecated. <strong>Note:</strong> "Deprecated" means that the item should no longer be used, but still functions. If you mean that it no longer works at all, use the term "obsolete."</p>

<p>Don't use the parameter in any browser-agnostic area (HTML, APIs, JS, CSS, …).</p>
<p><code><a href="https://github.com/mdn/yari/tree/master/kumascript/macros/deprecated_inline.ejs">deprecated_inline</a></code> inserts an in-line deprecated mark ({{Deprecated_Inline}}) to discourage the use of an API that is officially deprecated (or has been removed).</p>

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

This page has the main changes - essentially removing the things we don' support from the user guide.

@github-actions

Copy link
Copy Markdown
Contributor

Preview URLs

Flaws

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

URL: /en-US/docs/Mozilla/Firefox/Releases/2/Adding_feed_readers_to_Firefox
Title: Adding feed readers to Firefox
on GitHub
Flaw count: 2

  • macros:
    • /en-US/docs/Web/API/Window/navigator/registerContentHandler does not exist
    • /en-US/docs/Web/API/Window/navigator/registerContentHandler does not exist

URL: /en-US/docs/Web/CSS/-moz-user-input
Title: -moz-user-input
on GitHub
Flaw count: 3

  • macros:
    • /en-US/docs/Web/CSS/user-input does not exist
    • /en-US/docs/Web/CSS/-moz-user-modify redirects to /en-US/docs/Web/CSS/user-modify
    • /en-US/docs/Web/CSS/-moz-user-select redirects to /en-US/docs/Web/CSS/user-select

URL: /en-US/docs/Web/API/Window/releaseEvents
Title: Window.releaseEvents()
on GitHub
Flaw count: 6

  • macros:
    • /en-us/docs/web/api/globalcrypto (url: /en-US/docs/Web/API/GlobalCrypto) does not exist
    • /en-us/docs/web/api/speechsynthesisgetter (url: /en-US/docs/Web/API/SpeechSynthesisGetter) does not exist
    • /en-us/docs/web/api/windowmodal (url: /en-US/docs/Web/API/WindowModal) does not exist
    • /en-us/docs/web/api/onerroreventhandlerforwindow (url: /en-US/docs/Web/API/OnErrorEventHandlerForWindow) does not exist
    • /en-us/docs/web/api/chromewindow (url: /en-US/docs/Web/API/ChromeWindow) does not exist
  • broken_links:
    • Can't resolve /en-US/docs/Web/API/window.captureEvents

URL: /en-US/docs/Web/API/Document/all
Title: Document.all
on GitHub
Flaw count: 2

  • macros:
    • /en-US/docs/Web/API/HTMLAllCollection does not exist
    • /en-US/docs/Web/API/HTMLAllCollection does not exist

URL: /en-US/docs/Web/API/HTMLMenuItemElement
Title: HTMLMenuItemElement
on GitHub
Flaw count: 8

  • sectioning:
    • Excess <h2> tag that is NOT at root-level (id='Methods', text='Methods')
  • bad_bcd_links:
    • no explanation!
    • no explanation!
    • no explanation!
    • no explanation!
    • no explanation!
    • and 2 more flaws omitted

External URLs

URL: /en-US/docs/Glossary/Fetch_directive
Title: Fetch directive
on GitHub

No external URLs

URL: /en-US/docs/Glossary/Document_directive
Title: Document directive
on GitHub

No external URLs

URL: /en-US/docs/MDN/Structures/Macros/Commonly-used_macros
Title: Commonly-used macros
on GitHub


URL: /en-US/docs/Mozilla/Firefox/Releases/2/Adding_feed_readers_to_Firefox
Title: Adding feed readers to Firefox
on GitHub

No external URLs

URL: /en-US/docs/Web/HTML/Element/menuitem
Title: <menuitem>
on GitHub


URL: /en-US/docs/Web/SVG/Attribute/cap-height
Title: cap-height
on GitHub


URL: /en-US/docs/Web/SVG/Attribute/arabic-form
Title: arabic-form
on GitHub


URL: /en-US/docs/Web/SVG/Attribute/baseProfile
Title: baseProfile
on GitHub


URL: /en-US/docs/Web/SVG/Attribute/attributeType
Title: attributeType
on GitHub


URL: /en-US/docs/Web/CSS/-moz-user-input
Title: -moz-user-input
on GitHub


URL: /en-US/docs/Web/API/DocumentTouch
Title: DocumentTouch
on GitHub

No external URLs

URL: /en-US/docs/Web/API/IDBCursor
Title: IDBCursor
on GitHub


URL: /en-US/docs/Web/API/HTMLTableElement/align
Title: HTMLTableElement.align
on GitHub


URL: /en-US/docs/Web/API/Window/releaseEvents
Title: Window.releaseEvents()
on GitHub

No external URLs

URL: /en-US/docs/Web/API/Document/all
Title: Document.all
on GitHub


URL: /en-US/docs/Web/API/Document/anchors
Title: Document.anchors
on GitHub


URL: /en-US/docs/Web/API/HTMLMenuItemElement
Title: HTMLMenuItemElement
on GitHub


URL: /en-US/docs/Web/API/AudioBuffer/AudioBuffer
Title: AudioBuffer()
on GitHub


URL: /en-US/docs/Web/API/File/fileSize
Title: File.fileSize
on GitHub

No external URLs

@chrisdavidmills chrisdavidmills left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM; thanks @hamishwillee !

@chrisdavidmills
chrisdavidmills merged commit 2be3094 into mdn:main Apr 26, 2021
@hamishwillee
hamishwillee deleted the obsolete_removal1 branch April 27, 2021 01:34
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Aug 15, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants