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

Sonoma Bug: Enabled Yubiswitch menubar icon is 10x size #129

Open
octoxan opened this issue Sep 27, 2023 · 7 comments
Open

Sonoma Bug: Enabled Yubiswitch menubar icon is 10x size #129

octoxan opened this issue Sep 27, 2023 · 7 comments

Comments

@octoxan
Copy link

octoxan commented Sep 27, 2023

When you hit "Enable Yubikey" in the menubar item for Yubiswitch, the icon becomes about 5-10x the size and is really jarring.

image

@ludokx
Copy link
Contributor

ludokx commented Oct 11, 2023

Thanks for the quick merge @davidrothera! Do you have an idea when the next release would be by chance? This bug is not a huge deal but it is a bit jarring indeed, especially when using the yubikey multiple times a day ;p

@davidrothera
Copy link
Collaborator

Releases are a bit of a pain but I'll go through and try and get one signed and uploaded tonight.

@ludokx
Copy link
Contributor

ludokx commented Oct 11, 2023

Releases are a bit of a pain but I'll go through and try and get one signed and uploaded tonight.

Awesome, thanks @davidrothera! Not urgent but would definitely appreciate it :)

@davidrothera
Copy link
Collaborator

@ludokx so it looks like its bad news at the moment.

Sonoma requires Xcode 15 and trying to build an archive build against that toolchain is failing due to an issue in the ShortcutRecorder library which we use.

Updating ShortcutRecorder to a later version also removes the PTHotKey framework which we use for some of the hotkey support.

Long story short is that its not as simple as building a new version and releasing it, it looks like this is going to need significant work to update and that is something which I know I don't have at the moment (baby due this week) and I doubt @pallotron has either sadly.

I think rather than trying to apply patches on top of the current code it might be worth building a modern ground-up version of yubiswitch with Swift (and SwiftUI) as I think that will make it easier to maintain into the future.

@davidrothera
Copy link
Collaborator

FWIW it seems that the support for XPC in Swift has improved a lot since I looked at it last and the later versions of ShortcutRecorder also have good Swift support

https://github.com/Kentzo/ShortcutRecorder
https://github.com/richardnees/Swift-macOS-XPC-Example

I can't make any promises on working on this however being able to rewrite in Swift would be a lot easier than maintaining C/Obj-C 😛

@ludokx
Copy link
Contributor

ludokx commented Oct 11, 2023

Ha yes I ran into that as I was trying to build locally 😬 Agreed on SwiftUI, would be more able to help with that as well, unfortunately don't have too much time to help with the rewrite.

Focus on family first and get some sleep while you still can 😜

@KeithMcD
Copy link

A glass-half-full perspective on this bug is that it makes it VERY obvious if your Yubikey is still enabled.

Good luck on the bug fix and congrats on the baby!

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

No branches or pull requests

4 participants