Skip to content

Cannot show images with encoded path #715

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

Closed
tewe opened this issue Oct 17, 2019 · 12 comments
Closed

Cannot show images with encoded path #715

tewe opened this issue Oct 17, 2019 · 12 comments
Assignees

Comments

@tewe
Copy link

tewe commented Oct 17, 2019

Describe the bug

![](assets/Screen%2520Shot%25202019-01-31%2520at%252014.00.57.png) shows up as a missing image

To Reproduce
Steps to reproduce the behavior:

  1. Find an image with whitespace in the filename (as created by the macOS screen shot feature)
  2. Drag it into a note
  3. Toggle the preview off an on

Expected behavior
A preview of the image is displayed.

Actual behavior
The image shows up when first adding it, but disappears the next time the preview is rendered.

Workaround
Don't use spaces in file names.

Desktop:

  • OS: macOS
  • OS version: 10.13.6
  • FSNotes version: 3.9.1

Additional context
Looking at the note in an external editor I saw a correct image link, with the spaces in the filename URL-encoded as %20. It seems that FSNotes cannot reverse its own encoding.

@glushchenko
Copy link
Owner

Thanks, fixed

glushchenko added a commit that referenced this issue Oct 19, 2019
@tewe
Copy link
Author

tewe commented Oct 20, 2019

Your fix seems to stop encoding URLs inserted by FSNotes. But it still wouldn't be able to preview encoded URLs that were imported, pasted or even typed manually, right?

@glushchenko
Copy link
Owner

All images and file names should be encoded. If not — preview will not work

Sent with GitHawk

glushchenko added a commit that referenced this issue Oct 26, 2019
@tewe
Copy link
Author

tewe commented Dec 31, 2019

I'm afraid it still doesn't work. The following textbundle was created by dragging images into FSNotes 4.0.12. They all show up fine in https://marked2app.com but not at all in FSNotes.

![Apple Notes Print Dialog](assets/Notes%20Print%20Scale%20to%20Percentage.png)

![Apple Notes Preset Dialog](assets/Notes%20Preset%20Scale%20to%20Percentage.png)

![](assets/Preview%20Preset%20Scale%20to%20Image.png)

![](assets/Preview%20Preset%20Scale%20to%20Percentage.png)

![](assets/Preview%20Print%20Scale%20to%20Image.png)

![](assets/Preview%20Print%20Scale%20to%20Percentage.png)

@glushchenko glushchenko reopened this Jan 9, 2020
@glushchenko
Copy link
Owner

glushchenko commented Jan 9, 2020

Fixed in 4.0.15. Please try and let me know.

@tewe
Copy link
Author

tewe commented Jan 10, 2020

With 4.0.15 the images show up while editing (when at least I would prefer the Markdown instead) and still do not show up in the preview.

@glushchenko
Copy link
Owner

Please provide test file, I can not reproduce this.

@tewe
Copy link
Author

tewe commented Jan 10, 2020

I can easily reproduce it by dragging a screenshot into FSNotes, or using the snippet above. Here is a compressed text bundle.

glushchenko added a commit that referenced this issue Jan 10, 2020
@glushchenko
Copy link
Owner

@tewe
Copy link
Author

tewe commented Jan 10, 2020

That build does show images in preview! They have a context menu with "Open Image in New Window", which does nothing. Images also still show up when editing, making it impossible to edit e.g. the alternative text in their Markdown.

@glushchenko
Copy link
Owner

Use ⌘ + mouse click on images for editing Title

@tewe
Copy link
Author

tewe commented Jan 11, 2020

Cmd+click works. Thanks for fixing this.

But I believe that any attempt at rich text editing of Markdown is misguided. So now that FSNotes is looking more like Bear I'll start looking for an alternative.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants