Skip to content

Document the Boost theme colour modes - #1684

Merged
safatshahin merged 1 commit into
moodle:mainfrom
junpataleta:MDL-68037-dark-mode
Oct 21, 2026
Merged

safatshahin merged 1 commit into
moodle:mainfrom
junpataleta:MDL-68037-dark-mode

Conversation

@junpataleta

Copy link
Copy Markdown
Collaborator

Adds a Colour modes developer guide covering how Boost renders a mode with the Bootstrap 5.3 colour modes API, which custom properties and design system tokens follow it, how to audit a plugin's styles for hardcoded colours, and how to test in both modes. Also covers icons, editor content, contrast targets, and what a theme inheriting from Boost needs to know.

Notes the feature on the 5.3 developer update page, documents the new Behat --colourmode option on the running tests page, and cross-links from the styles.css plugin file, theme styles, and developer guides pages.

See MDL-68037.

Copilot AI lite review requested due to automatic review settings August 25, 2026 11:50
@netlify

netlify Bot commented Aug 25, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for moodledevdocs ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit ad62a9f
🔍 Latest deploy log https://app.netlify.com/projects/moodledevdocs/deploys/6ad89b20e7802fb02fe93110
😎 Deploy Preview https://deploy-preview-1684--moodledevdocs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@junpataleta junpataleta added the blocked-by-tracker This issue is blocked by an issue in https://tracker.moodle.org/ label Aug 25, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds developer documentation for Boost’s new light/dark colour modes (Bootstrap 5.3 colour modes API), including guidance for plugin authors on sourcing colours from theme tokens/custom properties, auditing for hardcoded colours, and testing (including Behat support).

Changes:

  • Adds a new “Colour modes” developer guide describing how Boost applies modes and how plugins/themes should support them.
  • Documents the Behat --colourmode option and links to the new guide.
  • Cross-links the new guide from the developer update page and relevant CSS/style documentation pages.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
general/development/tools/behat/running.md Documents --colourmode usage for Behat runs and links to the new guide.
docs/guides/colourmodes/index.md New comprehensive guide explaining Boost colour modes, tokens/properties, auditing, contrast, and testing guidance.
docs/guides.md Adds the new guide to the Developer Guides list.
docs/devupdate.md Notes the new Boost colour modes feature and points readers to the guide.
docs/apis/plugintypes/theme/styles.md Adds a callout linking theme/plugin styling guidance to colour modes.
docs/apis/_files/styles-css.mdx Adds a warning reminding plugin authors to check both colour modes and linking to the guide.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread general/development/tools/behat/running.md Outdated
Comment thread docs/guides/colourmodes/index.md Outdated
@junpataleta junpataleta removed the blocked-by-tracker This issue is blocked by an issue in https://tracker.moodle.org/ label Oct 21, 2026
@junpataleta
junpataleta requested a lite review from Copilot October 21, 2026 09:30

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Comment thread docs/guides/colourmodes/index.md Outdated
Comment thread docs/guides/colourmodes/index.md Outdated

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 Changes recommended

Documentation corrections are needed before approval.

3 open issues
1 resolved since last review

Review effort: Lite

Comment thread docs/devupdate.md Outdated
Comment thread docs/guides/colourmodes/index.md Outdated

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Comment thread docs/guides/colourmodes/index.md Outdated
Comment thread general/development/tools/behat/running.md Outdated
Adds a Colour modes developer guide covering how Boost renders a mode
with the Bootstrap 5.3 colour modes API, which custom properties and
design system tokens follow it, how to audit a plugin's styles for
hardcoded colours, and how to test in both modes. Also covers icons,
editor content, contrast targets, and what a theme inheriting from Boost
needs to know.

Notes the feature on the 5.3 developer update page, documents the new
Behat --colourmode option on the running tests page, and cross-links
from the styles.css plugin file, theme styles, and developer guides
pages.

See MDL-68037.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

Only minor documentation nits remain, with no blocking issues identified.

No open issues 🎉

2 resolved since last review

Review effort: Lite

@safatshahin safatshahin left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks @junpataleta
Looks good!

@safatshahin
safatshahin added this pull request to the merge queue Oct 21, 2026
Merged via the queue into moodle:main with commit 6954243 Oct 21, 2026
6 checks passed
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.

3 participants