-
-
Notifications
You must be signed in to change notification settings - Fork 137
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
Add an option to prevent browsertime from setting Firefox default preferences. #1650
Conversation
Hi @soulgalore! We have an issue where our prefs are being overriden by the settings in browsertime - this patch would let us prevent that. |
Hi @gmierz aha cool, what about changing the name to |
lib/support/cli.js
Outdated
@@ -483,6 +483,12 @@ module.exports.parseCommandLine = function parseCommandLine() { | |||
type: 'boolean', | |||
group: 'firefox' | |||
}) | |||
.option('firefox.noPrefs', { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thinking noDefaultPrefs maybe is a little better?
If you can fix that, I'll can add the same config to Chrome too :) |
Sounds good to me thanks! :) |
Thank you @gmierz :) |
This patch adds an option (--firefox.noPrefs) that lets us prevent browsertime from setting default preferences. This solves an issue where settings in an existing profile get overriden by the preferences set in browsertime.