Skip to content

Combining charts leads to tag duplication #374

New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom
Closed
bvenn opened this issue Mar 14, 2023 · 2 comments
Closed

Combining charts leads to tag duplication #374

bvenn opened this issue Mar 14, 2023 · 2 comments
Labels
Area: Core API Type: Bug
Milestone
Plotly.NET 5.0

Comments

@bvenn
Copy link
Collaborator

bvenn commented Mar 14, 2023

Description

When combining charts using Chart.combine or Chart.Grid, title, meta and link tags in the html head are duplicated.

Repro steps

Please provide the steps required to reproduce the problem

  1. Generate combined chart
#r "nuget: Plotly.NET, 4.0.0"

open Plotly.NET

[
    Chart.Point([1,2])
    Chart.Point([1,2])
]
|> Chart.combine
|> Chart.show
  1. Look at the html

image
@kMutagene
Copy link
Collaborator

kMutagene commented Mar 14, 2023 •
edited
Loading

For reference, it might be a bad idea to append head tags on combination at all. When a combined chart should have additional tags, they should be set after combination

@kMutagene kMutagene added this to the Plotly.NET 5.0 milestone Jul 9, 2023
@kMutagene
Copy link
Collaborator

fixed for the current default head tags. Might be worth to model mathjax reference the same way we currently model the plotly.js reference instead of adding it to additional head tags.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Area: Core API Type: Bug
Projects
None yet
Milestone
Plotly.NET 5.0
Development

No branches or pull requests
2 participants
@kMutagene @bvenn