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

Replace default font #15

Open
AlexanderTserkovniy opened this issue Oct 28, 2017 · 7 comments
Open

Replace default font #15

AlexanderTserkovniy opened this issue Oct 28, 2017 · 7 comments
Milestone

Comments

@AlexanderTserkovniy
Copy link

Hello,

Basically if you look at the characters of the font Comismsh (which is default), there is a ground issue.

Letter "l" (lowercase letter "L") is completely the same as "I" (uppercase letter "i"). While it is not an issue in development or article text, this is a huge problem in captcha, because you never know what is that symbol for sure.

http://prntscr.com/h32f17

Thanks.

@steambap
Copy link
Contributor

One thing you can do is to use the charPreset option to define a set of character that will be used in random character generation.

Personally, I do not think this will be fixed by using another font. For most fonts I worked with, it is hard to distinguish between 0 and letter O. However, I'm also interest in your opinion. Do you have a font suggestion? What is the license of the font?

@AlexanderTserkovniy
Copy link
Author

Hmm, thanks for the suggestion. It could a solution for this issue ticket as well.

Yes agree, basically that font (Slabo27px-Regular.ttf), which I have loaded instead of Comismsh also contains 0 and O problem. But I would say it is something, which could be understood eventually. But agree that it is still a problem.

As suggestions for fonts here are:

Ideal, just type o0, O0, Il in the corresponding input
https://fonts.google.com/specimen/Roboto+Slab
Apache License, Version 2.0

Ideal
https://fonts.google.com/specimen/Droid+Serif
Apache License, Version 2.0

Distinguishable
https://fonts.google.com/specimen/Lora
Open Font License

Less ideal, has problem with o0
https://fonts.google.com/specimen/Playfair+Display
Open Font License

@steambap
Copy link
Contributor

Correct me if I'm wrong, Apache license v2 requires this software to be licensed under Apache license also. So I cannot take Roboto and Droid.

Lora looks good to me. I'll consider use it to version 2.x.

@AlexanderTserkovniy
Copy link
Author

I am not sure, but I have not noticed that point in the licence agreement.

Thanks for the consideration.

@steambap
Copy link
Contributor

steambap commented Nov 2, 2017

I think the paragraph after section 4.d state that:

You may add Your own copyright statement to Your modifications and may provide additional or different license terms and conditions for use, reproduction, or distribution of Your modifications, or for any such Derivative Works as a whole, provided Your use, reproduction, and distribution of the Work otherwise complies with the conditions stated in this License.

@AlexanderTserkovniy
Copy link
Author

Not sure if it aligns with the statement, which you wrote:

Correct me if I'm wrong, Apache license v2 requires this software to be licensed under Apache license also.

I believe here is written quite the opposite thing, meaning that you "may" (may/may not, just as you wish) add your changes in any part of license agreement and add it to the license text, which will go alongside with the font.

@steambap
Copy link
Contributor

You are right. I guess I just need put more thought into font switch.

@lichaozhy lichaozhy added this to the 3.x milestone Oct 27, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

3 participants