Skip to content
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

From Japanese input, created file names become ".md" #73

Closed
tkkuma-k opened this issue Jul 7, 2020 · 13 comments
Closed

From Japanese input, created file names become ".md" #73

tkkuma-k opened this issue Jul 7, 2020 · 13 comments
Labels
bug Something isn't working

Comments

@tkkuma-k
Copy link

tkkuma-k commented Jul 7, 2020

I love this product, and need your help.
When I put [[something]] and click this with cmd, new file named "something.md" is created.
However when I put Japanese character in [[]], like [[新規]], and clicked this with cmd, the created file's name will be ".md".

@jevakallio
Copy link
Collaborator

Hi @tkkuma-k! Thanks for reporting this. This issue is being tracked in VSCode Markdown Notes that implements the new file feature:
kortina/vscode-markdown-notes#47

@digiguru
Copy link
Contributor

digiguru commented Jul 8, 2020

I've added a potential fix downstream...

kortina/vscode-markdown-notes#53

But this will also need a fix in foam for the auto-generated links before this work in GitHub (but should work in vs code once merged)

@digiguru
Copy link
Contributor

digiguru commented Jul 8, 2020

@tkkuma-k - would you be able to help making the test more authentic? I have written a test that checks as follows....

'Some Title' => 'some-title.md'
'Šömè Țítlê' => 'šömè-țítlê.md'

And I've used google translate to create a test for the Japanese language. Can you tell me an authentic way to say "Some title" in Japanese?

'タイトル' => 'タイトル.md'

Thanks

@tkkuma-k
Copy link
Author

tkkuma-k commented Jul 8, 2020

@digiguru
Thank you for your help!
'タイトル' is correct Japanese.

"Some title" in kanji character ...
'題目' => '題目.md'

and we often use the full-width alphabet and number.
'wide123' => 'wide123.md'

@digiguru
Copy link
Contributor

digiguru commented Jul 8, 2020

Okay, my cultural assumptions have grown today, thankyou for enlightening me about Japanese Vs Kanji

I will add a test for full width alphabet, thanks!

@digiguru
Copy link
Contributor

digiguru commented Jul 8, 2020

(done)

@digiguru
Copy link
Contributor

digiguru commented Jul 8, 2020

Oh wait, what feels more correct here?

'Some Title' => 'some-title'
or
'Some Title' => 'some-title'

I'm guessing the second.

@tkkuma-k
Copy link
Author

tkkuma-k commented Jul 8, 2020

To me,
'Some Title' => 'some-title'
feels like a more natural expression, but
'Some Title' => 'some-title'
works , too.

@tkkuma-k
Copy link
Author

tkkuma-k commented Jul 8, 2020

Half-wide character, like タイトル, was used in old documents and is still sometimes used in official documents, despite that it is discouraged.
Someone may use this. Because we can input this.
Please add this too.

'タイトル' => 'タイトル.md'

@digiguru
Copy link
Contributor

digiguru commented Jul 8, 2020

Thankyou @tkkuma-k - I have checked and that works.

The full / half width of the slug will be a config option in
kortina/vscode-markdown-notes#53

so you can set it project wide depending on your needs. Currently you can choose between having no slug, or having - or _. If approved my change will add full width and _.

@tkkuma-k
Copy link
Author

tkkuma-k commented Jul 9, 2020

@digiguru
Thank you!
After I think again and I have an essential thing to tell you.
When we use the full-width alphabet, number, or kanji character, we also use full-width space ' ', not normal space ' '.
I think these two spaces are different. Is this right?
If so, I think it is a better way that normal space ' ' will be '-' or '_', and full-width space ' ' will not be changed.

'Some Title' => 'some_title.md'
'Some Title' => 'some_title.md'
'Some Title' => 'some title.md'

We may not need full width - and _ options.
Thank you!

@jevakallio
Copy link
Collaborator

Fixed by Markdown Notes v0.0.11! Thanks @digiguru for implementation, @tkkuma-k for testing and feedback, and @kortina for merging and releasing!

https://github.com/kortina/vscode-markdown-notes/releases/tag/v0.0.11

@riccardoferretti
Copy link
Collaborator

Due to the high volume of issues, we're closing out older ones without recent activity. Please open a new issue if you need help!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants