Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Apps MFE does not inherit INDIGO_* settings #66

Closed
misilot opened this issue Feb 19, 2024 · 3 comments
Closed

Apps MFE does not inherit INDIGO_* settings #66

misilot opened this issue Feb 19, 2024 · 3 comments

Comments

@misilot
Copy link
Contributor

misilot commented Feb 19, 2024

The apps MFE does not inherit the INDIGO_ settings such as

INDIGO_FOOTER_LEGAL_LINKS
INDIGO_FOOTER_NAV_LINKS
INDIGO_PRIMARY_COLOR
@DawoudSheraz DawoudSheraz moved this from Pending Triage to Backlog in Tutor project management Feb 20, 2024
@DawoudSheraz
Copy link
Contributor

@misilot Hello. The current description does not have enough context for the underlying issue. Can you add more details about the issue, its impact, etc.? Thanks

@misilot
Copy link
Contributor Author

misilot commented Feb 20, 2024

Sure,

So INDIGO_PRIMARY_COLOR used to apply to the primary color in the _variable.scss file, but that was removed in #54

However, this variable does not pass to the MFE as part of the theming for the full site.

Secondary, both INDIGO_FOOTER_LEGAL_LINKS and INDIGO_FOOTER_NAV_LINKS do not trickle down to the MFE (apps.* subsites), so the footer is not consistent on the different pages as a learner goes through the site.

@hinakhadim
Copy link
Collaborator

Hi @misilot , Currently the MFEs don't get the INDIGO_ settings. The Links settings don't passed to MFEs because these are the array and the primary color settings don't passed to MFEs because MFEs are styled using brand-openedx npm package. Therefore, It doesn't make sense to pass theme_color variable to MFEs when it is of no use. Furthermore, the community has added ADR for making footer-links consistent on overall platform (including LMS and MFEs). But development on it is paused now, as we are exploring UI Plugin Approach for MFEs. This is where community is looking at. If there's anything I missed, feel free to share.

@github-project-automation github-project-automation bot moved this from Backlog to Done in Tutor project management Mar 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

No branches or pull requests

3 participants