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

Load user defined css after the main template css #4

Merged
merged 1 commit into from Mar 8, 2021

Conversation

mohamed-ali
Copy link

To enable customization of the main material design theme's css (for example to support RTL: example discussion d2l-ai/d2l-en#670 (comment) ). The custom css should be loaded after the main theme's css.

To enable customization of the main material design theme (for example to support RTL). The custom css should be loaded after the main theme's css.
@ChaiBapchya
Copy link

Has this been tested? Is there any way to ensure this doesn't break the existing functionality?

@mohamed-ali
Copy link
Author

@ChaiBapchya I tested locally and it works fine. I could set up, by tomorrow, a sample book on github pages that uses this version to see if everything works fine.

@ChaiBapchya
Copy link

Sure that will be awesome! Thanks!

@mohamed-ali
Copy link
Author

mohamed-ali commented Dec 29, 2020

Sorry I didn't have time earlier to setup a demo.

Here is a link to a demo using this branch to display RTL text

https://mohamed-ali.github.io/d2l-ar-demo/

@ChaiBapchya
Copy link

Looks good to me! Few pages aren't translated yet i guess [for e.g. https://mohamed-ali.github.io/d2l-ar-demo/chapter_introduction/index.html#id9] but I'm guessing this being a sample demo we don't need full coverage.

Copy link

@ChaiBapchya ChaiBapchya left a comment

Choose a reason for hiding this comment

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

Thanks!

@ChaiBapchya
Copy link

@mli good to merge?

@mli mli merged commit 98ac80b into mli:master Mar 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants