Skip to content

feat: set header rtl styles#10

Closed
andrey-canon wants to merge 1 commit intoopen-release/teak.nelpfrom
and/set-header-rtl-styles
Closed

feat: set header rtl styles#10
andrey-canon wants to merge 1 commit intoopen-release/teak.nelpfrom
and/set-header-rtl-styles

Conversation

@andrey-canon
Copy link
Collaborator

Description

Set the rtl styles for learning header and site desktop header

Before (learning header)

image

Before (desktop header)

image

After (learning header)

image

After (desktop header)

image

@andrey-canon andrey-canon force-pushed the and/set-header-rtl-styles branch from c2c22b3 to 15e6377 Compare January 22, 2026 00:11
Copy link

@johanseto johanseto left a comment

Choose a reason for hiding this comment

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

Working in my local env.
Something interesting is that without paragon and conditional user fields. Learner dashboard works as expected in the after image.

So to replicate, I had to configure those settings.

Screencast.from.22-01-26.08.26.44.webm
2026-01-22_08-17

@andrey-canon
Copy link
Collaborator Author

Thanks @johanseto. Your findings helped me realize how the MFE behaves: it loads the default Paragon styles and then uses postcss-rtlcss to generate valid RTL CSS based on the original styles.

Based on this, I applied the same process to generate a new default styles file, so this PR is no longer necessary.

@andrey-canon andrey-canon deployed to open-release/teak.nelp January 22, 2026 19:15 — with GitHub Actions Active
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