-
Notifications
You must be signed in to change notification settings - Fork 170
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
feat: add mirror mode support #200
Conversation
79e0527
to
880f8ae
Compare
issues to be solved UI is broken when rotating device
880f8ae
to
549f8ac
Compare
rename handleRotate to handleDeviceRotated chore: remove deprecated comment
b46c814
to
4cb8011
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works really well for me! Request changes since I think there's a bug with providing your own icons for the toolbar.
@@ -8,7 +8,7 @@ | |||
|
|||
import UIKit | |||
|
|||
enum ImageRotationType: CGFloat { | |||
public enum ImageRotationType: CGFloat { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why do we need this to be public now?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think I forgot deleting some test code. I will remove public
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed
|
||
static func verticallyFlipImage() -> UIImage? { | ||
if #available(macCatalyst 13.1, iOS 13.0, *) { | ||
guard let horizontallyFippedImage = horizontallyFlipImage(), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice idea that you just take the other icon and rotate / translate it!
Co-authored-by: Niklas Baudy <niklas.baudy@vanniktech.de>
Co-authored-by: Niklas Baudy <niklas.baudy@vanniktech.de>
Co-authored-by: Niklas Baudy <niklas.baudy@vanniktech.de>
Co-authored-by: Niklas Baudy <niklas.baudy@vanniktech.de>
Co-authored-by: Niklas Baudy <niklas.baudy@vanniktech.de>
Co-authored-by: Niklas Baudy <niklas.baudy@vanniktech.de>
Co-authored-by: Niklas Baudy <niklas.baudy@vanniktech.de>
Co-authored-by: Niklas Baudy <niklas.baudy@vanniktech.de>
use didSelectHorizontallyFlip and didSelectVerticallyFlip
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Amazing! Thank you for the implementation ❤️
No description provided.