Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WebExtensions - fix up all the tags except for apis #25117

Merged
merged 1 commit into from
Mar 7, 2023

Conversation

hamishwillee
Copy link
Collaborator

Fixes up all WebExtension tags. Yes, this is a huge PR. But then only one review. I scanned through them all and script still seems to be working well.

Part of mdn/mdn#262

@hamishwillee hamishwillee requested a review from a team as a code owner March 6, 2023 22:33
@hamishwillee hamishwillee requested review from jpmedley and removed request for a team March 6, 2023 22:33
@github-actions github-actions bot added Content:Other Any docs not covered by another "Content:" label Content:WebExt WebExtensions docs labels Mar 6, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Mar 6, 2023

Preview URLs (91 pages)
Flaws (58)

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

URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/Modify_a_web_page
Title: Modify a web page
Flaw count: 1

  • broken_links:
    • Can't resolve /en-US/docs/Mozilla/Add-ons/Add-on_Debugger

URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/Tips
Title: Tips and Tricks
Flaw count: 1

  • broken_links:
    • Can't resolve /en-US/docs/Web/JavaScript/New_in_JavaScript/ECMAScript_6_support_in_Mozilla

URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/What_are_WebExtensions
Title: What are extensions?
Flaw count: 1

  • broken_links:
    • Can't resolve /en-US/docs/Mozilla/Firefox/Multiprocess_Firefox

URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/Match_patterns
Title: Match patterns in extension manifests
Flaw count: 1

  • broken_links:
    • No need for the pathname in anchor links if it's the same page

URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/manifest.json/dictionaries
Title: dictionaries
Flaw count: 1

  • bad_bcd_queries:
    • No BCD data for query: webextensions.manifest.dictionaries

URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/Interact_with_the_clipboard
Title: Interact with the clipboard
Flaw count: 2

  • broken_links:
    • Can't resolve /en-US/docs/Web/Guide/HTML/Editable_content
    • Can't resolve /en-US/docs/Web/Guide/HTML/Editable_content

URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/Browser_support_for_JavaScript_APIs
Title: Browser support for JavaScript APIs
Flaw count: 48

  • macros:
    • Calling the Compat macro with any arguments is deprecated; instead use the 'browser-compat' front-matter key.
    • Calling the Compat macro with any arguments is deprecated; instead use the 'browser-compat' front-matter key.
    • Calling the Compat macro with any arguments is deprecated; instead use the 'browser-compat' front-matter key.
    • Calling the Compat macro with any arguments is deprecated; instead use the 'browser-compat' front-matter key.
    • Calling the Compat macro with any arguments is deprecated; instead use the 'browser-compat' front-matter key.
    • and 43 more flaws omitted

URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/Working_with_files
Title: Working with files
Flaw count: 3

  • broken_links:
    • Can't resolve /en-US/docs/Web/API/Document/dragenter_event
    • Can't resolve /en-US/docs/Web/API/Document/dragover_event
    • Can't resolve /en-US/docs/Web/API/Document/drop_event
External URLs (227)

URL: /en-US/docs/Mozilla/Add-ons/WebExtensions
Title: Browser Extensions


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/Internationalization
Title: Internationalization


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/Extending_the_developer_tools
Title: Extending the developer tools


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/Differences_between_API_implementations
Title: Differences between API implementations


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/Modify_a_web_page
Title: Modify a web page


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/Working_with_the_Tabs_API
Title: Working with the Tabs API


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/Tips
Title: Tips and Tricks


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/Intercept_HTTP_requests
Title: Intercept HTTP requests


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/Implement_a_settings_page
Title: Implement a settings page


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/What_are_WebExtensions
Title: What are extensions?


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/Safely_inserting_external_content_into_a_page
Title: Safely inserting external content into a page


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/Firefox_workflow_overview
Title: Firefox workflow overview


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/Match_patterns
Title: Match patterns in extension manifests


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/What_next_
Title: What next?


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/Examples
Title: Example extensions


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/Native_messaging
Title: Native messaging


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/manifest.json/theme
Title: theme


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/manifest.json/version_name
Title: version_name


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/manifest.json/permissions
Title: permissions


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/manifest.json/theme_experiment
Title: theme_experiment


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/manifest.json/incognito
Title: incognito


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/manifest.json/sidebar_action
Title: sidebar_action


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/manifest.json/browser_specific_settings
Title: browser_specific_settings


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/manifest.json/externally_connectable
Title: externally_connectable


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/manifest.json/storage
Title: storage


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/manifest.json/protocol_handlers
Title: protocol_handlers


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/manifest.json/page_action
Title: page_action


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/manifest.json/chrome_settings_overrides
Title: chrome_settings_overrides


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/manifest.json/commands
Title: commands


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/manifest.json/version
Title: version


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/manifest.json/content_security_policy
Title: content_security_policy


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/manifest.json/options_ui
Title: options_ui


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/manifest.json/web_accessible_resources
Title: web_accessible_resources


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/manifest.json/optional_permissions
Title: optional_permissions


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/manifest.json/host_permissions
Title: host_permissions


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/manifest.json/icons
Title: icons


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/user_interface
Title: User interface


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/user_interface/Popups
Title: Popups


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/user_interface/Sidebars
Title: Sidebars


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/user_interface/Context_menu_items
Title: Context menu items


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/user_interface/devtools_panels
Title: devtools panels


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/user_interface/Notifications
Title: Notifications


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/user_interface/Options_pages
Title: Options page


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/user_interface/Omnibox
Title: Address bar suggestions


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/user_interface/Browser_action
Title: Toolbar button


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/user_interface/Extension_pages
Title: Extension pages


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/user_interface/Page_actions
Title: Address bar button


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/user_interface/Browser_styles
Title: Browser styles


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/Your_first_WebExtension
Title: Your first extension


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/Build_a_cross_browser_extension
Title: Building a cross-browser extension


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/Your_second_WebExtension
Title: Your second extension


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/Content_Security_Policy
Title: Content Security Policy


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/Developing_WebExtensions_for_Thunderbird
Title: Developing WebExtensions for Thunderbird


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/Chrome_incompatibilities
Title: Chrome incompatibilities


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/Native_manifests
Title: Native manifests


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/Working_with_files
Title: Working with files


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/Sharing_objects_with_page_scripts
Title: Sharing objects with page scripts


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/Content_scripts
Title: Content scripts


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/Work_with_contextual_identities
Title: Work with contextual identities


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/Firefox_differentiators
Title: Firefox differentiators


URL: /en-US/docs/Mozilla/Add-ons/WebExtensions/Work_with_the_Cookies_API
Title: Work with the Cookies API

(comment last updated: 2023-03-07 05:01:00)

@wbamberg wbamberg requested review from rebloor and removed request for jpmedley March 6, 2023 22:46
@wbamberg
Copy link
Collaborator

wbamberg commented Mar 6, 2023

@rebloor , I've assigned this to you but if you like I am happy to review it.

@wbamberg
Copy link
Collaborator

wbamberg commented Mar 7, 2023

Actually one thing I would say is that this PR adds a lot of non-standard status values, which if we then start using that to drive "non-standard" banners, will mean we will add lots of these banners to the pages, which they don't currently have.

So in this PR we should suppress non-standard.

Note that the BCD does not list these features as non-standard, which is a good thing since it means the automation that keeps status in sync with BCD will not cause this to break.

@rebloor
Copy link
Contributor

rebloor commented Mar 7, 2023

@wbamberg seems like you might be on the ball with this one. Would you like to finish commenting and reviewing? From my POV your comment about non-standard makes sense. Otherwise, I'm happy to spot-check the changes and approve on that basis. The number of files changed and the impact this has on the performance of the files tab makes much more impractical.

@wbamberg
Copy link
Collaborator

wbamberg commented Mar 7, 2023

Thanks @rebloor . I'm happy to review :). But actually I agree about the size of this, because GitHub hides diffs beyond a certain number of files.

@hamishwillee could you split this into say 3 PRs, as well as remove non-standard as mentioned in #25117 (comment) ? (non-standard is weird for WebExtensions, so I think it is OK to omit it).

@hamishwillee
Copy link
Collaborator Author

Sure! This is now all the webext except for /api - with "non-standard" removed. There are still some "deprecated". New PRs to follow for APIs.

@hamishwillee hamishwillee changed the title WebExtensions - fix up all the tags WebExtensions - fix up all the tags except for apis Mar 7, 2023
@hamishwillee
Copy link
Collaborator Author

FYI @rebloor @wbamberg

Copy link
Collaborator

@wbamberg wbamberg left a comment

Choose a reason for hiding this comment

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

🎉

@wbamberg wbamberg merged commit d86e14d into mdn:main Mar 7, 2023
@hamishwillee hamishwillee deleted the webext_fixtags branch March 9, 2023 23:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content:Other Any docs not covered by another "Content:" label Content:WebExt WebExtensions docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants