Repository navigation
Conversation
|
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message. To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #1177 +/- ##
=============================================
+ Coverage 82.17% 82.19% +0.01%
Complexity 3399 3399
=============================================
Files 136 136
Lines 13187 13198 +11
=============================================
+ Hits 10837 10848 +11
Misses 2350 2350
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
What?
Closes #1176
Adds a
titleattribute to the singular Markdown<link rel="alternate">autodiscovery tag inMarkdown_Feeds::add_discovery_links().Why?
HTML autodiscovery standards and WordPress core conventions (such as
feed_links_extra()) require alternate format links to provide a descriptive title attribute. The site-wide Markdown feed link already includes a title, but the singular link omitted it, leaving the link anonymous to crawlers and user agents.How?
Updated
Markdown_Feeds::add_discovery_links()to passesc_attr( sprintf( __( '%s (Markdown)', 'ai' ), wp_strip_all_tags( (string) get_the_title( $post ) ) ) )into thetitleattribute of the singular<link>tag. Added an assertion toMarkdown_FeedsTest::test_discovery_links_on_singular()to verify the title attribute is present.Use of AI Tools
AI assistance: Yes
Tool(s): Google Antigravity
Model(s): Gemini
Used for: Identifying the missing attribute, drafting the patch, and adding the integration test assertion; final implementation and tests were verified locally by me.
Testing Instructions
/hello-world/).rel="alternate".title="<Post Title> (Markdown)".phpunit tests/Integration/Includes/Experiments/Markdown_Feeds/Markdown_FeedsTest.php.Screenshots or screencast
<link rel="alternate" type="text/markdown" href="http://example.com/hello-world/?output_format=markdown" /><link rel="alternate" type="text/markdown" title="Hello world! (Markdown)" href="http://example.com/hello-world/?output_format=markdown" />Changelog Entry