Allow management of settings

kcrawford / dockutil

command line tool for managing dock items

http://patternbuffer.wordpress.com

Apache License 2.0

1.37k stars 131 forks source link

Allow management of settings #101

Open mikedowler opened 3 years ago

mikedowler commented 3 years ago

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 commented 3 years ago

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

kcrawford commented 3 years ago

I wasn’t sure if these add value beyond using the defaults command to modify dock.

What are your thoughts?

On Nov 27, 2020, at 7:31 PM, Mike Dowler notifications@github.com wrote:

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.

You can view, comment on, or merge this pull request online at:

https://github.com/kcrawford/dockutil/pull/101

Commit Summary

Add handling for settings Increment version number File Changes

M scripts/dockutil (63) Patch Links:

https://github.com/kcrawford/dockutil/pull/101.patch https://github.com/kcrawford/dockutil/pull/101.diff — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or unsubscribe.

mikedowler commented 3 years ago

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 commented 3 years ago

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

shawndwells commented 2 years ago

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

wonderphil commented 2 years ago

yes yes yes, this would be awesome