Switch to FPP + Startup info box to inform users of changes in the future #130
Loading…
Reference in a new issue
No description provided.
Delete branch "any1here/FPP"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Requires https://codeberg.org/librewolf/settings/pulls/108
@any1here Is there still anything blocking this?
If not, interested to rebase it to we can merge it for the next release?
@alyx161 wrote in https://codeberg.org/librewolf/source/pulls/130#issuecomment-14300327:
Rebase wouldn't work, since I would need to move the settings to the new approach
Soo settings update is the blocker, go it :3
@alyx161 wrote in https://codeberg.org/librewolf/source/pulls/130#issuecomment-14317292:
I updated the settings pane to the new approach, hope the other stuff still works.
@alyx161 Change of plans. Instead of going with +AllTargets, I have opted to follow Nightly's RFPTargets. The current patch won't compile, since the extra nightly RFPTargets are coming in 154.
@any1here wrote in https://codeberg.org/librewolf/source/pulls/130#issuecomment-20616905:
Any sources for this and your reasoning for doing this?
This would be a drastic change to the current behavior
I thought the consent was to consider this for later?
CC @degausser
https://bugzilla.mozilla.org/show_bug.cgi?id=2050515#c11
@ltguillaume wrote in https://codeberg.org/librewolf/source/pulls/130#issuecomment-20626895:
@alyx161 wrote in https://codeberg.org/librewolf/source/pulls/130#issuecomment-20627279:
We could do the
+AllTargetsapproach, but this would just kick the can down the road and have people get their defaults changed twice.+AllTargetsalso has various quirks, especially with the Canvas* RFPTargets, which breaks some of the permissions for extraction.We are also hurting user experience for little to no gain, since this level of protection doesn't make sense outside of TB/MB. We are enabling everything, but gaining nothing. ETP Strict Firefox was good enough to beat fingerprint.com, which is the best we can also realistically aim for.
Will maybe edit this with more info tomorrow.
View command line instructions
Manual merge helper
Use this merge commit message when completing the merge manually.
Checkout
From your project repository, check out a new branch and test the changes.Merge
Merge the changes and update on Forgejo.Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.