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

✨ IMPROVE: improve math styling #369

Merged
merged 1 commit into from
Aug 7, 2021

Conversation

akhmerov
Copy link
Contributor

@akhmerov akhmerov commented Aug 7, 2021

Closes #345, #272.

I didn't figure out how to set up the dev environment, so the compiled assets aren't updated. I hope CI picks it up.


Some motivation:

@welcome
Copy link

welcome bot commented Aug 7, 2021

Thanks for submitting your first pull request! You are awesome! 🤗

If you haven't done so already, check out EBP's Code of Conduct and our Contributing Guide, as this will greatly help the review process.

Welcome to the EBP community! 🎉

@akhmerov akhmerov marked this pull request as draft August 7, 2021 21:27
@akhmerov
Copy link
Contributor Author

akhmerov commented Aug 7, 2021

Doesn't seem to work on either count :(

@choldgraf
Copy link
Member

What was the sticking point for setting up the dev environment?

@akhmerov
Copy link
Contributor Author

akhmerov commented Aug 7, 2021

What was the sticking point for setting up the dev environment?

Nothing I couldn't handle, just the sheer the amount of tools that I don't (yet) regularly use to install and learn, compared to the expected amount of changes (I incorrectly expected this to be an easy fix).

It was also not completely clear what it'd take—I'd need to install pre-commit to compile the assets, that would in turn install the EBP pre-commit hook, and that will bring in require some minification tools (?).

@choldgraf
Copy link
Member

choldgraf commented Aug 7, 2021

Are you going off of the instructions on the contributing page?

I think the main thing is just to do:

pip install tox    (installs tox)
tox -e docs-update   (builds the documentation from the local copy)

that should be all you need to do. If you want pre-commit to run when you commit things, then first install pre-commit and do:

pre-commit install

and it'll run from then on

@akhmerov akhmerov marked this pull request as ready for review August 7, 2021 22:07
@akhmerov
Copy link
Contributor Author

akhmerov commented Aug 7, 2021

Thanks, this worked, and helped me figure out the issues.

@akhmerov
Copy link
Contributor Author

akhmerov commented Aug 7, 2021

With modifications (here I added an equation without a label as a check):


image


Previously:


image

@choldgraf choldgraf changed the title improve math styling ✨ IMPROVE: improve math styling Aug 7, 2021
@choldgraf choldgraf merged commit 2416ef6 into executablebooks:master Aug 7, 2021
@welcome
Copy link

welcome bot commented Aug 7, 2021

Congrats on your first merged pull request in this project! 🎉
congrats

Thank you for contributing, we are very proud of you! ❤️

@choldgraf
Copy link
Member

this looks great - thanks for improving this!

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.

Equation label misaligned
2 participants