- Notifications
You must be signed in to change notification settings - Fork5.5k
Fixes 3182#5598
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Fixes 3182#5598
Uh oh!
There was an error while loading.Please reload this page.
Conversation
Issue 3182 "Can't paste two images from clipboard in markdown cell"happens when the user tries to paste a second image into a textcell and the image gets the same attachment key as the previous one.This patch fixes that issue by adding an index to each picture that'spasted after the previous one. Example:* User pastes an image blob A. The text `` is added to the text cell and the blob is saved as attachment `image.png`* The user pastes a second image blob B. The same happens as with blob A, except the key is now `image-1.png` rather than `image.png`Special case:If the user pastes the exact same blob A twice, it will not be added tothe attachments, but the textcell will get a second entry of``.
withrocks commentedJul 10, 2020
I wasn't able to read the general Jupyter general contribution guidelines because the link is broken:https://jupyter.readthedocs.io/en/latest/contributor/content-contributor.html I hope that won't be an issue. |
kevin-bates commentedJul 10, 2020
Hi@withrocks - your PR content seems fine although I'm not qualified to review it. The broken link should point here:https://jupyter.readthedocs.io/en/latest/contributing/content-contributor.html I've opened issue#5599 for this. |
blink1073 left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
LGTM, thanks!
Uh oh!
There was an error while loading.Please reload this page.
Issue#3182 "Can't paste two images from clipboard in markdown cell"
happens when the user tries to paste a second image into a text
cell and the image gets the same attachment key as the previous one.
This patch fixes that issue by adding an index to each picture that's
pasted after the previous one. Example:
is added to the text cell and the blob is saved as attachment
image.pngA, except the key is now
image-2.pngrather thanimage.pngSpecial case:
If the user pastes the exact same blob A twice, it will not be added to
the attachments, but the textcell will get a second entry of
.