Skip to content

fix(web): headings font color inheritance - #710

Merged
hejsztynx merged 1 commit into
mainfrom
@ksienkiewicz/fix-web-heading-font-color-inheritance
Jul 20, 2026
Merged

hejsztynx merged 1 commit into
mainfrom
@ksienkiewicz/fix-web-heading-font-color-inheritance

Conversation

@hejsztynx

Copy link
Copy Markdown
Collaborator

Summary

The color of the headings - h1, h2, ..., was not properly defined. You could create a CSS rule e.g h1 { color: 'white' } in an app's higher context and that color would get applied, it did not use the editor's color. Now it properly inherits it.

Compatibility

OS Implemented
iOS
Android
Web

Checklist

  • E2E tests are passing
  • Required E2E tests have been added (if applicable)

Copilot AI review requested due to automatic review settings July 20, 2026 08:43

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 fixes heading (h1h6) text color on web so headings inside the enriched editor/view consistently inherit the surrounding editor text color, instead of being overridden by app-level/global h1 { color: ... } rules.

Changes:

  • Add color: inherit; to h1h6 rules scoped under .eti-editor and .et-view to ensure heading color follows the editor/view container styling.

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

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

LGTM!

@hejsztynx
hejsztynx merged commit ec1c03f into main Jul 20, 2026
9 checks passed
@hejsztynx
hejsztynx deleted the @ksienkiewicz/fix-web-heading-font-color-inheritance branch July 20, 2026 10:04
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