Skip to content
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

Possible Small Enhancements #1747

Closed
8 tasks done
Jmkrnet opened this issue Oct 18, 2023 · 16 comments
Closed
8 tasks done

Possible Small Enhancements #1747

Jmkrnet opened this issue Oct 18, 2023 · 16 comments

Comments

@Jmkrnet
Copy link

Jmkrnet commented Oct 18, 2023

When upgrading to Firefox (FF) ESR 102 and now to ESR 115 I have accumulated some small enhancements (all should apply to current "user.js" version 118):

edited by 🩳


SECTION 0800 or 5000 or 9000 MAYBE ADD:
Adds calculator for basic math calculations to address bar. For example typing "1+1" to address bar gives result "= 2" and it works offline without connecting to any search engines.
// user_pref("browser.urlbar.suggest.calculator", true);

DONE

  • declined

0211 POSSIBLY REMOVE AS IT SEEMS TO BE HANDLED BY RFP (AND MAYBE 0210 IS ALSO REDUNDANT?):

DONE


1006 POSSIBLE COMMENT ENHANCEMENT:

  • I was not able to make this work on LINUX with FF ESR 102. Link shortcuts created by dragging URL from address bar to desktop are always (i.e. with the pref set to false or true) created with a generic icon and there is no "shortcutCache" or similar directory with "*.ico" files in FF profile or cache directories. Seems to me like it is a WINDOWS only feature (some summary info: https://quantumwarp.com/kb/articles/31-media/935-windows-firefox-favicon-internet-shortcuts-and-caching)?
  • I did not test this again on FF ESR 115, but it probably did not change.

DONE


2603.helperApps.deleteTempFileOnExit, 2815.cookies POSSIBLE COMMENT ENHANCEMENT:

  • Seems to be true by default at least on my OS (LINUX) with FF ESR 102 and 115.
  • POSSIBLE TEXT TO ADD IF CONFIRMED: [DEFAULT true]

DONE


5508.eme.enabled POSSIBLE COMMENT ENHANCEMENT:

  • Seems to be false by default at least on my OS (LINUX) with FF ESR 102 and 115.
  • POSSIBLE TEXT TO ADD IF CONFIRMED: [DEFAULT false]

DONE

  • is default true on windows. I only add default notations if I need to for active prefs, or if it's the same on all desktop platforms. The less shit to track and keep up to date the better

2660 POSSIBLE COMMENT ENHANCEMENT:

DONE


I am not sure how you define hidden prefs, but if it means non-obsolete prefs that are not visible in "about:config" unless FF or user defines some value for them (i.e. a "Delete" instead of "Reset" button is shown in "about:config"), here are prefs I noticed not marked as "[HIDDEN PREF]" that seem to be hidden at least on my OS (LINUX) with FF ESR 102 and 115:

  • user_pref("browser.startup.homepage_override.mstone", "ignore");
  • user_pref("extensions.webcompat.enable_shims", true);
  • user_pref("browser.contentblocking.category", "strict");
  • user_pref("geo.provider.use_gpsd", false);

DONE

  • these are all runtime, set by the browser, so whilst technically "hidden" (see next comment below), they're not really because they will always have a value in about:config. But will notate regardless to be consistent
@Thorin-Oakenpants
Copy link
Contributor

  • browser.urlbar.suggest.calculator - has nothing to do with this repo - privacy/tracking/etc
  • javascript.use_us_english_locale
    • is deprecated in FF119 - which is why you won't find it on searchfox
    • RFP no longer protects your locale (and it was limited to en-US with that pref)
    • at some stage they will hook up locale to match language in some form, which will be better

hidden pref

  • it does not show in about:config unless modified (regardless of the value, it is modified)

@Thorin-Oakenpants
Copy link
Contributor

helperApps.deleteTempFileOnExit - is default true on windows as well

@Thorin-Oakenpants
Copy link
Contributor

Thorin-Oakenpants commented Oct 19, 2023

0210 (en-US,en), since RFP no longer matches locale to it, can also be removed or moved to optional. So everyone feel free to use any language. I don't know if I want to keep it in the user.js so prefsCleaner resets it - I would need to check what it gets reset to

update: done - see #1759 and 5759127

@Thorin-Oakenpants
Copy link
Contributor

https://searchfox.org/mozilla-central/search?q=helperApps.deleteTempFileOnExit&path=&case=false&regexp=false

helperApps.deleteTempFileOnExit - true in firefox.js, false in all.js ... how does this work (rhetorical)?

ignoring android (FYI: @SkewedZeppelin) I guess we just test each platform

  • linux is true (but packagers could change it)
  • windows is true
  • mac (FF119) is false <-- assuming it even applies to mac

So I think we can ignore adding a default note

@Thorin-Oakenpants
Copy link
Contributor

browser.shell.shortcutFavicons

So yup, I would say this is a windows only thing

@Thorin-Oakenpants
Copy link
Contributor

Thorin-Oakenpants commented Nov 3, 2023

scopes

man that's a long one. I use searchfox links to files, but hg.mozilla.org revisions links to lines - but this revision moved the file and didn't change the content we want, so I can't use a sweet short link

lets see what the diffs are:

kaply

1 (SCOPE_PROFILE)
2 (SCOPE_USER)
4 (SCOPE_APPLICATION)
8 (SCOPE_SYSTEM)

moz code

  // General descriptions of where items are installed.
  // Installed in this profile.
  SCOPE_PROFILE: 1,
  // Installed for all of this user's profiles.
  SCOPE_USER: 2,
  // Installed and owned by the application.
  SCOPE_APPLICATION: 4,
  // Installed for all users of the computer.
  SCOPE_SYSTEM: 8,
  // Installed temporarily
  SCOPE_TEMPORARY: 16,
  // The combination of all scopes.
  SCOPE_ALL: 31,

yeah, so that's a good link to add, because that archive is fucking old now and somewhat out of date

@Thorin-Oakenpants
Copy link
Contributor

actually, fuck that long link ;) it hasn't changed anything, because temporary extensions cannot load on restart.. and I can just add this

 * 1=profile, 2=user, 4=application, 8=system, 16=temporary, 31=all

Thorin-Oakenpants added a commit that referenced this issue Nov 3, 2023
@Thorin-Oakenpants
Copy link
Contributor

Thorin-Oakenpants commented Nov 3, 2023

extensions.webcompat.enable_shim - is runtime - set based on ETP settings = so not "hidden" from sight ever - but technically it is [hidden] by my own definition above

edit: not sure what to do here - I defined hidden above as the way I understood and have used it - so users know why they can't see it, i.e they have to create it and add it

I suspect the other three are runtime as well

What should I do people? give them hidden tags or not?

@Thorin-Oakenpants
Copy link
Contributor

^ repeat - someone answer ... starting to feel like my childhood in the basement again

@SkewedZeppelin
Copy link
Collaborator

maybe managed?

@Thorin-Oakenpants
Copy link
Contributor

not sure if I want another tag

so here's the thing - I label items to help, but that doesn't mean I have to label ALL items that fit that tag: eg

  • not all items have a [FFversion+] - although that's mostly really old prefs
  • not all items have a [DEFAULT: - and maintaining it is a burden, especially when it differs per OS
  • etc

So i kinda feel like these items that aren't "hidden" in about:config don't really need a [HIDDEN] tag - but I get that as a purist, it is fine. And if someone says "but it's not hidden" I can use the same definition I always have, and then tell them to delete it and reload about:config and tell if they can see it

So the question is - do I add hidden tags or not?

@Thorin-Oakenpants
Copy link
Contributor

And if I don;t label them, then if someone says .. but that's a HIDDEN pref, please label it - I can say "I only label them when they aren't set by runtime to help users"

So I win either way

So do I add the labels or not ... quick poll of one .. you're up Tad

@Thorin-Oakenpants
Copy link
Contributor

ok, ignoring them, because of maintenance, plus I am the boss so my second argument wins. just need to check that geo one - because that may only get set if the system supports it

@crssi

This comment was marked as off-topic.

@Thorin-Oakenpants
Copy link
Contributor

might revisit my last decision .. I mean if it universally (all desktop platforms) has a trash bin symbol, then it's [HIDDEN]

@Thorin-Oakenpants
Copy link
Contributor

okie dokie

@Jmkrnet thanks .. have some 🍰 or 🥧

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

4 participants