search

Clipy / KeyHolder

Record shortcuts in macOS, like Alfred.app.
https://clipy-app.com
MIT License
392 stars 33 forks source link

Add #if os(macOS) macro #72

Closed akring closed 7 months ago

akring commented 7 months ago

Thank you very much for developing such a useful library.

I'm working on a SwiftUI-based project that also supports iOS/macOS, and when I'm using Keyholder, everything is perfect, the only problem is that it breaks the preview of the iOS platform and keeps it in an uncompilable state.

After some experimentation, I found that I needed to add the #if os (macOS) macro to the Keyholder and associated Sauce/Magnet to explicitly tell the preview not to import and compile this part of the code on iOS platforms.

SwiftUI and cross-platform apps are becoming more and more common, and preview is a very powerful feature in SwiftUI. So I think this change is necessary.

Thanks!

akring commented 7 months ago

Hi, I would like to ask if there is anything that needs to be modified in this PR?