Marker generator

[floogulinc]

https://buzz-about-git-marker-generator.mn-pollinators.vercel.app/markers

This is currently working off the assumption that we change to AR_MATRIX_CODE_5x5_BCH_22_7_7 type markers, see #124.

[vercel]

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/mn-pollinators/buzz-about/337quhptd
✅ Preview: https://buzz-about-git-marker-generator.mn-pollinators.vercel.app

[floogulinc]

This will need to be changed to use AR_MATRIX_CODE_4x4_BCH_13_9_3. See #128

[cypress]

---

Test summary

68 • 0 • 0 • 0

---

Run details

Project	Buzz About
Status	Passed
Commit	6e0739b
Started	Jan 3, 2021 12:24 AM
Ended	Jan 3, 2021 12:27 AM
Duration	02:46 💡
OS	Linux Ubuntu - 18.04
Browser	Electron 83

View run in Cypress Dashboard ➡️

---

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

[floogulinc]

One thing to keep in mind is that just adding the pdfmake library has increased our main bundle size by about 1mb. It might be a good time to look at code-splitting so we can start only loading these large libraries (pdfmake, three-ar, etc) when needed. (#181)

[floogulinc]

We should look at browser support and potentially disable the print button where it's not supported.

See bpampuch/pdfmake#800 and https://pdfmake.github.io/docs/0.1/getting-started/client-side/supported-browsers/

[helloworld12321]

Works like a charm—thanks, Paul!

I've made a few review suggestions—it's mostly

• rewording one thing in the UI
• removing some unused methods
• a bit of documentation
• narrowing the type-checking on one field
• some indentation + whitespace

None of that is super critical, though 🙂

(Also, if you disagree with any of the suggestions, I'd be happy to leave things as they are!)

---

By the way, thanks a million for getting all the PDF stuff done—looking at the libraries you're using, I'm reminded just how much I don't like working with PDFs, and how glad I am that I didn't have to implement any of this.

[helloworld12321]

Browser compatibility looks pretty good, by the way—I tested on Firefox, desktop Safari, and mobile Safari.

[helloworld12321]

In the future, if we need to add anything to the PDF generation, I could see moving it to its own service. Right now, though, I think it's probably fine where it is 🙂

[helloworld12321]

Awesome, thank you! I'm going to give this a quick look over to make sure everything still works, and then I'll approve it :)

[helloworld12321]

Looks fantastic—thanks, Paul.