Skip to content

Test graph PDF output with more than 1000 points - #3988

Open
lorenzozanee wants to merge 1 commit into
plotly:devfrom
lorenzozanee:fix/graph-print-large-data
Open

lorenzozanee wants to merge 1 commit into
plotly:devfrom
lorenzozanee:fix/graph-print-large-data

Conversation

@lorenzozanee

Copy link
Copy Markdown

Adds a deterministic browser integration fixture for 1000 and 1001 point graphs. The test emulates print media and exercises Chrome Page.printToPDF while preserving the existing graph and browser-log assertions.

The assertion verifies a valid non-empty PDF and a visible SVG trace; it does not inspect rendered PDF pixels, so cross-browser validation remains for CI and maintainers.

Contributor Checklist

  • I have broken down my PR scope into the following TODO tasks
    • task 1
    • task 2
  • I have run the tests locally and they passed. (refer to testing section in contributing)
  • I have added tests, or extended existing tests, to cover any new features or bugs fixed in this PR

optionals

  • I have added entry in the CHANGELOG.md
  • If this PR needs a follow-up in dash docs, community thread, I have mentioned the relevant URLS as follows
    • this GitHub #PR number updates the dash docs
    • here is the show and tell thread in Dash community

Fixes #3889

@sonarqubecloud

Copy link
Copy Markdown

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.

[BUG] Graphs excluded from print pages if they have over 1000 datapoints

1 participant