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

Allow management of settings #101

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

Conversation

mikedowler
Copy link

Added code to allow settings (e.g. icon size, whether to show recent apps, etc) to be configured. Seems reasonable to me that you may want to set one or more of these settings at the same time as configuring the dock apps.

@mikedowler
Copy link
Author

@kcrawford Have you had a chance to look at this at all? Any issues with accepting the merge?

@kcrawford
Copy link
Owner

kcrawford commented Dec 28, 2020 via email

@mikedowler
Copy link
Author

So, the particular use case that prompted this was wanting to remove the "Recent Applications" setting introduced in Catalina. As I see it, the changes:

  • are directed at the same goal as the core functionality, namely allowing modification of the appearance/functionality of the macOS dock
  • use the same mechanics to achieve this, namely modification of the same plist file in each user profile
  • don't interfere with the original functionality

I appreciate that not everyone will use the additional feature, but I would, and so I assume that there are at least some others who might. And because of the points above, I don't feel that we are adding any unnecessary bloat to the project by including them - this is not an attempt to modify settings in general, or to expand scope outside Dock configuration. Creating an additional tool to implement the additional feature would require duplicating much of the content of dockutil, so it is much more efficient to include it in the same tool.

@mpermann
Copy link

I agree this would be a good addition to the script. It's certainly something that I would use.

@shawndwells
Copy link

This would be extremely valuable to me as well. Specifically the 'recent apps' part.

@wonderphil
Copy link

yes yes yes, this would be awesome

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.

5 participants