Skip to content

Improve instruction headings - #1611

Merged
jamiebenstead merged 1 commit into
mainfrom
improve-instruction-headings
Aug 26, 2026
Merged

Improve instruction headings#1611
jamiebenstead merged 1 commit into
mainfrom
improve-instruction-headings

Conversation

@jamiebenstead

@jamiebenstead jamiebenstead commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

closes https://github.com/RaspberryPiFoundation/digital-editor-issues/issues/420
With these changes we get 1750 for free

Updated heading sizes:
image

@jamiebenstead
jamiebenstead marked this pull request as ready for review August 26, 2026 12:08
Copilot AI lite review requested due to automatic review settings August 26, 2026 12:08

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

This PR updates the styling for headings within the project instructions panel to improve visual hierarchy and readability (per the linked issue context in the PR description).

Changes:

  • Add explicit h1 styling using typography.style-2(bold) and consistent top spacing.
  • Add explicit h3 styling using typography.style-1(bold) and consistent top spacing.
  • Keep existing h2 styling, aligning all heading levels to the same margin-block-start token.

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

@DNR500 DNR500 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.

Nice, might be worth just checking if any other header sizes h4–h6 that we might want to support as follow up

@jamiebenstead
jamiebenstead merged commit b920eb2 into main Aug 26, 2026
12 checks passed
@jamiebenstead
jamiebenstead deleted the improve-instruction-headings branch August 26, 2026 12:41
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