Implement preserveAppOrder user preference
#30
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Reorder app menu
|
lgtm, merging |
|
Thanks @EthanSK! Those 3 issues I linked can optionally be closed :) I thought the release workflow was automated, but I'm guessing it's manual 😅 |
|
yep, just pushing a release now (edit: apple taking a while to notarize the app, guess its app fixing for a new macos version season) |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hey Ethan, this is an excellent app, and is exactly what I was looking for!
I noticed that there were a few issues open, and wanted to try and help where I could. I haven't personally written Swift before, so my apologies if there are glaringly obvious issues with my contribution.
The three things this PR changes:
Preserve app orderuser preference: this provides a configurable workaround for Invisible Divider/Icon #27I intend to also tackle #23, but I wanted to push this PR before considering it.
Thanks so much!
Screenshot of the updated user preferences (note the new
Preserve app ordercheckbox under the Running Apps header)