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

fix: Improvements on Windows overview #2606

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

dngray
Copy link
Member

@dngray dngray commented Jun 2, 2024

Changes proposed in this PR:

  • I agree to the terms listed below:
    Contribution terms (click to expand) 1) I am the sole author of this work. 2) I agree to grant Privacy Guides a perpetual, worldwide, non-exclusive, transferable, royalty-free, irrevocable license with the right to sublicense such rights through multiple tiers of sublicensees, to reproduce, modify, display, perform, relicense, and distribute my contribution as part of this project. 3) I have disclosed any relevant conflicts of interest in my post. 4) I agree to the Community Code of Conduct.

Copy link

github-actions bot commented Jun 2, 2024

Your preview is ready!

Name Link
🔨 Latest commit 31fd268
😎 Preview https://2606--glowing-salamander-8d7127.netlify.app/

dngray added a commit that referenced this pull request Jun 2, 2024
dngray added a commit that referenced this pull request Jun 2, 2024
@dngray dngray marked this pull request as draft June 2, 2024 09:44
dngray added a commit that referenced this pull request Jun 2, 2024
dngray added a commit that referenced this pull request Jun 2, 2024
dngray added a commit that referenced this pull request Jun 2, 2024
docs/os/windows/index.md Outdated Show resolved Hide resolved
Copy link
Member Author

@dngray dngray left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added a few comments for discussion

dngray added a commit that referenced this pull request Jun 2, 2024
dngray added a commit that referenced this pull request Jun 2, 2024
Copy link
Contributor

@rollsicecream rollsicecream left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some fixes here and there.

docs/os/windows/index.md Outdated Show resolved Hide resolved
docs/os/windows/index.md Outdated Show resolved Hide resolved
docs/os/windows/index.md Outdated Show resolved Hide resolved
docs/os/windows/index.md Outdated Show resolved Hide resolved
dngray added a commit that referenced this pull request Jun 2, 2024
Copy link
Member

@jonaharagon jonaharagon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Going to give this some thought. I don't think these changes capture the original intent of the Windows guides being added though, and omit some key details. This almost reads as if Windows is recommended, which I don't think is our goal.

@dngray
Copy link
Member Author

dngray commented Jun 2, 2024

This almost reads as if Windows is recommended, which I don't think is our goal.

I don't think it does that at all. I reworded some sentences to a more neutral tone which brings it more in line with the other content already on the site. The main thing people will care about is the guides that we actually provide (which are currently marked as "coming soon").

I also haven't really removed anything except for some of the repetitiveness, and mentions of "editions". I did remove the recommendation to stick with Windows 10, as this isn't helpful to anyone who buys a device shipped with Windows 11 OEM.

Also in some cases Windows 11 has things which Windows 10 does not, for example DNS over HTTPS as part of the system resolver.

dngray added a commit that referenced this pull request Jun 2, 2024
@dngray dngray force-pushed the pr-improvements-overview branch 2 times, most recently from 3bbf8a6 to 789de88 Compare June 2, 2024 15:15
dngray added a commit that referenced this pull request Jun 2, 2024
@dngray dngray marked this pull request as ready for review June 2, 2024 15:21
dngray added a commit that referenced this pull request Jun 2, 2024
@dngray dngray changed the title fix: Opening on Windows page fix: Improvements on Windows overview Jun 2, 2024
dngray added a commit that referenced this pull request Jun 2, 2024
@privacyguides-bot
Copy link
Collaborator

This pull request has been mentioned on Privacy Guides. There might be relevant details there:

https://discuss.privacyguides.net/t/windows-guide/250/249

@dngray dngray added the t:correction content corrections or errors label Jun 3, 2024
docs/os/windows/index.md Outdated Show resolved Hide resolved
docs/os/windows/index.md Outdated Show resolved Hide resolved
docs/os/windows/index.md Outdated Show resolved Hide resolved
docs/os/windows/index.md Outdated Show resolved Hide resolved
docs/os/windows/index.md Outdated Show resolved Hide resolved
docs/os/windows/index.md Outdated Show resolved Hide resolved
docs/os/windows/index.md Outdated Show resolved Hide resolved
Comment on lines 30 to 38
With Windows 11 there are a number of restrictions or defaults such as:

Microsoft often abuses the automatic updates feature to add new functionality to your device that collects your data and is enabled by default.
- Requiring the use of a Microsoft account instead of a local account.
- Making it more difficult to find local account options for Windows **Pro** and **Enterprise**.
- Enabling all data collection options by default, requiring users to "opt out".
- Heavily integrating Microsoft services like Bing, OneDrive, and Teams in ways which are difficult to remove and presented as the only option to users.
- Making the default browser always default to Edge, or switch back to Edge.
- Adding cloud-based AI features to many areas in Windows and various Microsoft Apps.
- Unnecessarily storing sensitive data. Even data which is stored locally and not sent to Microsoft is still a target for hackers or malware on your device.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think that the "dark patterns" term introduced in this PR can be integrated here, but I'll leave that choice to the author.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It was intended to be used there but, I reworded something and we didn't end up using it.

dngray added a commit that referenced this pull request Jun 4, 2024
@dngray dngray mentioned this pull request Jun 13, 2024
dngray added a commit to IkelAtomig/privacyguides.org that referenced this pull request Jun 13, 2024
dngray added a commit to IkelAtomig/privacyguides.org that referenced this pull request Jun 13, 2024
dngray added a commit to IkelAtomig/privacyguides.org that referenced this pull request Jun 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
t:correction content corrections or errors
Projects
Status: Needs Changes
Development

Successfully merging this pull request may close these issues.

None yet

6 participants