FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

fix: save_png doesn't work when character \xa0 is used by mariobuikhuizen · Pull Request #1644 · bqplot/bqplot · GitHub

/ bqplot Public

fix: save_png doesn't work when character \xa0 is used - #1644

Merged
maartenbreddels merged 1 commit into
bqplot:masterfrom
mariobuikhuizen:fix_export_nbsp
May 21, 2024
ConversationCommits (1)ChecksFiles changed
Merged

fix: save_png doesn't work when character \xa0 is used#1644
maartenbreddels merged 1 commit into
bqplot:masterfrom
mariobuikhuizen:fix_export_nbsp

Conversation

mariobuikhuizen commented Apr 30, 2024

Copy link
Copy Markdown
Contributor

The character \xa0 is converted to   when .outerHTML() is called, causing an error in the data-URL where the outer HTML is used.

Code changes

Revert &nbps;, generated by .outerHtml(), back to the original character \xa0.

User-facing changes

save_png() will now work when \xa0 is used.

Steps to reproduce

Run the following:

from bqplot import *

x_sc = LinearScale()
y_sc = LinearScale()

scatter_chart = Scatter(x=x_data, y=y_data, scales={'x': x_sc, 'y': y_sc})

x_ax = Axis(label='X', scale=x_sc)
y_ax = Axis(label='Y', scale=y_sc, orientation='vertical')

fig = Figure(title="Ti \xa0 tle", marks=[scatter_chart], axes=[x_ax, y_ax])
fig
fig.save_png("fig.png")

Notice the save dialog is not appearing.

The dataURL produced with  :

fix: save_png doesn't work when character \xa0 is used
449e5b5 
The character \xa0 is converted to   when .outerHTML() is
called, causing an error in the data-URL where the outer HTML is
used.

mariobuikhuizen commented May 8, 2024

Copy link
Copy Markdown
Contributor Author

Some other unicode characters are converted to entites, like \u2003 ->   and
\u2009 ->  , but they just work.

maartenbreddels merged commit 4d5c293 into bqplot:master May 21, 2024

maartenbreddels commented May 21, 2024

Copy link
Copy Markdown
Member

We looked into this a bit, but documentation is scarce on this, and this is at least an improvement. Thanks @mariobuikhuizen

maartenbreddels commented May 21, 2024

Copy link
Copy Markdown
Member

meeseeksdev please backport to 0.12.x

lumberbot-app Bot commented May 21, 2024

Copy link
Copy Markdown

Owee, I'm MrMeeseeks, Look at me.

There seem to be a conflict, please backport manually. Here are approximate instructions:

  1. Checkout backport branch and update it.
git checkout 0.12.x
git pull
  1. Cherry pick the first parent branch of the this PR on top of the older branch:
git cherry-pick -x -m1 4d5c2932d8d55e1af96e7129f0550c18c0c3b977
  1. You will likely have some merge/cherry-pick conflict here, fix them and commit:
git commit -am "Backport PR #1644: fix: save_png doesn't work when character \\xa0 is used"
  1. Push to a named branch:
git push YOURFORK 0.12.x:auto-backport-of-pr-1644-on-0.12.x
  1. Create a PR against branch 0.12.x, I would have named this PR:

"Backport PR #1644 on branch 0.12.x (fix: save_png doesn't work when character \xa0 is used)"

And apply the correct labels and milestones.

Congratulations — you did some good work! Hopefully your backport PR will be tested by the continuous integration and merged soon!

Remember to remove the Still Needs Manual Backport label once the PR gets merged.

If these instructions are inaccurate, feel free to suggest an improvement.

maartenbreddels pushed a commit to maartenbreddels/bqplot that referenced this pull request May 21, 2024
fix: save_png doesn't work when character \xa0 is used (bqplot#1644)
27bff04 
The character \xa0 is converted to   when .outerHTML() is
called, causing an error in the data-URL where the outer HTML is
used.

(cherry picked from commit 4d5c293)
maartenbreddels pushed a commit to maartenbreddels/bqplot that referenced this pull request May 21, 2024
Backport PR bqplot#1644: fix: save_png doesn't work when character \x…
0461ef6 
…a0 is used
maartenbreddels pushed a commit to maartenbreddels/bqplot that referenced this pull request Nov 26, 2024
Backport PR bqplot#1644: fix: save_png doesn't work when character \x…
1027a31 
…a0 is used
maartenbreddels pushed a commit to maartenbreddels/bqplot that referenced this pull request Feb 11, 2025
Backport PR bqplot#1644: fix: save_png doesn't work when character \x…
20627e4 
…a0 is used
maartenbreddels pushed a commit to maartenbreddels/bqplot that referenced this pull request Feb 11, 2025
Backport PR bqplot#1644: fix: save_png doesn't work when character \x…
d360e05 
…a0 is used
maartenbreddels pushed a commit that referenced this pull request Feb 11, 2025
Backport PR #1644: fix: save_png doesn't work when character \xa0 is …
5d885a4 
…used
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

Still Needs Manual Backport

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

Back | FazBrowse Home | New Git URL