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

Lots of plugins #137

Draft
wants to merge 100 commits into
base: master
Choose a base branch
from
Draft

Conversation

bensleveritt
Copy link
Collaborator

No description provided.

The Grand Design added 30 commits January 5, 2017 15:05
And so that it always wipes out all items.
The previous `show_delay` was incorrect.  It was changing the speed of the
transition, not the delay before the transition started.  I left
`show_delay` in there and mapped it to the (incorrect) `auto_hide_speed`
for backwards compatibility but removed reference to it from the README.
There's quite a bit of duplication among many of these functions.  I'm
going to add a few helper functions to clean some of it up.  It will make
adding new plugins easier as well as make the user experience more
consistent.  Not to mention cutting down on possible bugs.
This was much too prone to manual error.  For now I just went through the
commands that were predominantly 'defaults write' commands as that what
I'm currently the most focused on.

I also removed the duplication of the restarting of processes and made it
happen after any command (which has been given a value) has been run.
The Grand Design added 30 commits January 8, 2017 13:09
Apparently from having many contributors the docs got a bit out of date
(format-wise).  Some (most) of the "examples" were actually usage.

Examples should contain sample data.  Usage should contain placeholders.

Take the `service` plugin for example:

This is usage:

    m service load <service_name>

This is an example:

    m service load com.apple.sessionlogoutd

I didn't rewrite everything but I made some good progress.
Some places can't be replaced, but these could
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.

1 participant