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

Don’t create a Tribler subfolder in the Start #6239

Merged
merged 1 commit into from
Aug 24, 2021
Merged

Conversation

da2x
Copy link
Contributor

@da2x da2x commented Aug 10, 2021

Install the Tribler shortcut directly into the root of the Start Menu. A subfolder adds an unnecessary extra click to open Tribler. This has been the best practice since Windows 8 when Windows stopped listing folders above icons in the Start Menu. This also gives Tribler a better appearance and brand recognition in the start menu as the listing will feature Tribler’s icon instead of a generic folder icon.

Don’t create a shortcut to uninstall Tribler in the Start Menu. The entry shows up when searching for Tribler in the Start Menu, and it’s easy to missclick on it from Search (as results rearrange as they pop in) and in the Start Menu. Rely on the regular uninstallation flow instead. (Right-clicking on the icon the Start Menu and choosing Uninstall, or going to the Windows Settings app: Apps: Remove apps.)

Install the Tribler shortcut directly into Programs.

Don’t create a shortcut to uninstall Tribler.
@tribler-ci
Copy link
Contributor

Can one of the admins verify this patch?

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.5% 0.5% Duplication

@qstokkink
Copy link
Contributor

ok to test

@kozlovsky kozlovsky merged commit cbd15ef into Tribler:main Aug 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants