Skip to content

Backport PR #1539: fix: figure is not updating when margins change#1581

Merged
maartenbreddels merged 1 commit intobqplot:0.12.xfrom
mariobuikhuizen:auto-backport-of-pr-1539-on-0.12.x
Mar 14, 2023
Merged

Backport PR #1539: fix: figure is not updating when margins change#1581
maartenbreddels merged 1 commit intobqplot:0.12.xfrom
mariobuikhuizen:auto-backport-of-pr-1539-on-0.12.x

Conversation

@mariobuikhuizen
Copy link
Copy Markdown
Contributor

References

Code changes

User-facing changes

Backwards-incompatible changes

@maartenbreddels
Copy link
Copy Markdown
Member

@martinRenou what do you think, should we port https://github.com/bqplot/bqplot/pull/1550/files to 0.12.x?

@martinRenou
Copy link
Copy Markdown
Member

Yes we should backport some of the recent PRs, I can look into it

@maartenbreddels maartenbreddels force-pushed the auto-backport-of-pr-1539-on-0.12.x branch from cb603ed to ab70701 Compare March 14, 2023 20:08
@maartenbreddels
Copy link
Copy Markdown
Member

@mariobuikhuizen needs a prettifier run

@martinRenou maybe we should let the rest of CI run, even if prettifier fails (so it's a sub step), and maybe have pre commit.. ?

@mariobuikhuizen mariobuikhuizen force-pushed the auto-backport-of-pr-1539-on-0.12.x branch from ab70701 to c7d8b95 Compare March 14, 2023 20:41
@maartenbreddels maartenbreddels merged commit c6685b5 into bqplot:0.12.x Mar 14, 2023
@maartenbreddels
Copy link
Copy Markdown
Member

awesome!

@pllim
Copy link
Copy Markdown
Contributor

pllim commented Mar 14, 2023

Do you remember which Jdaviz ticket this was supposed to fix? I lost track. 😬

@mariobuikhuizen
Copy link
Copy Markdown
Contributor Author

It's this one: spacetelescope/jdaviz#1671.

@pllim
Copy link
Copy Markdown
Contributor

pllim commented Mar 14, 2023

Great, thanks! Can you please let us know when this patch is released and we can then update the bqplot pin downstream? 🙏

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.

4 participants