Skip to content

Custom themes now import highlights in css#31758

Merged
t3chguy merged 1 commit intoelement-hq:developfrom
Philldomd:philldomd/missing-highlights-in-codeblock
Jan 16, 2026
Merged

Custom themes now import highlights in css#31758
t3chguy merged 1 commit intoelement-hq:developfrom
Philldomd:philldomd/missing-highlights-in-codeblock

Conversation

@Philldomd
Copy link
Copy Markdown
Contributor

@Philldomd Philldomd commented Jan 16, 2026

Added import for codeblock highlights in css for custom themes

Checklist

Fixes #31757

More information about the change is written there.

Added import for codeblock highlights in css for custom themes

Signed-off-by: Örnfeldt Philip (66140321) <philip.ornfeldt@forsakringskassan.se>
@Philldomd Philldomd requested a review from a team as a code owner January 16, 2026 09:35
@github-actions github-actions Bot added the Z-Community-PR Issue is solved by a community member's PR label Jan 16, 2026
@import "../../../../res/css/_font-sizes.pcss";
@import "../../legacy-light/css/_paths.pcss";
@import "../../legacy-light/css/_fonts.pcss";
@import "../../legacy-light/css/_legacy-light.pcss";
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Isn't atom-one-light.min.css already imported by _legacy-light which is imported here

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Ah no it is in legacy-light.pcss (not underscored) instead

Copy link
Copy Markdown
Contributor Author

@Philldomd Philldomd Jan 16, 2026

Choose a reason for hiding this comment

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

Sadly not, it only lists:
/* diff highlight colors */
.hljs-addition {
background: #dfd;
}

.hljs-deletion {
background: #fdd;
}
which are present in built theme-dark-custom.css
but not the rest

It is present in legacy-light.pcss but that is not imported
It could be that it should include legacy-light.pcss instead of url loading the style. But this falls more in line with the rest of the setups in css i think

Copy link
Copy Markdown
Member

@t3chguy t3chguy left a comment

Choose a reason for hiding this comment

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

Looks sane to me

@t3chguy t3chguy enabled auto-merge January 16, 2026 09:57
@t3chguy t3chguy added this pull request to the merge queue Jan 16, 2026
Merged via the queue into element-hq:develop with commit e811cf0 Jan 16, 2026
51 of 53 checks passed
rbondesson pushed a commit to ZacksBot/element-web that referenced this pull request Jan 22, 2026
Added import for codeblock highlights in css for custom themes

Signed-off-by: Örnfeldt Philip (66140321) <philip.ornfeldt@forsakringskassan.se>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T-Defect Z-Community-PR Issue is solved by a community member's PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Code blocks doesn't display highlights correctly when using custom themes

2 participants