-
-
Notifications
You must be signed in to change notification settings - Fork 190
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
Download from Original Lepton always turn into Photon Style with white lines between tabs #402
Comments
Never mind, it seems that whatever was causing it is fixed now. |
@black7375 I have the same issue after upgrading to v5.2.1. Using Lepton Photon Style, all my prefs in |
@Roy-Orbison
A restart is required after setting the technical limit, and for file I think you used |
@black7375 Thank you for looking into this but the problem is not quite as you describe. I had restarted twice and ensured my Some of your options seem to be binary, e.g. Any preferences that conflict should still be set, but to their opposite rather than commented out, e.g.: // Original, Proton
// user_pref("userChrome.tab.dynamic_separtor", true);
// user_pref("userChrome.tab.static_separator", false);
// Photon
user_pref("userChrome.tab.dynamic_separtor", false);
user_pref("userChrome.tab.static_separator", true); or user_pref("userChrome.tab.dynamic_separtor", false); // Original, Proton: true, Photon: false
user_pref("userChrome.tab.static_separator", true); // Photon: true, Original, Proton: false |
Yes, exactly. |
I mean the files you distribute should not have commented lines, they should all be set even if to |
I fixed my problem not by un/commenting or modifying anything in |
@Roy-Orbison Yes, Your right. |
Describe the bug
As per title
Expected behavior
For it to look like Original Lepton again
Screenshots
Environment:
user.js
setupAdditional context
I tried using the PowerShell install, and I chose the Update option before I tried to install manually.
The text was updated successfully, but these errors were encountered: