Skip to content

Allow management of settings#101

Open
mikedowler wants to merge 2 commits intokcrawford:mainfrom
mikedowler:allow-prefs
Open

Allow management of settings#101
mikedowler wants to merge 2 commits intokcrawford:mainfrom
mikedowler:allow-prefs

Conversation

@mikedowler
Copy link
Copy Markdown

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
Copy Markdown
Author

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

@kcrawford
Copy link
Copy Markdown
Owner

kcrawford commented Dec 28, 2020 via email

@mikedowler
Copy link
Copy Markdown
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
Copy Markdown

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

@shawndwells
Copy link
Copy Markdown

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

@wonderphil
Copy link
Copy Markdown

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