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

More options for image size #21

Closed
wants to merge 1 commit into from

Conversation

steffansluis
Copy link

Why is this a discrete variable and not just a pixel value? In any case, this very simple addition allows for bigger images without breaking anything. Apologies for not following the contributing guidelines. Due to the simple nature of the change I did not feel it would add in any way to the value of the change.

@neocotic
Copy link
Owner

The reason is because the aim of the first release (and the repo as a whole) was to create a usable API wrapper on top of tz's jsqrencode project. Since then I've just been working on improving the API, without really changing the guts.

Although increasing the number of available sizes seems like a good fix, there are some scaling problems that become more apparent at larger sizes. I raised #2 long ago to track this, but I've not had time to. The problem is that the margin/padding increases as the size increases causing the actual QR code to not fill the area.

For this reason, I'm hesitant to support larger sizes until that issue's fixed, otherwise it will no doubt result in duplicate issues being raised. However, once this has been done, I will probably support pixel dimensions to be specified as well as scaling.

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

Successfully merging this pull request may close these issues.

2 participants