inbasic / ignotifier

Multiple account Gmail notifier (without storing passwords)
https://webextension.org/listing/gmail-notifier.html
Other
389 stars 116 forks source link

no age shaming #631

Closed ImprovedTube closed 8 months ago

ImprovedTube commented 9 months ago

users of old browsers might not use browser extensions or know what they are doing. if you fill in the blank, i can make another PR or commit like: chrome.runtime.onInstalled.addListener(function(installed){if(installed.reason=='install'){if 116 > Number(navigator.userAgent.match(new RegExp(Chrome + '/([0-9]+)'))){alert("[Just to make sure:] Some of our features might require chrome version 116+.... _______________________") } ( - or rather add it precisely, when that feature is first enabled as indicated by its button or storage change.)

inbasic commented 8 months ago

The extension is broken if the "offscreen" API is not supported. So what is the point of letting the users install the extension? If the extension is incompatible, at least they can have the old version.

ImprovedTube commented 8 months ago

offscreen API

interesting! @inbasic

hope you don't mind i wasn't intending a 100% PR-success rate for the first time.. :D