Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fixing a bug with label color in graph not applying. #4188

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

barollet
Copy link

Overview: What does this pull request change?

Fixes a bug with label color in Graph Mobject being always white (the default color for MathTex) because of wrong argument name.

Motivation and Explanation: Why and how do your changes improve the library?

I like to have good argument names so we can actually use it haha.

Links to added or changed documentation pages

Further Information and Comments

Actually I thought my last PR with 4 lines diff would be the smallest I do, but this one is actually smaller. Maybe I'll manage to make a 1 line PR at some point.

Reviewer Checklist

  • The PR title is descriptive enough for the changelog, and the PR is labeled correctly
  • If applicable: newly added non-private functions and classes have a docstring including a short summary and a PARAMETERS section
  • If applicable: newly added functions and classes are tested

@barollet
Copy link
Author

TestScene.mp4

Better with colors, compare with actual documentation https://docs.manim.community/en/stable/reference/manim.mobject.graph.Graph.html#labeledmodifiedgraph

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
Status: 🆕 New
Development

Successfully merging this pull request may close these issues.

1 participant