Archived
Switch Red/Green in KDL section #334
Closed
shifterovich2 wants to merge 2 commits from
switch-red-green-kdl into master
pull from: switch-red-green-kdl
merge into: :master
:master
:dependabot/bundler/nokogiri-1.13.6
:dependabot/bundler/addressable-2.8.0
:freddy-m-patch-3
:pr-add_RemoveMyPhone_sponsor
:pr-browser_cleanup_1257_1328_1430
:freddy-m-patch-2
:freddy-m-patch-1
:pr-vpn_hated_one_video
:cdn
:update-nitrohorse-image
:promote-metager-to-card
:hardware
:pr-add_azirevpn
:pr-add_mailfence
:shop
:1673
:pr/1658
:i18n-simple
:sponsorship-edits-nov2019
:i18n
:ipfs
:blacklight447-ptio-patch-3
:blog
:remove-windows-icons
:pr/1147
:i18n-testing
:add-beautify
No Reviewers
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
Milestone
No items
No Milestone
No due date set.
Dependencies
No dependencies set.
Reference: privacyguides/privacytools.io#334
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.
Description
Don't merge until the design is resolved.
Resolves #333
Not sure whether I like it more now. Maybe use the same layout as #ukusa?
HTML Preview
http://htmlpreview.github.io/?https://github.com/shifterovich2/privacytools.io/blob/switch-red-green-kdl/index.html
@Shifterovich
Is @shifterovich2 an account that you own?
Special key disclosure laws exist for Belgian citizens. I've moved it to the right section.
Yes, it's the account I use on public/school computers.
@kewde What do you think about the layout? Use the same as #ukusa or keep it like this?
@hugoncosta Your opinion on the layout?
The preview says it cannot load, could you give me another link?
Right, I apparently deleted the branch. I'll update the link once I have time to upload the files to my website.
btw I was thinking, I don't believe we should change the layout. We use this on the number eyes section too, and we have the worst offenders first. I believe that the current layout should stay, since both KDL and #Eyes are warnings and not necessarily tools/advices. Our philosophy of putting the "best" option with a green box and to the left should remain, by putting the "worst" offenders (in this case) to the left and in a red box.
@kewde?
@Shifterovich the link is dead so I can't see it anymore. Can you snap a pic of it?
https://htmlpreview.github.io/?https://raw.githubusercontent.com/privacytoolsIO/privacytools.io/89ce79e4637bc8349f091bbc5e538feafbb1384e/index.html#kdl
I'm still not conviced. I believe that the panel on the left should be the most important - either the "stay away" or "you should really use this". If we change the KDL from "Good" to "Bad", we should also change the Global Mass Surveillance and put the Fourteen Eyes first (switch like in here).
Agree, I think I liked it better before aesthetically as well. Just wanted to know what you think. @kewde @beardog108 what do you think? Merge or close?
I don't really care about this change, it's just moving boxes 📦 and the order of (worst -> best or best -> worst) is clear here.