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

Clean up Windows Start Menu entry #49

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

Conversation

RedBlackAka
Copy link

@RedBlackAka RedBlackAka commented Nov 5, 2024

Description

Simplifies the Start Menu entries created by the Windows installer to just one entry, in order to make it appear cleaner and modern.

Motivation and Context

Why have a Start Menu folder when it is just one entry (plus the uninstaller shortcut, but it is hidden since W10, so removal is appropriate and it is practically only one entry).

How Has This Been Tested?

Viewed on W10.

Types of changes

  • Tweak (non-breaking change to improve existing functionality)
  • Code cleanup (non-breaking change which makes code smaller or more readable)

Checklist:

  • My code has been run through clang-format.
  • I have read the contributing document.
  • My code is not on the master branch.
  • The code has been tested.
  • All commit messages are properly formatted and commits squashed where appropriate.
  • I have included updates to all appropriate documentation.

Simplifies the Start Menu entries created by the Windows installer to just one entry, in order to make it appear cleaner and modern.
@Fenrirthviti
Copy link
Member

The uninstaller shortcut is not hidden, it shows up on my end on Windows 11 at least. This may be an environment setting specific to you.

I don't know that I agree with these changes, as many applications still provide the uninstall shortcut, and I don't know that this is actually making things easier for users in any meaningful capacity.

@RedBlackAka
Copy link
Author

RedBlackAka commented Nov 5, 2024

The uninstaller shortcut is not hidden, it shows up on my end on Windows 11 at least. This may be an environment setting specific to you.

I don't know that I agree with these changes, as many applications still provide the uninstall shortcut, and I don't know that this is actually making things easier for users in any meaningful capacity.

My bad. But why have a own shortcut for the unistaller, when you already have right click > Uninstall on the main entry? I deem it an outdated (basically obsolete) and visually unclean practice. You would not find this on most new applications either.

@Fenrirthviti
Copy link
Member

Also for posterity, even Microsoft still puts single-shortcut applications in their own folder:
image

Additionally, I don't see an uninstall option when I right click on a menu item in the start menu for OBS Studio, which is probably handled by some environment setting somewhere that's poorly documented.

I understand that this might seem "cleaner" and may be recommended, but changing it at this stage doesn't seem worthwhile, IMO, especially with no cleanup of the old shortcuts as part of the consideration.

Raymond seems to agree that it's better as well, for what it's worth: https://devblogs.microsoft.com/oldnewthing/20190319-00/?p=102342

@RedBlackAka
Copy link
Author

Also for posterity, even Microsoft still puts single-shortcut applications in their own folder: image

Additionally, I don't see an uninstall option when I right click on a menu item in the start menu for OBS Studio, which is probably handled by some environment setting somewhere that's poorly documented.

I understand that this might seem "cleaner" and may be recommended, but changing it at this stage doesn't seem worthwhile, IMO, especially with no cleanup of the old shortcuts as part of the consideration.

Raymond seems to agree that it's better as well, for what it's worth: https://devblogs.microsoft.com/oldnewthing/20190319-00/?p=102342

Ahh I see. He does seem to both agree and disagree. The right click > Uninstall may actually require a NSIS entry that is currently not present. Could check things and find a way to make sure the old entries are also getting cleaned up when running both installer/uninstaller. I would really like this change and could handle the rest. And if you really do not want it, understandable, you can close this PR.

Both installer and uninstaller (just in case) remove old Start Menu shortcuts if they exist, so that there are no files left behind and the transition to the new shortcut is smooth when updating.
@RedBlackAka
Copy link
Author

Alright the installer removes the old shortcuts now, so the change does no longer leave behind files when updating. Smooth transition. Also re-added them to the uninstaller, just in case. Plus the Uninstall thing not appearing is a W11 thing (they want you to explicitly use the Settings there). This could be considered done, thanks!

@RedBlackAka
Copy link
Author

Status? I have given my arguments and put in the work, so a response would be appreciated

@Fenrirthviti
Copy link
Member

PRs are reviewed as the project has the time and resource to so. We have no response at the moment as we are focused on our current beta cycle.

@RedBlackAka RedBlackAka deleted the branch obsproject:master December 7, 2024 16:33
@RedBlackAka RedBlackAka closed this Dec 7, 2024
@RedBlackAka RedBlackAka deleted the master branch December 7, 2024 16:33
@RedBlackAka RedBlackAka restored the master branch December 7, 2024 16:35
@RedBlackAka RedBlackAka reopened this Dec 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants