From ce53e1f376234297c2a449343a48de8a7f4dc56d Mon Sep 17 00:00:00 2001
From: Murray Stevenson <50844517+murraystevenson@users.noreply.github.com>
Date: Thu, 18 Jul 2024 12:47:14 -0700
Subject: [PATCH] LightEditor : Add todos

---
 src/GafferSceneUIModule/LightEditorBinding.cpp | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/src/GafferSceneUIModule/LightEditorBinding.cpp b/src/GafferSceneUIModule/LightEditorBinding.cpp
index f6275f1a30..fcb077c804 100644
--- a/src/GafferSceneUIModule/LightEditorBinding.cpp
+++ b/src/GafferSceneUIModule/LightEditorBinding.cpp
@@ -274,6 +274,8 @@ class InspectorColumn : public PathColumn
 
 };
 
+/// \todo `MuteColumn` and `SetMembershipColumn` should not exist and we intend
+/// to continue refactoring until it is possible to remove them entirely.
 class MuteColumn : public InspectorColumn
 {
 
@@ -340,6 +342,8 @@ class MuteColumn : public InspectorColumn
 			}
 
 			result.value = nullptr;
+			/// \todo Remove this once AttributeInspector can provide a default value when
+			/// no attribute exists, then InspectorColumn would always provide the toggle tooltip.
 			if( auto toolTipData = runTimeCast<const StringData>( result.toolTip ) )
 			{
 				std::string toolTip = toolTipData->readable();