Browse Source

Fixed feedback from bdisp

Tig 8 months ago
parent
commit
d3d3df8f15
2 changed files with 6 additions and 5 deletions
  1. 6 3
      Terminal.Gui/Views/TabMouseEventArgs.cs
  2. 0 2
      Terminal.Gui/Views/TileView.cs

+ 6 - 3
Terminal.Gui/Views/TabMouseEventArgs.cs

@@ -1,7 +1,10 @@
-namespace Terminal.Gui;
+#nullable enable
+using System.ComponentModel;
+
+namespace Terminal.Gui;
 
 /// <summary>Describes a mouse event over a specific <see cref="Tab"/> in a <see cref="TabView"/>.</summary>
-public class TabMouseEventArgs : EventArgs
+public class TabMouseEventArgs : HandledEventArgs
 {
     /// <summary>Creates a new instance of the <see cref="TabMouseEventArgs"/> class.</summary>
     /// <param name="tab"><see cref="Tab"/> that the mouse was over when the event occurred.</param>
@@ -13,7 +16,7 @@ public class TabMouseEventArgs : EventArgs
     }
 
     /// <summary>
-    ///     Gets the actual mouse event.  Use <see cref="MouseEventArgs.Handled"/> to cancel this event and perform custom
+    ///     Gets the actual mouse event.  Use <see cref="HandledEventArgs.Handled"/> to cancel this event and perform custom
     ///     behavior (e.g. show a context menu).
     /// </summary>
     public MouseEventArgs MouseEvent { get; }

+ 0 - 2
Terminal.Gui/Views/TileView.cs

@@ -174,11 +174,9 @@ public class TileView : View
     // BUG: v2 fix this hack
     // QUESTION: Does this need to be fixed before events are refactored?
     /// <summary>Overridden so no Frames get drawn</summary>
-    /// <param name="clipRegion"></param>
     /// <returns></returns>
     protected override bool OnDrawingBorderAndPadding () { return true; }
 
-    /// <param name="clipRegion"></param>
     /// <inheritdoc/>
     protected override bool OnRenderingLineCanvas () { return false; }