We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Parent story: #35300
Copy the following tasks into the PR description of the XBlock and update the status on PR.
Convert Sass variable into css variables
Compile the css file of the block following given steps.
source_comments: int = SASS_COMMENTS_NONE
npm run compile-sass-dev
xmodule/static/css-builtin-blocks
Replace add_sass_to_fragment to add_css_to_fragment in blocks .py file
.py
Remove all .scss files linked to the block under xmodule/assets.
npm run build
./manage.py lms collectstatic
./manage.py cms collectstatic
Here is initial reference PR [Some the changes are only 1 time changes so don't need them in next block PR's.]
The content you are editing has changed. Please copy your edits and refresh the page.
The text was updated successfully, but these errors were encountered:
Merge pull request #35506 from openedx/farhan/sass-to-css-video-block
0c91b62
feat! Dropping Sass support from builtin video block #35570
farhan
No branches or pull requests
Parent story: #35300
Copy the following tasks into the PR description of the XBlock and update the status on PR.
Tasks:
Convert Sass variable into css variables
Compile the css file of the block following given steps.
xmodule/static/css-builtin-blocks
.Replace add_sass_to_fragment to add_css_to_fragment in blocks
.py
fileRemove all .scss files linked to the block under xmodule/assets.
Testing Notes:
npm run build
to run webpack and compile sass files../manage.py lms collectstatic
in lms shell to re-collect static files../manage.py cms collectstatic
in cms shell to re-collect static files.Reference PR:
Here is initial reference PR [Some the changes are only 1 time changes so don't need them in next block PR's.]
Tasks
The text was updated successfully, but these errors were encountered: