Merge pull request #318 from specklesystems/david/move-text-class-update-pr-to-main

fix (sdk) Text class updates: removed origin from Text class and added screenAligned prop

src/Speckle.Objects/Annotation/Text.cs

@@ -1,6 +1,5 @@
 using Speckle.Objects.Geometry;
 using Speckle.Sdk.Models;
-using Point = Speckle.Objects.Geometry.Point;
 
 namespace Speckle.Objects.Annotation;
 
@@ -15,11 +14,6 @@ public class Text : Base
   /// </summary>
   public required string value { get; set; }
 
-  /// <summary>
-  /// Origin point, relation to the text is defined by AlignmentHorizontal and AlignmentVertical
-  /// </summary>
-  public required Point origin { get; set; }
-
   /// <summary>
   /// Height in linear units or pixels (if Units.None)
   /// </summary>
@@ -31,6 +25,11 @@ public class Text : Base
   /// </summary>
   public required string units { get; set; }
 
+  /// <summary>
+  /// If true, the text is oriented to face the screen (camera-aligned).
+  /// </summary>
+  public required bool screenOriented { get; set; }
+
   /// <summary>
   /// Horizontal alignment: Left, Center or Right
   /// </summary>
@@ -42,9 +41,9 @@ public class Text : Base
   public AlignmentVertical alignmentV { get; set; }
 
   /// <summary>
-  /// Plane will be null if the text object orientation follows camera view
+  /// Plane axis vectors will be ignored if screenOriented is true
   /// </summary>
-  public Plane? plane { get; set; }
+  public required Plane plane { get; set; }
 
   /// <summary>
   /// Maximum width of the text field (in 'units').