Skip to content

Conversation

saforem2
Copy link
Member

Description

Screenshots (if applicable)

Before
After

Related Issue(s)

Type of Change

  • Documentation content update (new page, formatting/typo changes, adding more info, etc.)
  • Functionality bug fix
  • New feature (mkdocs feature, mkdocs-material style changes, HTML/CSS/JS customization, developer or repo tool)

Checklist

  • I have run make serve or make build-docs locally and verified that my changes render correctly
  • I have added at least one Label to this PR

@saforem2 saforem2 requested a review from felker as a code owner October 10, 2025 17:11
@saforem2 saforem2 requested review from atanikan and Copilot October 10, 2025 17:11
Copy link
Contributor

@Copilot 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 fixes and cleans up the CSS stylesheet for the ALCF documentation theme. The changes focus on updating color variables and removing commented-out code for better maintainability.

  • Updates color variables from green/neutral colors to blue/med colors in navigation and code highlighting
  • Removes commented-out CSS rules and adds new commented placeholders for future customization
  • Reorganizes code background color property placement for better readability

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

Copy link
Member

@atanikan atanikan left a comment

Choose a reason for hiding this comment

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

LGTM

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@atanikan atanikan merged commit 42215ab into main Oct 10, 2025
1 check passed
@atanikan atanikan deleted the fix-stylesheet branch October 10, 2025 17:14
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