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

Allow setting additional cards to print #275

Merged
merged 17 commits into from
Aug 15, 2023

Conversation

guicamest
Copy link
Contributor

@guicamest guicamest commented Aug 11, 2023

Fixes #269, #63, #31, #30.

There is now a setting to increase the amount of cards to be printed. The wifi-cards are displayed without any changes in size (i.e. it is NOT an amount of cards per page setting). Additionally, users can choose to hide the tip (or legend) to possibly fit more cards in one page. Some margins and rendering were changed to avoid unnecessary empty space in the final printing layout

Website now looks like this:
Screenshot 2023-08-14 201035

Attaching some previews with A4:

3 Landscape cards fit into portrait printed page
land 3 - portrait

4 Landscape cards fit into landscape printed page
land 4 - landscape

4 Landscape cards (without password nor legend) fit into portrait printed page
land 4 no pass no legend - portrait

3 Rotated cards fit into landscape printed page
portrait 3 - landscape print

4 Rotated cards fit (without password) into portrait printed page
portrait 4 no pass

4 Rotated cards fit (without tip) into portrait printed page
portrait 4 no tip

4 Rotated cards fit into portrait printed page
portrait 4

6 Rotated cards fit (without password nor tip) into landscape printed page
portrait 6 no pass no tip - landscape print

6 Rotated cards fit (without password nor tip) into portrait printed page
portrait 6 no pass no tip

@bndw
Copy link
Owner

bndw commented Aug 11, 2023

This is awesome! Looking forward to reviewing and giving this a try in the next couple of days.

Copy link
Owner

@bndw bndw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One comment about the positioning of the new checkbox; with that change in place I'd be happy to merge!

src/components/Settings.js Outdated Show resolved Hide resolved
@guicamest guicamest requested a review from bndw August 14, 2023 18:12
@bndw bndw merged commit f1d79a1 into bndw:master Aug 15, 2023
1 check passed
@bndw
Copy link
Owner

bndw commented Aug 15, 2023

Thanks for this @guicamest. It may take me some time to deploy this update to wificard.io, apologies in advance.

@guicamest
Copy link
Contributor Author

No problem @bndw ! Happy to have contributed to this great open source project!

I've already used it to print some locally; I guess docker users have it available too by now 👍

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.

Feature Request > Suggestions for Space Usage/Efficiency on the Resulting Papers.
2 participants