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

Notes encoding #20

Closed
tomaspaukste opened this issue Mar 22, 2021 · 4 comments
Closed

Notes encoding #20

tomaspaukste opened this issue Mar 22, 2021 · 4 comments

Comments

@tomaspaukste
Copy link

Hi, firstly - great app :) But found one place to improve - when saving files on Windows it uses Windows-XXXX encoding and Linux line endings (LF). It would be great to change it all to simple utf-8 encoding or to have ability choose it (in app settings or in some config file). Because it's almost impossible to open created note files (when they have non English letters inside) with other simple text editors (only in npp with choosing settings). Thanks!

@Ogefest
Copy link
Owner

Ogefest commented Mar 22, 2021

Hello my first issue reporter :)

I only want to confirm, you mean file content encoding? I'm linux user day by day and I was pretty sure that utf8 is common standard for encoding. Add default encoding in workspace settings seems be quite easy to do, if it's improve Notepack it make sense for me.

@tomaspaukste
Copy link
Author

Hi, yes I mean file encoding. It seems that app gets default encoding from Windows locale settings. I'm using Lithuanian settings, so files are saved with character set Windows-1257.

But here you need to be careful, maybe add this utf-8 setting as optional, because if you will set it as default, current users after app update will not see their non-English symbols correctly.

Also another remark - Windows/Linux/Macintosh each system has different text line ending symbol (Windows - CR LF, Linux LF, Macintosh - CR), so it would be reasonable to follow it. Now current file in Windows has Linux line endings LF.

@Ogefest
Copy link
Owner

Ogefest commented Mar 27, 2021

It take me more time than I thought before but I needed to prepare windows dev environment which was quite hard for long time linux user.

Anyway, I added extra checkbox in workspace configuration. If default encoding in system is UTF-8 like Ubuntu this checkbox is hidden but when system encoding is for example windows-1257 (I tested with windows-1250 - Poland here) then extra checkbox is available, if you set this on then all content saved is converting to UTF-8. Tested in many ways seems works as expected. You can check by yourself with 0.9.13 release

I have to think about end lines, for sure this could be an issue in some cases but right now I'm try to avoid overload settings in workspace configuration.

Thank you involve, if you have some ideas feel free to write them. I have plans to make Notepack best knowledge app for desktops

@Ogefest Ogefest closed this as completed Mar 27, 2021
@tomaspaukste
Copy link
Author

Thanks! Now it works well :)

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

No branches or pull requests

2 participants