-
Notifications
You must be signed in to change notification settings - Fork 132
Configuration for hiding random tips #308
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
Comments
Thanks a lot for that detailed (researched) feature request! I see you looked into the source… 😉
Good idea, but issue is stuck at finding a good and minimal UI. See/follow #225 for details.
For the given space explaining the latter would be too much, but maybe an entry in the FAQ could help? That could also be linked? (ref #67)
Not done as IMHO it would partially be too intrusive. Only as very little links at the bottom, but IMHO that would also not fit into the style.
I decided against that as the UI should be minimal and wasting space for that not very likely event of such a tip being shown is literally just a waste of screen space. The resizing was less bad IMHO. GenerallyIn general, please be aware the likelihood to see such a prompt like the "rate me prompt" is very low and e.g. only shown after you have used the extension for sometime and not shown again if you have dismissed it once. That is why I did not found it necessary to have a feature in there to disable that. So either the rules to show that need to be fine-tuned or I would also not be generally be opposed against such a settings, just remember every option comes at a cost. But probably that old more a test-thing monospace options could be removed – I mean I naturally have no data for it, but I doubt anybody actually uses it… 🙃 |
…er he wants to see random tips
First, thanks for making the great extension.
Background
The tips shows randomly, in 20% chance maybe? And it also randomly resizes the popup window, sometimes shows unexpectedly large window like if this message appears:
offline-qr-code/src/_locales/en/messages.json
Line 138 in 04a6089
Proposed solution
Please give a option for hiding the random tips.
Alternatives
Preserving fixed space for the tips can be an option, instead of randomly hiding tips.
etc
The text was updated successfully, but these errors were encountered: