tinywebex/randomtips

Shows a random non obstrusive tip to the user under some constraints.

randomtips's Issues

Allow link to be changed per browser/interactively

We'd want a callback for the actions/URL, so we can adjust the URL dynamically.
The use case is to differentiate browsers.

But to make it more useful in general, I think it would be useful to have a callback given there, which returns the URL.
Also let's make sure it's "async", as that is often needed.

for rugk/unicodify#23 (comment)

`Error: show no random tip, because randomize did not pass` is not an error

This is not an error and should not be output as one, as it makes finding actual errors in the console more difficult:

It is output nearly every time the options page or browser action popup is opened for all extensions that use this library, including the Awesome Emoji Picker.

Support using the notifications API

Provide the option of using the notifications API to display notifications. This is needed for extensions that do not have a popup (like Unicodify) or that need to display notifications when both the popup and options page are not open. This could also be used to show errors to the user instead of silently failing.

Additionally, you could provide the option of using the future notification bar API in Thunderbird (see Bug 1674002).

Lazy loading broken

see rugk/unicodify#23 (comment)

Add "maximumActionButton" to not show tip to users that clicked on button

So you can limit the tip not to be shown when the user already clicked on a "show more" button or similar. Like maximumDismiss it can be null by default, but a common use case is likely to set it to 1.

How to do?

• Problem: We need to track the clicks on the button, but also want to keep "link-only" buttons. (especially as they show the link aim in a good way.)
• Solution: IMHO, the MessageHandler needs another function in the actionButton object, that additionally binds to the button, so it does not intercept/overwrite the click, but can actually count it. That may then be set by RandomTips manually…