Skip to content
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.

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

Matplotlib image has extremely small height on margin in html #6961

Closed
finkelshtein opened this issue Sep 23, 2023 · 2 comments
Closed

Matplotlib image has extremely small height on margin in html #6961

finkelshtein opened this issue Sep 23, 2023 · 2 comments
Assignees
Labels
bug Something isn't working
Milestone

Comments

@finkelshtein
Copy link

Bug description

Hi,
It may be related to #6865 which closed the issue I raised in #6863.
#6865 fixed the issue for PDF, however, now the image is not visible in HTML (it has a very small height).

Steps to reproduce

---
title: "matplotlib demo"
format:
  html: 
      self-contained: true
      grid: 
          margin-width: 350px
  pdf: default
jupyter: python3
---

Demo

```{python}
#| echo: false
#| column: margin

import matplotlib.pyplot as plt

plot = plt.figure()
plot.set_figwidth(2)
plot.set_figheight(3)

x = [1, 2]
y = [2, 3]

plt.plot(x,y)

```

Expected behavior

No response

Actual behavior

issue

Your environment

  • Quarto 1.4.376
  • IDE: VS Code 1.82.2
  • Windows 10

Quarto check output

No response

@finkelshtein finkelshtein added the bug Something isn't working label Sep 23, 2023
@finkelshtein
Copy link
Author

Just to add, the bug is also reproducible on the Gallery example
https://github.com/quarto-dev/quarto-gallery/blob/main/page-layout/tufte.qmd
It seems that, in that example, "Full width" feature does not work either.

@cscheid cscheid self-assigned this Sep 23, 2023
@cscheid cscheid added this to the v1.4 milestone Sep 23, 2023
@cscheid
Copy link
Collaborator

cscheid commented Sep 23, 2023

Thanks for the report. I can repro this. It's a regression from a recent big change, and our tests aren't covering it properly. I'll fix right away.

@cscheid cscheid added the verify-fixed Already fixed, awaiting verification from the issue reporter to confirm. label Sep 29, 2023
@cscheid cscheid removed the verify-fixed Already fixed, awaiting verification from the issue reporter to confirm. label Sep 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants