Skip to content

Conversation

@eshaben
Copy link
Collaborator

@eshaben eshaben commented Aug 5, 2025

This pull request introduces a new page-level toggle feature.

Page-level toggle feature:

  • Added new template toggle-page.html to support page-level tab toggling, including logic to move tabbed labels next to the page title and synchronize tab selection with Table of Contents visibility.
  • Introduced toggle-page.css with styles for the page-level toggle, including pill-style tab labels, responsive layout, and active tab highlighting.

@eshaben eshaben marked this pull request as ready for review August 20, 2025 01:16
Copilot AI review requested due to automatic review settings August 20, 2025 01:16
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 adds functionality and styles for toggling page content using a page-level toggle interface. The implementation moves tabbed labels into an h1 wrapper to create a pill-style toggle that controls content visibility and updates the table of contents accordingly.

  • Adds a new HTML template that manipulates page content to relocate tabbed labels next to the page title
  • Implements JavaScript functionality to manage toggle states and TOC visibility
  • Introduces CSS styles for a pill-style toggle interface with smooth transitions

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 5 comments.

File Description
material-overrides/toggle-page.html New template that extends main.html with content manipulation and toggle functionality
material-overrides/assets/stylesheets/toggle-page.css New stylesheet defining pill-style toggle appearance and responsive behavior
material-overrides/assets/stylesheets/polkadot.css Updates existing styles to support the new toggle functionality and fix spacing issues

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@eshaben eshaben marked this pull request as draft September 2, 2025 15:19
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