Skip to content

Conversation

@0xlukem
Copy link
Contributor

@0xlukem 0xlukem commented Oct 24, 2025

📝 Description

This PR removes the hr from the cards on the content pages to better align with the new styling.

This PR needs to be merged with this one: papermoonio/polkadot-mkdocs#167

🔍 Review Preference

Choose one:

  • ✅ I have time to handle formatting/style feedback myself
  • ⚡ Docs team handles formatting (check "Allow edits from maintainers")

🤖 AI-Ready Docs

If content changed, regenerate AI files:

  • ✅ I ran python3 scripts/generate_llms.py
  • ⚡ Docs team will regenerate (check "Allow edits from maintainers")

✅ Checklist

@0xlukem 0xlukem requested a review from a team as a code owner October 24, 2025 21:29
Copilot AI review requested due to automatic review settings October 24, 2025 21:29
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR removes <hr> elements from card components across documentation pages to align with new styling guidelines. The changes affect tutorial, infrastructure, and development documentation sections.

Key Changes

  • Removal of horizontal rule separators from card UI components
  • Cleanup of code blocks by removing content while preserving structure
  • Updates to AI-generated index files reflecting the content changes

Reviewed Changes

Copilot reviewed 68 out of 69 changed files in this pull request and generated no comments.

Show a summary per file
File Description
tutorials/polkadot-sdk/system-chains/asset-hub/index.md Removed <hr> from Asset Hub card
tutorials/polkadot-sdk/index.md Removed <hr> from SDK source code card
tutorials/onchain-governance/index.md Removed <hr> from OpenGov card
tutorials/interoperability/xcm-channels/index.md Removed <hr> from HRMP configurations card
tutorials/interoperability/index.md Removed <hr> from two interoperability cards
tutorials/index.md Removed <hr> from five tutorial cards
tutorials/dapps/index.md Removed <hr> from two dApp tutorial cards
infrastructure/staking-mechanics/index.md Removed <hr> from three staking mechanics cards
infrastructure/running-a-validator/operational-tasks/index.md Removed <hr> from two operational tasks cards
infrastructure/running-a-validator/onboarding-and-offboarding/index.md Removed <hr> from four onboarding/offboarding cards
infrastructure/running-a-validator/index.md Removed <hr> from three validator cards
develop/toolkit/parachains/spawn-chains/zombienet/index.md Removed <hr> from Zombienet tutorial card
develop/toolkit/parachains/spawn-chains/index.md Removed <hr> from spawn chains card
develop/toolkit/parachains/fork-chains/index.md Removed <hr> from fork chains tutorial card
develop/toolkit/parachains/fork-chains/chopsticks/index.md Removed <hr> from two Chopsticks cards
develop/toolkit/api-libraries/index.md Removed <hr> from two API libraries cards
develop/parachains/testing/index.md Removed <hr> from two testing cards
develop/parachains/maintenance/index.md Removed <hr> from two maintenance cards
develop/parachains/deployment/index.md Removed <hr> from chain spec builder card
develop/parachains/customize-parachain/index.md Removed <hr> from three customization cards
develop/interoperability/xcm-guides/index.md Removed <hr> from four XCM guide cards
develop/interoperability/index.md Removed <hr> from four interoperability cards
.ai/site-index.json Updated metadata reflecting modified content
.ai/pages/* Updated AI-generated pages with removed content

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@0xlukem 0xlukem added B0 - Needs Review Pull request is ready for review A2 - Maintenance Minor Pull request contains minor updates to an existing page (i.e., modifying parameters, steps, etc.) labels Oct 24, 2025
Copy link
Collaborator

@nhussein11 nhussein11 left a comment

Choose a reason for hiding this comment

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

LGTM but please could someone push the ia files again?

@eshaben eshaben added B1 - Ready to Merge Pull request is ready to be merged and removed B0 - Needs Review Pull request is ready for review labels Oct 31, 2025
@eshaben eshaben merged commit f6bfa3c into master Oct 31, 2025
5 of 6 checks passed
@eshaben eshaben deleted the 0xlukem/card-styling branch October 31, 2025 19:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A2 - Maintenance Minor Pull request contains minor updates to an existing page (i.e., modifying parameters, steps, etc.) B1 - Ready to Merge Pull request is ready to be merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants