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

Upgrade @edx/brand to point to EducateWorkforce version for nutmeg release. #32

Conversation

ztraboo
Copy link

@ztraboo ztraboo commented Apr 12, 2024

  • Upgrade @edx/brand to point to EducateWorkforce branding
  • Setup a stick footer on PageLoading by increasing the page vertical height.
  • Tab in the course outline sequential pages.

ztraboo added 3 commits April 10, 2024 10:55
Reached out to support for this. It appears the page loading doesn't take changes in for the `brand-openedx` CSS changes, so we need to adjust the height manually.

https://discuss.openedx.org/t/sticky-footer-mfe-learning-page-loading/12721
Verified that the instructional design team would like the the `subsection` sequences indented. This will allow for visual separation of the content and provide better nesting.

I tried using bootstrap spacing https://getbootstrap.com/docs/4.3/utilities/spacing/ of `pl-5` but it was too far in for the indent and I needed to specify an exact value to align the checkmark progress with the title of the section name.

Ref: #25
@ztraboo ztraboo self-assigned this Apr 12, 2024
@ztraboo ztraboo added the enhancement New feature or request label Apr 12, 2024
@ztraboo ztraboo added this to the cu-release/nutmeg.2 milestone Apr 12, 2024
@ztraboo ztraboo merged commit f9ee0eb into develop/nutmeg.master Apr 12, 2024
1 of 2 checks passed
@ztraboo ztraboo changed the title Upgrade @edx/brand to point to EducateWorkforce version. Upgrade @edx/brand to point to EducateWorkforce version for nutmeg release. Apr 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant