User.js: ToDo: format change

[Just-me-ghacks]

• [x] replace "Covenience" with "Convenience"
• [x] replace "surpises" with "surprises"
• [x] replace "shoulder surfers,forensics" with "shoulder surfers, forensics"
• [x] remove double space "disable URLbar autofill - PRIVACY"
• [x] "Try it You are not missing" - you?
• [x] and impacts video imperformance ???
• [x] replace "signifcant" with "significant"
• [x] remove the space at the end of line 976.
• [x] Facebook but youtube, microsoft - why capital F?
  
  
  
  • Roman says: cheers, I need to use Capital for when talking about them as a company but lowercase as a website
  
  
• [x] capital letter? "then also make sure 0806"
  
  
  
  • Roman says: not sure what you mean, but URLbar was probably written like to emphasize it was the URl bar and not some other bar, its now just urlbar all lower case, if that was what you were referring to
  
  

[earthlng]

http://pasted.co/5c0cfcc1

Only 1 change from my comparison: 2703 was commented out in master, active in your pastebin

Apart from a few other things, I changed the numbers under 0410, not a fan of things like 0410b2, but feel free to change it back or do a mix , or whatever you think is best

some of the things Just Me noticed are corrected in my pastebin, but not everything. (only the first 2 I think)

[Just-me-ghacks]

capital letter? "then also make sure 0806" - My bad, ignore it. I didn't see the comma before "then".

• [x] disable dns prefetching - disable DNS prefetching
• [x] dns leak issue - DNS leak issue
• [x] disable referer from - disable referrer from
  
  
  
  • no change. referrer vs referer .. there are 3 of the former and they are site links or a pref name, everything else is the later and there are 10 of them. The correct word is a single r
  
  
• [x] 2001: disable WebRTC - expand to "WebRTC (Web Real-Time Communication)"
• [x] 2010: disable WebGL - expand to "WebGL (Web Graphics Library)"
• [x] "and fear of breakage/bugs */" - remove double space

Roman says: use

* [ ] comment

and then I can just tick rather than edit

[earthlng]

IMO we should always use 0410a, b, c, etc. for "meta"-numbers. It allows for more than just 041[1-9] prefs in one group.

EDIT: fe. 0420 could be SB + TP "description only" number, 0421 SB, 0422 TP and 0415 would become 0423 because it applies to both. We would have SB + TP all under 042x that way.
Sorry, a bit late with the suggestions but I mainly looked at other things during the comparing.

[Just-me-ghacks]

• [x] replace with ' <ul> <li>initially started putting pref names inside or ' but stopped, too much going on. All of the former are now removed

[x] 2671: disable SVG - expand to "SVG (Scalable Vector Graphics)"

[x] 2663: disable MathML - expand to "MathML (Mathematical Markup Language)"

[earthlng]

maybe slightly OT, but hear me out ;)

I still like the idea of making this overall more modular by using a preferences folder.
But we would only use a separate file IF we can really isolate certain prefs for a certain feature, SB + TP come to mind.
We could release with safebrowsing-trackerprotection.js.prefs and without needing to edit anything, users could just rename the file to disable SB + TP.
Since you plan to enable SB+TP by default anyway, this would be a neat way to handle it.
Of course this would only work when we want to ship with a feature completely on it's default values.

https://github.com/ghacksuserjs/ghacks-user.js/issues/32#issuecomment-282502075 Restore video settings

If we had multiple files, we could consider using 'pref' instead of 'user_pref' for certain files (fe media)
I know it has a lot of downsides, but one major advantage is that the file could simply be renamed or deleted and the original defaults would be restored.

[earthlng]

Give it one last pass if you can :)

• 0410a - F48 instead of FF48 (?)
• 0410b - exists twice
• 2672 - move the "than sorry" to the previous line. it's still shorter than the title line
• 3021b - why do you insist on removing the line [NOTE] does not apply to middle-click or Ctrl-clicking links. ? imo that's nice to know
• 3021c - no space between /* and number

IMO it's always good to have an empty newline at EOF

[Just-me-ghacks]

• [x] 0410d: disable mozilla safebrowsing // mozilla or Mozilla?
  (0410c: disable Google safebrowsing & 0440: disable Mozilla's blocklist)
• [x] 0810: disable css querying page history - css history leak // CSS?
• [x] 1405: disable woff2 - expand to WOFF (Web Open Font Format)
  
  
  
  • Roman says, its WOFF version 2
  
  
• [x] This will break some sites eg Dropbox, Google Docs? gmail? // capital letters inconsistency?
• [x] Roboform // RoboForm
  
  
  
  • Roman says: can't even find roboform or the other one mentioned at ghacks (Internet Download Manager) on AMO.
  
  
• [x] lastest ESR // latest
• [x] site funtionality // functionality

@Thorin-Oakenpants - I like your writing style and the original user.js. Please don't approve everything and don't change things just for the sake of changing something. Just saying...

[earthlng]

• [x] 1200 - start the sentence after the URL on a new line
• [x] 1208 - 0=disabled instead of 0-disabled
• [x] 1215 - should be 0 = blabla, 1 = blabla -- same as 1211. plus add (FF50+) note
• [x] browser.usedOnWindows10.introURL // removed in FF50 (was in my diff) => 0 results in DXR for 'usedOnWindows10'
• [x] 2617 - says disable in title but enables it
• [x] add comment // Form & Search History to privacy.clearOnShutdown.formdata
• [x] uppercase first letter // Active Logins for privacy.clearOnShutdown.sessions
• [x] better title for 1200 fe. HTTPS ( SSL / OCSP / CERTS / ENCRYPTION / HSTS / HPKP )
• [x] 1222 doesn't fit in 1200, has nothing to do with HTTPS (2600?)
• [x] 2613 is HW fingerprinting and would fit better under 2500
• [x] 2671: change title to _disable in-content SVG (Scalable Vector Graphics) (FF53+)_
• [x] 1401: title is incorrect, it's not about "downloading fonts"

==Roman comments added==

• [x] maybe move 0430 to 1200
  
  
  
  • wontfix: : I think it fits nicely with keeping ff quiet, and also 400 is about blocklists, revoked certs .. and reporting certs is part of that. Whereas 1200 is more about what certs etc to use. I will leave where it is.
  
  
• [x] 3021b - I think it applies when you close a tab with middle-clicking the tab
  
  
  
  • wontfix: Roman says: we have lots of other click related preferences .. I don't think we should have to state on each one that key+mouse combos may do different behavior. When I was editing, that one on 3021b to me just looked out of place. So out it goes.
  
  
• [x] rename 2660 to 2653 -> e10s
  
  
  
  • wontfix for now: this can be dealt with when all the e10s stuff is worked out
  
  
• [x] IMO we should make 2700 only for Cookies, ie. move 2707 to cache (?), and 2705+2706 to 2400
  
  
  
  • wontfix on this ticket. I agree 2707 could move, the whole section is a tad messy. cookies on their own don't warrant a section. I tried to add dom storage since they seem to be handled together by a lot of addons. Create a ticket for discussion.
  
  

--WTF

• [ ] 0810: we should add the testpage: http://lcamtuf.coredump.cx/yahh/ (also add to wiki)

I'm sorry, but you said make any changes you want ;)

[earthlng]

testsite: you need to enable layout.css.visited_links_enabled and test it in a Normal Window. PB doesn't seem to work. Check the source and visit a few of the URLs from the list and then start the test. It uses a lot of http sites instead of https, so it can't detect some of the sites anymore because they moved to https.
But I did the test and apart from one, it correctly identified all other sites I visited (fe. didn't detect http://reddit.com/r/conspiracy, because that's https nowadays)