Skip to content

Conversation

@Schefflera-Arboricola
Copy link

@Schefflera-Arboricola Schefflera-Arboricola commented Dec 3, 2025

fixes #4500

Overview: What does this pull request change?

Replaced fill_color= with color= in all MathTex() calls throughout the codebase

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

Before: MathTex("x^2", fill_color=BLACK) --> white text
After: MathTex("x^2", color=BLACK) --> black text

white node labels on white node --> labels were not visible.

Links to added or changed documentation pages

https://manimce--4501.org.readthedocs.build/en/4501/reference/manim.mobject.graph.Graph.html -- here you can see the labels showing up in the examples (LabeledModifiedGraph example)

Further Information and Comments

NA

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

@Schefflera-Arboricola Schefflera-Arboricola marked this pull request as ready for review December 3, 2025 21:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 🆕 New

Development

Successfully merging this pull request may close these issues.

Graph: node labels' color defaulting to white instead of black

1 participant