-
Notifications
You must be signed in to change notification settings - Fork 4
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
Feature/layout edge #29
Conversation
gaetanzanella
commented
Nov 18, 2021
•
edited
Loading
edited
- Added UIRectEdge+Utilities
- Added NSDirectionalRectEdge+Utilities
1913cf4
to
949c362
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.
Modulo les commentaires d'alex, LGTM
Change-Id: Icee7ae133aee1afb55960a365e8e2b8cd909d63c
Change-Id: I6762e2d17b0572e42ecd60538752c75d8f415f90
Change-Id: I3a48d69f84bff0b10a77233a85d8441a6c91321f
Change-Id: Id4e3b2d42989ec1f0472119fc34ed07cb26d5d30
Change-Id: I6874cfb1b5e455749fe4abc1ac1815230ddfc634
Change-Id: Ie4f8bc60644ad7ff423590fd297641d01653fc23
Change-Id: I737a1dd968fc8fcb89ca202894704dbb3f420a76
Change-Id: I56daa895cfa022c4d61bfed002e390253f8e77fa
Disable SecureArchiver if CryptoKit not available Change-Id: I4745a8e14387464652c154b4ad236b6cdca664ce
1b72716
to
4db02b6
Compare
import Nimble | ||
import ADUtils | ||
|
||
@available(iOS 13.0, tvOS 13.0, *) |
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.
on peut delete le @available(iOS 13.0, tvOS 13.0, *)
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.
en effet, je le retire
@@ -5,6 +5,8 @@ | |||
// Created by Thomas Esterlin on 02/07/2021. | |||
// Methods inspired by these : "https://fred.appelman.net/?p=119", "https://developer.apple.com/videos/play/wwdc2019/709" | |||
|
|||
#if canImport(CryptoKit) |
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.
on n'a plus besoin de ce check avec iOS 14+ non ?
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.
ha oui en effet, je me demandais pourquoi il avait été mis et j'avais pas pensé a la version d'iOS
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.
ouais j'imagine que c'était pour du iOS 12 par exemple 🤷