Skip to content

Plugin Directory: Add IDs to the section headings on the advanced view - #434

Open
harm0nic wants to merge 3 commits into
WordPress:trunkfrom
harm0nic:patch-1
Open

harm0nic wants to merge 3 commits into
WordPress:trunkfrom
harm0nic:patch-1

Conversation

@harm0nic

@harm0nic harm0nic commented Dec 3, 2024 •

Copy link
Copy Markdown

Set ID for Advanced Options H2 heading. This is useful for plugin developers who want to make it easier to direct users to previous versions of a plugin.

Without this, Downloads History was the most convenient ID to anchor.

https://meta.trac.wordpress.org/ticket/7844

Summary by CodeRabbit

  • Enhancements
    • Added anchor links to the Advanced Options, Community Options, Commercial Options, and Danger Zone section headings.
    • Added an anchor link to the Statistics heading for easier navigation.

harm0nic and others added 2 commits December 3, 2024 14:01
Set ID for Advanced Options H2 heading. This is useful for plugin developers who want to make it easier to direct users to previous versions of a plugin.

Without this, Downloads History was the most convenient ID to anchor.
Builds on the Advanced Options anchor so that Statistics, Community Options,
Commercial Options and The Danger Zone can be linked to directly as well.
The `plugin-` prefix keeps them clear of the existing `#community`,
`#commercial` and `#advanced` IDs on the page.

See https://meta.trac.wordpress.org/ticket/7844.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@obenland obenland changed the title Update template-tags.php to set ID for Advanced Options heading Plugin Directory: Add IDs to the section headings on the advanced view Oct 21, 2026
@github-actions

github-actions Bot commented Oct 21, 2026 •

Copy link
Copy Markdown

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 props-bot label.

Core Committers: Use this line as a base for the props when committing in SVN:

Props harmonic_design, obenland.

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

@coderabbitai

coderabbitai Bot commented Oct 21, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 39906851-db72-4a22-8a63-1ffbdc590e34
📥 Commits

Reviewing files that changed from the base of the PR and between 91e240c and 43c7a87.

📒 Files selected for processing: 2
  • wordpress.org/public_html/wp-content/themes/pub/wporg-plugins-2024/inc/template-tags.php
  • wordpress.org/public_html/wp-content/themes/pub/wporg-plugins-2024/template-parts/section-advanced.php

Included review availability: This review used your included allowance. Your plan provides up to 2 included reviews per hour; 1 remain after this review.

📝 Walkthrough

Added HTML IDs to the Advanced Options, Community Options, Commercial Options, Danger Zone, and Statistics headings on plugin pages.

Changes: Plugin heading anchors

Layer / File(s) Summary
Add IDs to plugin headings
wordpress.org/public_html/wp-content/themes/pub/wporg-plugins-2024/inc/template-tags.php, wordpress.org/public_html/wp-content/themes/pub/wporg-plugins-2024/template-parts/section-advanced.php
Added IDs to five headings. The Statistics heading text and translation remain unchanged.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Feature

Suggested reviewers: stevendufresne

Merge Risk: ⚪ Minimal · up to 43c7a

The heading-anchor additions are limited in scope and present no identified merge-blocking risk.

Pre-merge checks: Passed 5
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title clearly describes the main change: adding IDs to section headings in the advanced plugin view.
Docstring Coverage Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 4 functions across 2 files.
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants