Closed BobWs closed 10 months ago
Any stuff can be cut out or modified by patching https://github.com/ProtonMail/WebClients/ sources before it gets assembled for packaging into the app, see https://github.com/vladimiry/ElectronMail/tree/master/patches/protonmail.
If @protonmail saves "explicitly closed" state of this ad in the browser's local storage (I didn't look into the code, just a guess), then this state will be lost, since the app by design keep all the browser/web-specific storages in memory only, so the data not persisted between the app restarts.
keep all the browser/web-specific storages in memory only, so the data not persisted between the app restarts
I guess that's what's happening, because after every new start of the app they all appear again.
Do you want to take look at it and see if you can patch it?
Do you want to take look at it and see if you can patch it?
If I do this, then as an opt-in feature, so @protonmail upsell messages not suppressed by default. For opt-in feature, simple static patching won't be a solution (dynamically switchable feature), so more dev work would be required. There is no ETA, so if I do this, you will see it done.
It's coming in the next release. See the new "Disable Proton upsell messages" toggle in the "General" app settings. The toggle is disabled by default, so this is an opt-in feature.
It's coming in the next release. See the new "Disable Proton upsell messages" toggle in the "General" app settings. The toggle is disabled by default, so this is an opt-in feature.
That's a great solution, nice! Thank you for implementing it!
will it also hide the 'upgrade now' banner?
will it also hide the 'upgrade now' banner?
If it's similar to the image in the first message https://github.com/vladimiry/ElectronMail/issues/629#issue-1845281306, then yes, it will.
there also an upgrade button on header next to settings. will it be hidden as well ?
will it be hidden as well ?
Nope. Looking into it. The above-displayed "Disable Proton upsell messages" toggle is going to be renamed to "Disable Proton upsell messages & buttons".
there also an upgrade button on header next to settings. will it be hidden as well ?
Done now.
toggle is going to be renamed to "Disable Proton upsell messages & buttons".
The new toggle title name is "Disable Proton upsell ads". So the toggle disables upsell messages and buttons too.
just updated to v5.2.0 and it looks great now. thank you !
the banner is still visible in spam and trash folders...
the banner is still visible in spam and trash folders...
Not for me. Screenshot?
the banner is still visible in spam and trash folders...
same for me
It's different on paid plan, so it didn't look like an ad for me. But on free plan it indeed looks like an ad, so has to be taken care too. Thanks for this finding.
free account:
paid account:
yea, i'm a freeloader 😳
Just installed the latest version, nice work thank you for tackling it so quickly!
free account:
paid account:
v5.2.1 comes with disableable spam/trash ad banners. No change made for paid users since it's not an ad there, but a shortcut to an available feature and the banner there comes with "close" icon which closes it permanently.
Hi, Not a bug or issue report, but a question! Do you know a way to block the annoying ads of proton about upgrade to paid account and all those other promotions? See screenshot for example