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

Remove legacy Learning Mode CSS as it's part of Sensei now #7431

Merged
merged 1 commit into from
Oct 16, 2023

Conversation

donnapep
Copy link
Contributor

@donnapep donnapep commented Oct 14, 2023

Fixes #7429.

Changes proposed in this Pull Request:

Removes the learning-mode.css file as it's part of Sensei now and users have had plenty of time to update both the plugin and the theme. We also want to try to minimize the use of Sensei-specific code in the theme, and instead have it in Sensei where possible.

Testing Instructions

  1. Checkout Eliminate usage of temporary Course theme hook sensei#7219.
  2. View the lessons and quizzes in Learning Mode. There should be no change to the UI.

@donnapep donnapep self-assigned this Oct 14, 2023
@donnapep donnapep added this to the Course 1.3.5 milestone Oct 14, 2023
@donnapep donnapep requested a review from a team October 14, 2023 12:53
Copy link
Member

@m1r0 m1r0 left a comment

Choose a reason for hiding this comment

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

👍

@donnapep donnapep linked an issue Oct 16, 2023 that may be closed by this pull request
@donnapep donnapep merged commit b476716 into feature/frontend-improvements Oct 16, 2023
@donnapep donnapep deleted the remove/course-theme-hook branch October 16, 2023 15:10
renatho pushed a commit that referenced this pull request Nov 1, 2023
* Update styles of question feedback (#7373)

* Specify the grade style to be applied only in the question header

* Update background color for gold variation

* Update background color for default variation

* Update border color for gold variation

* Tweak CSS formatting

* Tweak color for border of ordering question

---------

Co-authored-by: Donna Peplinskie <donnapep@gmail.com>

* Update version and description

* Add new changelog

* Remove legacy Learning Mode CSS as it's part of Sensei now (#7431)

* Course: Add body class for theme variations (#7432)

* Add custom property with variation name

* Add function to add body class and enqueue proper stylesheets

* Add stylesheets for all variations

* Add version number

* Style the "Awaiting Grade" notice (#7401)

* Style "Awaiting Grade" notice

* Move custom CSS to other areas of theme.json

* Remove typography setting as it needs to be done via CSS

* Do the same for the variations

* Fix notice style getting broken for css parsing issue

---------

Co-authored-by: Imran Hossain <imranh920@gmail.com>

* Bump versions and update change log

---------

Co-authored-by: Imran Hossain <imranh920@gmail.com>
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.

Remove the learning mode from the course theme
2 participants