Reason for MAX_WINDOW_HEIGHT

[czurnieden]

is there a technical reason to restrict MAX_WINDOW_HEIGHT to 600px for non-mobile environments? I found it a tad bit too small if looked at from about 10m (don't know how much that is in old money, 30 feet?) with a "normal" smartphone. The flagships of Apple and Samsung have no problems reading it, anything below has problems. I am aware that his use-case is really rare and I can change it myself for...uhm...myself but I like to ask first, before I dive deep into a debugging session because of some minor technical issue that I was not aware of.
Oh, and: thank you very much for the work, this app is really useful! for avoiding the annoying "Can you send me the link, please?!"

[rugk]

Ref

offline-qr-code/src/popup/modules/UserInterface.js

Line 34 in 543e91d

export const MAX_WINDOW_HEIGHT = 600;

Do you have a screenshot (or photo, after all in this case to see whether it is "too small" related to the device) and what device do you use? It makes lot of difference if you have a TV or smartphone or whatever.

Thus. please mind to fill out the issue template you choose, when you open an issue the next time. This makes it easier to troubleshoot issues or understand feature requests etc.

Apart from that, it was apparently introduced by @lvchamao in #353 to fix #35 (Note that this change is not yet released on AMO, so do you use the dev version directly here?)