Archived
Make JavaScript work with LibreJS #1213
Merged
dawidpotocki
merged 1 commits from 2019-08-25 04:01:50 +00:00
librejs into master
Dismiss Review
Are you sure you want to dismiss this review?
Labels
Clear labels
:mag:🤖 Search Engines
I2P
OS
Self-contained networks
Social media
Tor
WIP
XMPP
[m]
approved
dependencies
duplicate
feedback wanted
high priority
iOS
low priority
stale
streaming
todo
wontfix
₿ cryptocurrency
ℹ️ help wanted
↔️ file sharing
⚙️ web extensions
✨ enhancement
❌ software removal
🆕 software suggestion
🆘 critical
🇦🇶 translations
🌐 Social News Aggregators
🌐 hosting
🌐 website issue
🏠 housekeeping
🐛 bug
👁️ browsers
👥 team chat
💢 conflicting
💬 discussion
💻 hardware
📁 file storage
📝 correction
📧 email
🔎 research required
🔐 password managers
🔒 VPN
🔒 file encryption
🖊️ digital notebooks
🗄️ DNS
🗨️ instant messaging (im)
🚫 Windows
🤖 Android
🦊 Firefox
🧰 productivity tools
The Invisible Internet Project (I2P)
Operating Systems
Anything covering the Tor network
active work in progress, do not merge or PR (yet)!
Extensible Messaging and Presence Protocol
Matrix protocol
approved, waiting for a PR
Pull requests that update a dependency file
A label for stalebot if it gets added
Anything related to media streaming.
Issues or bugs that will not be fixed and/or do not have significant impact on the project.
Browser Extension related issues
Anything covering a translated version of the site
*Technical* issues with the website.
Anything primarily related to site cleanup.
Correction of content on the website
Virtual Private Network
Domain Name System
Firefox & forks, about:config etc.
No labels
🏠 housekeeping
Milestone
No items
No Milestone
No due date set.
Dependencies
No dependencies set.
Reference: privacyguides/privacytools.io#1213
Reference in New Issue
Block a user
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Deploy preview for privacytools-io ready!
Built with commit
2a478ded0fhttps://deploy-preview-1213--privacytools-io.netlify.com
@@ -34,0 +44,4 @@g.defer = true;g.src = u + "matomo.js";s.parentNode.insertBefore(g, s);})();Doesn't matter too much but could we make this a bit more readable?
@@ -77,0 +77,4 @@</a><p class="mt-2"><strong>No Ads, No Google Analytics, No Affiliates, No Cross-Site Requests.</strong><em>privacytools.io is a socially motivated website that provides information for protecting your data security and privacy. Never trust any company with your privacy, always encrypt.</em>Could we add a slight space here?
<br />I'm thinking for the creativecommons links we could use
https://Super cool!
LGTM!
LGTM
LGTM, I guess, but I would like to have @JonahAragon to check it
Uhh... YOLO, right? I don't see why this shouldn't work.
Have you removed all the inline JS in this PR? We could probably update the CSP too.
Well… I only found this in
scripts.htmland LibreJS does not inform meabout other stuff. So probably, yes.