From fba734bb159dd0a7028667e34ccb7e93ac73142f Mon Sep 17 00:00:00 2001
From: a1xd <68629610+a1xd@users.noreply.github.com>
Date: Sun, 29 Aug 2021 18:20:04 -0400
Subject: enable directionality options with by-component

---
 .../Directionality/DirectionalityOptions.cs        | 64 +++++++---------------
 1 file changed, 20 insertions(+), 44 deletions(-)

diff --git a/grapher/Models/Options/Directionality/DirectionalityOptions.cs b/grapher/Models/Options/Directionality/DirectionalityOptions.cs
index c21b932..9288132 100644
--- a/grapher/Models/Options/Directionality/DirectionalityOptions.cs
+++ b/grapher/Models/Options/Directionality/DirectionalityOptions.cs
@@ -72,59 +72,39 @@ namespace grapher.Models.Options.Directionality
 
         public DomainArgs GetDomainArgs()
         {
-            if (!ByComponentCheckBox.Checked)
+            return new DomainArgs
             {
-                return new DomainArgs
+                domainXY = new Vec2<double>
                 {
-                    domainXY = new Vec2<double>
-                    {
-                        x = Domain.Fields.X,
-                        y = Domain.Fields.Y,
-                    },
-                    lpNorm = LpNorm.Field.Data,
-                };
-            }
-            else
-            {
-                return new DomainArgs
-                {
-                    domainXY = new Vec2<double>
-                    {
-                        x = 1,
-                        y = 1,
-                    },
-                    lpNorm = 2,
-                };
-
-            }
+                    x = Domain.Fields.X,
+                    y = Domain.Fields.Y,
+                },
+                lpNorm = ByComponentCheckBox.Checked ? 2 : LpNorm.Field.Data
+            };
         }
 
         public Vec2<double> GetRangeXY()
         {
-            if (!ByComponentCheckBox.Checked)
-            {
-                return new Vec2<double>
-                {
-                    x = Range.Fields.X,
-                    y = Range.Fields.Y,
-                };
-            }
-            else
+            return new Vec2<double>
             {
-                return new Vec2<double>
-                {
-                    x = 1,
-                    y = 1,
-                };
-            }
-
+                x = Range.Fields.X,
+                y = Range.Fields.Y,
+            };
         }
 
         public void SetActiveValues(DriverSettings settings)
         {
             Domain.SetActiveValues(settings.domainArgs.domainXY.x, settings.domainArgs.domainXY.y);
-            LpNorm.SetActiveValue(settings.domainArgs.lpNorm);
             Range.SetActiveValues(settings.rangeXY.x, settings.rangeXY.y);
+
+            if (settings.combineMagnitudes)
+            {
+                LpNorm.SetActiveValue(settings.domainArgs.lpNorm);
+            }
+            else
+            {
+                LpNorm.SetToUnavailable();
+            }
         }
 
         public void Hide()
@@ -167,15 +147,11 @@ namespace grapher.Models.Options.Directionality
         public void ToByComponent()
         {
             LpNorm.SetToUnavailable();
-            Domain.SetToUnavailable();
-            Range.SetToUnavailable();
         }
 
         public void ToWhole()
         {
             LpNorm.SetToAvailable();
-            Domain.SetToAvailable();
-            Range.SetToAvailable();
         }
 
         private void DrawHidden()
-- 
cgit v1.2.3