Skip to content

Conversation

@camdecoster
Copy link
Contributor

@camdecoster camdecoster commented Nov 11, 2025

Description

Disallow borders in plot container in CSS reset.

Changes

  • Add borders of zero to CSS reset for plot container

Demo video or screenshots

Before After
image image

Testing

  • Be on this branch
  • Edit devtools/test_dashboard/style.css to include the following CSS:
    div {
      border: 1px red solid;
    }
  • Load a mock in Plotly DevTools
  • Note that the plot container has lots of red borders.
  • The changes from this PR need to be compiled to see the changes. Run npm run preprocess to compile the CSS.
  • Load a mock in Plotly DevTools
  • Note that the plot container has no red borders

@camdecoster camdecoster marked this pull request as ready for review November 11, 2025 01:46
@camdecoster camdecoster requested a review from emilykl November 11, 2025 01:46
@camdecoster camdecoster changed the title Disallow borders on plot container fix: Disallow borders on plot container Nov 12, 2025
@camdecoster camdecoster merged commit 1a52bbc into master Nov 12, 2025
6 checks passed
@camdecoster camdecoster deleted the cam/disallow-borders-graph-div branch November 12, 2025 20:14
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.

3 participants