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

feat: Customize crop toolbar color #178

Merged
merged 2 commits into from
Jul 12, 2022
Merged

Conversation

guoyingtao
Copy link
Owner

No description provided.

Add CropToolbarConfig to CropToolbarProtocol
extract Config, Definition and Enum from  Mantis.swift
fix some typos
@@ -21,7 +21,7 @@ Pod::Spec.new do |s|
s.social_media_url = "http://twitter.com/guoyingtao"
s.platform = :ios
s.swift_version = "5.0"
s.ios.deployment_target = "11.0"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same question as here: #177 (comment)

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is because of my previous PR #169
I added a CropToolbarIconProvider also I decided to use SF Symbol for CropToolbar icons instead of drawing them by myself, which needs iOS 13.
With apple's iOS 16 coming soon, I think it might be a good timing to move to iOS 13 because it also supports Combine, SwiftUI etc.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I know that iOS 16 is coming soon, and from a developers perspective we always want higher min versions. I personally still support up to 12+

Other than asking nicely, is there anything I can do so that you consider supporting 12+? I know the benefit of using SF Symbols is nice, but it's not even like we are able to replace all images with symbols.

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you still want to support iOS 12 for a while, I think I need to create a dedicated branch for it then rebase new code, make dedicated releases from that branch.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd very much appreciate that.

@guoyingtao guoyingtao merged commit eaa8f06 into master Jul 12, 2022
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.

None yet

2 participants