Selaa lähdekoodia

Added BUGBUGs

Tig 1 vuosi sitten
vanhempi
commit
3d00102125

+ 3 - 0
Terminal.Gui/Views/ListView.cs

@@ -122,7 +122,10 @@ public class ListView : View, IDesignable
         CanFocus = true;
 
         // Things this view knows how to do
+        // 
+        // BUGBUG: SHould return false if selectokn doesn't change (to support nav to next view)
         AddCommand (Command.LineUp, () => MoveUp ());
+        // BUGBUG: SHould return false if selectokn doesn't change (to support nav to next view)
         AddCommand (Command.LineDown, () => MoveDown ());
         AddCommand (Command.ScrollUp, () => ScrollVertical (-1));
         AddCommand (Command.ScrollDown, () => ScrollVertical (1));

+ 1 - 0
Terminal.Gui/Views/TableView/CellActivatedEventArgs.cs

@@ -1,5 +1,6 @@
 namespace Terminal.Gui;
 
+// TOOD: SHould support Handled
 /// <summary>Defines the event arguments for <see cref="TableView.CellActivated"/> event</summary>
 public class CellActivatedEventArgs : EventArgs
 {

+ 5 - 0
Terminal.Gui/Views/TableView/TableView.cs

@@ -56,6 +56,7 @@ public class TableView : View
                     Command.Right,
                     () =>
                     {
+                        // BUGBUG: SHould return false if selectokn doesn't change (to support nav to next view)
                         ChangeSelectionByOffset (1, 0, false);
 
                         return true;
@@ -66,6 +67,7 @@ public class TableView : View
                     Command.Left,
                     () =>
                     {
+                        // BUGBUG: SHould return false if selectokn doesn't change (to support nav to next view)
                         ChangeSelectionByOffset (-1, 0, false);
 
                         return true;
@@ -76,6 +78,7 @@ public class TableView : View
                     Command.LineUp,
                     () =>
                     {
+                        // BUGBUG: SHould return false if selectokn doesn't change (to support nav to next view)
                         ChangeSelectionByOffset (0, -1, false);
 
                         return true;
@@ -86,6 +89,7 @@ public class TableView : View
                     Command.LineDown,
                     () =>
                     {
+                        // BUGBUG: SHould return false if selectokn doesn't change (to support nav to next view)
                         ChangeSelectionByOffset (0, 1, false);
 
                         return true;
@@ -266,6 +270,7 @@ public class TableView : View
                     Command.Accept,
                     () =>
                     {
+                        // BUGBUG: This should return false if the event is not handled
                         OnCellActivated (new CellActivatedEventArgs (Table, SelectedColumn, SelectedRow));
 
                         return true;