Set axis linecolor to black by default when converting from matplotlib by robertoffmoura · Pull Request #5311 · plotly/plotly.py

Skip to content

Navigation Menu

Sign in

Appearance settings

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Conversation

@robertoffmoura

Copy link Copy Markdown

Contributor

  • I have read through the contributing notes and understand the structure of the package. In particular, if my PR modifies code of plotly.graph_objects, my modifications concern the code generator and not the generated files.
  • I have added tests or modified existing tests.
  • For a new feature, I have added documentation examples (please see the doc checklist as well).
  • I have added a CHANGELOG entry if changing anything substantial.
  • For a new feature or a change in behavior, I have updated the relevant docstrings in the code.

@gvwilson gvwilson requested a review from emilykl

August 11, 2025 17:07

@gvwilson gvwilson added P1

needed for current cycle

community

community contribution

fix

fixes something broken

labels

Aug 11, 2025

@robertoffmoura robertoffmoura force-pushed the rm/default-axis-linecolor-black branch from e65619c to 0335226 Compare

August 12, 2025 09:31

@emilykl

Copy link Copy Markdown

Contributor

emilykl commented

Oct 1, 2025

@robertoffmoura Thanks for the contribution!

I'm happy to merge this but just to make sure I have the full context, can you explain why this is the correct behavior? Are the axes lines black by default in matplotlib?

@robertoffmoura

Copy link Copy Markdown

Contributor Author

Hi @emilykl,
Yes, the axes lines are black by default in matplotlib.

@robertoffmoura robertoffmoura force-pushed the rm/default-axis-linecolor-black branch from 0335226 to 9e6fea8 Compare

October 9, 2025 15:46

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@emilykl emilykl Awaiting requested review from emilykl

At least 1 approving review is required to merge this pull request.

Assignees

@emilykl emilykl

Labels

community

community contribution

fix

fixes something broken

P1

needed for current cycle

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@robertoffmoura @emilykl @gvwilson