-
Notifications
You must be signed in to change notification settings - Fork 232
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
Consider removing old settings #33
Comments
Would disagree with that: there are also ESR browsers out there. Historically significant options may be simply put in a single section of the file. |
This issue is just a consideration, not an actual issue which needs to be fixed right away. I think it would be prudent to keep track of deprecated settings along the way, and remove them as new releases of Firefox ESR comes out. That being said, The the settings are also not present in the current version of Firefox ESR. |
The following settings are also not present in the current Firefox and Firefox ESR releases.
|
We can certainly clean up the
I'm not too keen on having separate versions of this, especially when this would only be a cosmetic change. |
As a user of this project I would like this. Or just comment out the relevant lines, adding |
I think the preferred approach is to append affected Firefox version numbers to the pref description, like we did for the Battery API prefs (https://github.com/pyllyukko/user.js/blob/master/user.js#L72). This needs to be made clearer in the README, and then this issue can be closed. |
To simplify the config, consider removing old settings that no longer apply. As of Firefox 39.0.3, the following settings no longer apply:
The text was updated successfully, but these errors were encountered: