Skip to content

docs: adds instance theming related documentation#17

Merged
tecoholic merged 1 commit intomainfrom
tecoholic/SE-6592-themes
Mar 16, 2026
Merged

docs: adds instance theming related documentation#17
tecoholic merged 1 commit intomainfrom
tecoholic/SE-6592-themes

Conversation

@tecoholic
Copy link
Copy Markdown
Member

@tecoholic tecoholic force-pushed the tecoholic/SE-6592-themes branch from 4218eb7 to 9fe6b5c Compare March 16, 2026 05:21
Comment on lines +26 to +32
1. If the instance is using a release branch like `release/teak`, create a branch from it. If the instance already has a custom branch, go to next step.
2. Remove the following from `.gitignore`
* `lms/static/sass/_lms-overrides.scss`
* `lms/static/sass/common-variables.scss`
3. Create those 2 files. Could be empty, or add your customizations.
4. Commit and push your branch
5. Use the new branch in the `git clone` command under `PICASSO_EXTRA_COMMANDS`.
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.

@tecoholic In tutor-contrib-grove, we have a way of adding custom CSS overriding the design tokens SIMPLE_THEME_EXTRA_CSS. However, what I understand is that, this still only affects MFE themes and not the legacy UI. However, what you suggest is to implement legacy UI theming. Is this correct?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Yes. That's my understanding as well. If we look at the implementation, the SIMPLE_THEME_EXTRA_CSS is added to the customization.scss of the MFE build and not the legacy theme. This set of instructions is specifically for the legacy UI theming.

I had mentioned the SIMPLE_THEME_EXTRA_CSS usage as a "tip" in the MFE customization section of the docs as well.

Copy link
Copy Markdown
Member

@kaustavb12 kaustavb12 left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@tecoholic tecoholic merged commit f19a668 into main Mar 16, 2026
1 check 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.

2 participants