Skip to content

Allow default scaleanchor removal - #6712

Merged
archmoj merged 5 commits into
plotly:masterfrom
lvlte:allow-default-scaleanchor-removal
Aug 21, 2023
Merged

Allow default scaleanchor removal#6712
archmoj merged 5 commits into
plotly:masterfrom
lvlte:allow-default-scaleanchor-removal

Conversation

@lvlte

@lvlte lvlte commented Aug 20, 2023

Copy link
Copy Markdown
Contributor

Resolves #6709

@lvlte

lvlte commented Aug 20, 2023

Copy link
Copy Markdown
Contributor Author

Not sure what is happening with the 'make-baselines' test..

@archmoj

archmoj commented Aug 21, 2023

Copy link
Copy Markdown
Contributor

Thanks for the PR.
FYI - The baseline tests were now passed.

@archmoj archmoj added feature something new community community contribution status: reviewable labels Aug 21, 2023
@archmoj

archmoj commented Aug 21, 2023

Copy link
Copy Markdown
Contributor

It's looking very good to me.
If possible, please add draftlogs/6712_add.md file.
Thank you!

@archmoj
archmoj requested a review from alexcjohnson August 21, 2023 13:58

@alexcjohnson alexcjohnson left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💃 very nicely done @lvlte!

@archmoj
archmoj merged commit 9a039bd into plotly:master Aug 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community community contribution feature something new

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove the scaleanchor constrain on axes having an image trace

3 participants