Add warning if sizeOffset is set when it should not

eonist · Jun 29, 2023 · f5f15f9 · f5f15f9
1 parent 6a4a5ec
commit f5f15f9
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/Sources/Spatial/view/View+Access.swift b/Sources/Spatial/view/View+Access.swift
@@ -24,9 +24,10 @@ extension View {
     */
    public func anchorAndSize(to: View, sizeTo: View? = nil, width: CGFloat? = nil, height: CGFloat? = nil, align: Alignment = .topLeft, alignTo: Alignment = .topLeft, multiplier: CGSize = .init(width: 1, height: 1), offset: CGPoint = .zero, sizeOffset: CGSize = .zero, useMargin: Bool = false) {
       self.activateAnchorAndSize { _ in
-         let anchor = Constraint.anchor(self, to: to, align: align, alignTo: alignTo, offset: offset, useMargin: useMargin)
+         let anchor: AnchorConstraint = Constraint.anchor(self, to: to, align: align, alignTo: alignTo, offset: offset, useMargin: useMargin)
          let size: SizeConstraint = {
             if let width = width, let height = height { // This method exists when you have size, but don't want to set size based on another view
+               if sizeOffset != .zero { Swift.print("⚠️️ sizeOffset does not affect constraints when width and height are predfined, set offset to the width and height directly") }
                return Constraint.size(self, size: .init(width: width, height: height), multiplier: multiplier)
             } else {
                return Constraint.size(self, to: sizeTo ?? to, width: width, height: height, offset: sizeOffset, multiplier: multiplier)

diff --git a/Sources/Spatial/view/View+Size.swift b/Sources/Spatial/view/View+Size.swift
@@ -34,7 +34,7 @@ extension Constraint {
    }
    /**
     * Creates a size constraint
-    * - Fixme: ⚠️️ This doesn't have offset, maybe it should, for now I guess you can always inset the size
+    * - Fixme: ⚠️️ This doesn't have offset, maybe it should, for now I guess you can always inset the size, or add etc
     * ## Examples:
     * let sizeConstraint = Constraint.size(square, size: .init(width: 100, height: 100))
     * - Parameters: