Skip to content

Conversation

@kumichou
Copy link
Collaborator

  • Incorporates Zen jiggle (no click) in MacOS Tahoe fix from MarinSavca
  • Corrects the Jiggler Overlay being off to the right side of screen in Sequoia and Tahoe
  • Use a prefix to handle NSLog() statements. No more Comment/Uncomment needed
  • Correct the deprecated detection for Mounted Volumes

@bhaller
Copy link
Owner

bhaller commented Oct 24, 2025

Hi @kumichou. Thanks for this, looks great. Given the discussion at #37, shall I wait for those changes before merging this? I've just sent you an invite to be a collaborator, so I guess you'll be able to merge your PRs yourself, too, but perhaps it's useful to follow a review process and have me merge, for the time being?

Don't forget to add your name in a couple of spots, like the About panel. That will be a bit fussy, since it isn't really designed to show more than one author; do what seems best to you. For now let's keep it badged as a Stick Software product, if that's OK. If you fully take it over later on, we can talk about whether that continues to make sense.

@kumichou
Copy link
Collaborator Author

@bhaller I want to merge the PR from @MarinSavca first, then I will merge my changes since I incorporated his and want him to get credit for that fix.

@kumichou
Copy link
Collaborator Author

I approved PR #35 if you would like to then merge that. I'll verify my changes don't have any conflicts before that will be merged.

@bhaller bhaller merged commit 103a4e3 into bhaller:master Oct 24, 2025
@bhaller
Copy link
Owner

bhaller commented Oct 24, 2025

Well, I screwed that up. Sorry. I don't use Git very much; I'm used to working solo. There's a "Revert" button I can click above, which looks like it creates a new PR to revert what I just did, but I'm not sure that's a good idea. Let me know how to proceed. Sorry again. :-<

@kumichou
Copy link
Collaborator Author

kumichou commented Oct 24, 2025

@bhaller I asked MarinSavca to update his PR first then we will figure out what needs merged from there.

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.

2 participants