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

(#2) Make the QR-code center-aligned #35

Merged
merged 1 commit into from
Nov 11, 2015
Merged

(#2) Make the QR-code center-aligned #35

merged 1 commit into from
Nov 11, 2015

Conversation

mjmasn
Copy link
Contributor

@mjmasn mjmasn commented Aug 2, 2015

Without this patch applied the QR-code will be aligned to the top-left of the output image/canvas. If the QR-code is smaller than the image size, there will be whitespace to the bottom-right of the image. This patch fixes this by calculating an offset to make the whitespace equal on all sides of the QR-code.

Fixes #2

@throrin19
Copy link
Contributor

+1

@neocotic
Copy link
Owner

neocotic commented Nov 9, 2015

The only problem that I have with this PR is that it makes too many unrelated changes to formatting. If these could be reverted so that only the necessary changes are included, I'll happily merge this in.

@mjmasn
Copy link
Contributor Author

mjmasn commented Nov 9, 2015

@neocotic done :)

@neocotic
Copy link
Owner

Great! I'll take a look at this tomorrow night. Sorry for the delay with this.

@neocotic neocotic added the bug label Nov 11, 2015
@neocotic neocotic added this to the 1.1.4 milestone Nov 11, 2015
@neocotic neocotic self-assigned this Nov 11, 2015
@neocotic neocotic merged commit b309653 into neocotic:master Nov 11, 2015
@neocotic
Copy link
Owner

@mjmasn I've merged your fix and it's been included in v1.1.4 which I've just published. Thanks for all your help!

@mjmasn
Copy link
Contributor Author

mjmasn commented Nov 12, 2015

Sweet, thanks!

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

Successfully merging this pull request may close these issues.

Padding issues
3 participants