-
Notifications
You must be signed in to change notification settings - Fork 129
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
Introduce smart QR code size handling and remove user options? #187
Comments
IMHO…
|
So @bfred-it, what i also don't like about your idea is the big removal of all user options. I mean, as the use cases above show, such a "clever" add-on could sometimes not really be efficient for you. So I'd suggest to possibly still have these options:
Note these are checkboxes, so you can check both, none or one at one time. (Both shall be enabled by default.) Both only apply on resizable QR code pages. If fixed sizes are there, we try to automatically determinate a good size. What I like about these settings is that they are self-explanatory and do not need such a big explanation as the current ones. IMHO these would satisfy the use cases presented here. BTW, I hope you see, this is way harder than one might think… 😉 |
Oh your avatar always fits… 😜 But no, it's not that simple. Again you dismiss two use cases and completely ignore things like #90. |
Problems
Solution
@bfred-it suggested in #35 (comment):
So this would result in the removal of three size options and trying to "intelligently" do what the user might want.
This would also:
Analysis of places
Actually, I see it's kinda hard, because the add-on can appear in many places with different restrictions:
The last column is a hard one, because it's already subjective, but actually, we might even have to remember different sizes, based on whether it is a tablet screen or not…
Questions
The text was updated successfully, but these errors were encountered: