jplattel / obsidian-clipper

A Chrome extension that easily clips selections to Obsidian
https://jplattel.github.io/obsidian-clipper/
744 stars 60 forks source link

Not clipping on all pages - bug? Feature Request? #47

Open malevans opened 2 years ago

malevans commented 2 years ago

Describe the bug

Not clipping pages with illegal characters in title.

The extension very often fails to create the note in Obsidian. I can see it opens Obsidian but then fails.

I can see in the console that it gives the following error

'File name cannot contain any of the following characters: * " \ / < > : | ?'

all the examples I have noted have these special characters in the title.

I have the extension configured to clip the note to

@Inbox/{title} - {date}

I guess that is the root cause but not sure if there is anyway to prevent the error while keeping the title as part of the note title.

Is it possible to define a valid character to use as a replacement for the illegal characters in the note title?

Or is there a workaround? At the moment I have to only use the {datetime} to create the note title but that adds extra steps to the note creation process.

Appreciate any thoughts,

Cheers

ivanvmoreno commented 2 years ago

Can definitely replicate this behaviour - definitely noticeable in e.g. YouTube videos, which tend to heavily use the pipe | character on their titles.

In case it may be useful, the Obsidian extension Auto Link Title has worked out the logic to parse titles containing any of the characters * " \ / < > : | ?, replacing them with a whitespace character.

malevans commented 2 years ago

Yes I use Auto Link Title already and it works fine. Would be useful to be able to replicate that logic if possible.

On Mon, Apr 25, 2022 at 1:44 AM Iván Moreno @.***> wrote:

Can definitely replicate this behaviour - definitely noticeable in e.g. YouTube videos, which tend to heavily use the pipe | character on their titles.

In case it may be useful, the Obsidian extension Auto Link Title https://github.com/zolrath/obsidian-auto-link-title has worked out the logic to parse titles containing any of the characters * " \ / < > : | ?, replacing them with a whitespace character.

— Reply to this email directly, view it on GitHub https://github.com/jplattel/obsidian-clipper/issues/47#issuecomment-1107866436, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACOXZNP4ISDLB35RCUXJFULVGVT35ANCNFSM5TUNVW3Q . You are receiving this because you authored the thread.Message ID: @.***>

-- Cheers Mal Evans 0404 542 054

http://au.linkedin.com/in/malevans

zinzinzibidi commented 2 years ago

Same for me. Doesn't work on medium.com and on most of web site.