From b06e4229c4ac7213e2739a7ef4f0c6ee57dae6cc Mon Sep 17 00:00:00 2001 From: Charlie Kindel Date: Wed, 2 Nov 2022 15:34:47 -0600 Subject: [PATCH] Revert "Revert "Fixes #1743 - CharMap renders painfully slow"" This reverts commit 9945c2dd545f01729dedba5c96950c47657d0723. --- Terminal.Gui/Core/ConsoleDriver.cs | 5 +- Terminal.Gui/Core/View.cs | 15 ++- Terminal.Gui/Views/ScrollBarView.cs | 28 ++-- Terminal.Gui/Views/ScrollView.cs | 5 +- UICatalog/Properties/launchSettings.json | 4 + UICatalog/Scenarios/CharacterMap.cs | 162 +++++++++++------------ UnitTests/ViewTests.cs | 18 +++ 7 files changed, 127 insertions(+), 110 deletions(-) diff --git a/Terminal.Gui/Core/ConsoleDriver.cs b/Terminal.Gui/Core/ConsoleDriver.cs index cd180af1c..8ace3d396 100644 --- a/Terminal.Gui/Core/ConsoleDriver.cs +++ b/Terminal.Gui/Core/ConsoleDriver.cs @@ -683,7 +683,7 @@ namespace Terminal.Gui { public abstract void Move (int col, int row); /// - /// Adds the specified rune to the display at the current cursor position + /// Adds the specified rune to the display at the current cursor position. /// /// Rune to add. public abstract void AddRune (Rune rune); @@ -717,10 +717,11 @@ namespace Terminal.Gui { col >= 0 && row >= 0 && col < Cols && row < Rows && clip.Contains (col, row); /// - /// Adds the specified + /// Adds the to the display at the cursor position. /// /// String. public abstract void AddStr (ustring str); + /// /// Prepare the driver and set the key and mouse events handlers. /// diff --git a/Terminal.Gui/Core/View.cs b/Terminal.Gui/Core/View.cs index 0fcd73b07..72f88d357 100644 --- a/Terminal.Gui/Core/View.cs +++ b/Terminal.Gui/Core/View.cs @@ -3080,12 +3080,17 @@ namespace Terminal.Gui { /// The view. /// The method name. /// if it's overridden, otherwise. - public bool IsOverridden (View view, string method) + public static bool IsOverridden (View view, string method) { - Type t = view.GetType (); - MethodInfo m = t.GetMethod (method); - - return (m.DeclaringType == t || m.ReflectedType == t) && m.GetBaseDefinition ().DeclaringType == typeof (Responder); + MethodInfo m = view.GetType ().GetMethod (method, + BindingFlags.Instance + | BindingFlags.Public + | BindingFlags.NonPublic + | BindingFlags.DeclaredOnly); + if (m == null) { + return false; + } + return m.GetBaseDefinition ().DeclaringType != m.DeclaringType; } } } diff --git a/Terminal.Gui/Views/ScrollBarView.cs b/Terminal.Gui/Views/ScrollBarView.cs index 80c5b6ee3..ad0f469f5 100644 --- a/Terminal.Gui/Views/ScrollBarView.cs +++ b/Terminal.Gui/Views/ScrollBarView.cs @@ -462,7 +462,7 @@ namespace Terminal.Gui { return; } - Driver.SetAttribute (GetNormalColor ()); + Driver.SetAttribute (Host.HasFocus ? ColorScheme.Focus : GetNormalColor ()); if ((vertical && Bounds.Height == 0) || (!vertical && Bounds.Width == 0)) { return; @@ -613,13 +613,13 @@ namespace Terminal.Gui { int posBarOffset; /// - public override bool MouseEvent (MouseEvent me) + public override bool MouseEvent (MouseEvent mouseEvent) { - if (me.Flags != MouseFlags.Button1Pressed && me.Flags != MouseFlags.Button1DoubleClicked && - !me.Flags.HasFlag (MouseFlags.Button1Pressed | MouseFlags.ReportMousePosition) && - me.Flags != MouseFlags.Button1Released && me.Flags != MouseFlags.WheeledDown && - me.Flags != MouseFlags.WheeledUp && me.Flags != MouseFlags.WheeledRight && - me.Flags != MouseFlags.WheeledLeft && me.Flags != MouseFlags.Button1TripleClicked) { + if (mouseEvent.Flags != MouseFlags.Button1Pressed && mouseEvent.Flags != MouseFlags.Button1DoubleClicked && + !mouseEvent.Flags.HasFlag (MouseFlags.Button1Pressed | MouseFlags.ReportMousePosition) && + mouseEvent.Flags != MouseFlags.Button1Released && mouseEvent.Flags != MouseFlags.WheeledDown && + mouseEvent.Flags != MouseFlags.WheeledUp && mouseEvent.Flags != MouseFlags.WheeledRight && + mouseEvent.Flags != MouseFlags.WheeledLeft && mouseEvent.Flags != MouseFlags.Button1TripleClicked) { return false; } @@ -630,24 +630,24 @@ namespace Terminal.Gui { Host.SetFocus (); } - int location = vertical ? me.Y : me.X; + int location = vertical ? mouseEvent.Y : mouseEvent.X; int barsize = vertical ? Bounds.Height : Bounds.Width; int posTopLeftTee = vertical ? posTopTee + 1 : posLeftTee + 1; int posBottomRightTee = vertical ? posBottomTee + 1 : posRightTee + 1; barsize -= 2; var pos = Position; - if (me.Flags != MouseFlags.Button1Released + if (mouseEvent.Flags != MouseFlags.Button1Released && (Application.MouseGrabView == null || Application.MouseGrabView != this)) { Application.GrabMouse (this); - } else if (me.Flags == MouseFlags.Button1Released && Application.MouseGrabView != null && Application.MouseGrabView == this) { + } else if (mouseEvent.Flags == MouseFlags.Button1Released && Application.MouseGrabView != null && Application.MouseGrabView == this) { lastLocation = -1; Application.UngrabMouse (); return true; } - if (showScrollIndicator && (me.Flags == MouseFlags.WheeledDown || me.Flags == MouseFlags.WheeledUp || - me.Flags == MouseFlags.WheeledRight || me.Flags == MouseFlags.WheeledLeft)) { - return Host.MouseEvent (me); + if (showScrollIndicator && (mouseEvent.Flags == MouseFlags.WheeledDown || mouseEvent.Flags == MouseFlags.WheeledUp || + mouseEvent.Flags == MouseFlags.WheeledRight || mouseEvent.Flags == MouseFlags.WheeledLeft)) { + return Host.MouseEvent (mouseEvent); } if (location == 0) { @@ -668,7 +668,7 @@ namespace Terminal.Gui { //} if (lastLocation > -1 || (location >= posTopLeftTee && location <= posBottomRightTee - && me.Flags.HasFlag (MouseFlags.Button1Pressed | MouseFlags.ReportMousePosition))) { + && mouseEvent.Flags.HasFlag (MouseFlags.Button1Pressed | MouseFlags.ReportMousePosition))) { if (lastLocation == -1) { lastLocation = location; posBarOffset = keepContentAlwaysInViewport ? Math.Max (location - posTopLeftTee, 1) : 0; diff --git a/Terminal.Gui/Views/ScrollView.cs b/Terminal.Gui/Views/ScrollView.cs index c42308dd0..23bfedbce 100644 --- a/Terminal.Gui/Views/ScrollView.cs +++ b/Terminal.Gui/Views/ScrollView.cs @@ -32,6 +32,7 @@ namespace Terminal.Gui { private class ContentView : View { public ContentView (Rect frame) : base (frame) { + CanFocus = true; } } @@ -325,7 +326,7 @@ namespace Terminal.Gui { { Driver.SetAttribute (GetNormalColor ()); SetViewsNeedsDisplay (); - Clear (); + //Clear (); var savedClip = ClipToBounds (); OnDrawContent (new Rect (ContentOffset, @@ -507,7 +508,7 @@ namespace Terminal.Gui { { if (me.Flags != MouseFlags.WheeledDown && me.Flags != MouseFlags.WheeledUp && me.Flags != MouseFlags.WheeledRight && me.Flags != MouseFlags.WheeledLeft && - me.Flags != MouseFlags.Button1Pressed && me.Flags != MouseFlags.Button1Clicked && +// me.Flags != MouseFlags.Button1Pressed && me.Flags != MouseFlags.Button1Clicked && !me.Flags.HasFlag (MouseFlags.Button1Pressed | MouseFlags.ReportMousePosition)) { return false; } diff --git a/UICatalog/Properties/launchSettings.json b/UICatalog/Properties/launchSettings.json index eda283ad8..93ff11151 100644 --- a/UICatalog/Properties/launchSettings.json +++ b/UICatalog/Properties/launchSettings.json @@ -44,6 +44,10 @@ "WSL": { "commandName": "WSL2", "distributionName": "" + }, + "Charmap": { + "commandName": "Project", + "commandLineArgs": "\"Character Map\"" } } } \ No newline at end of file diff --git a/UICatalog/Scenarios/CharacterMap.cs b/UICatalog/Scenarios/CharacterMap.cs index 2036e566d..667e0aafa 100644 --- a/UICatalog/Scenarios/CharacterMap.cs +++ b/UICatalog/Scenarios/CharacterMap.cs @@ -2,6 +2,7 @@ //#define BASE_DRAW_CONTENT using NStack; +using System; using System.Collections.Generic; using System.Linq; using System.Text; @@ -11,11 +12,12 @@ using Rune = System.Rune; namespace UICatalog.Scenarios { /// /// This Scenario demonstrates building a custom control (a class deriving from View) that: - /// - Provides a simple "Character Map" application (like Windows' charmap.exe). + /// - Provides a "Character Map" application (like Windows' charmap.exe). /// - Helps test unicode character rendering in Terminal.Gui /// - Illustrates how to use ScrollView to do infinite scrolling /// - [ScenarioMetadata (Name: "Character Map", Description: "A Unicode character set viewier built as a custom control using the ScrollView control.")] + [ScenarioMetadata (Name: "Character Map", + Description: "A Unicode character set viewier built as a custom control using the ScrollView control.")] [ScenarioCategory ("Text and Formatting")] [ScenarioCategory ("Controls")] [ScenarioCategory ("ScrollView")] @@ -26,28 +28,15 @@ namespace UICatalog.Scenarios { _charMap = new CharMap () { X = 0, Y = 0, - Width = CharMap.RowWidth + 2, Height = Dim.Fill (), - Start = 0x2500, - ColorScheme = Colors.Dialog, - CanFocus = true, }; - Win.Add (_charMap); - var label = new Label ("Jump To Unicode Block:") { X = Pos.Right (_charMap) + 1, Y = Pos.Y (_charMap) }; - Win.Add (label); - - (ustring radioLabel, int start, int end) CreateRadio (ustring title, int start, int end) - { - return ($"{title} (U+{start:x5}-{end:x5})", start, end); - } - var radioItems = new (ustring radioLabel, int start, int end) [] { - CreateRadio("ASCII Control Characterss", 0x00, 0x1F), + CreateRadio("ASCII Control Characters", 0x00, 0x1F), CreateRadio("C0 Control Characters", 0x80, 0x9f), CreateRadio("Hangul Jamo", 0x1100, 0x11ff), // This is where wide chars tend to start CreateRadio("Currency Symbols", 0x20A0, 0x20CF), - CreateRadio("Letterlike Symbols", 0x2100, 0x214F), + CreateRadio("Letter-like Symbols", 0x2100, 0x214F), CreateRadio("Arrows", 0x2190, 0x21ff), CreateRadio("Mathematical symbols", 0x2200, 0x22ff), CreateRadio("Miscellaneous Technical", 0x2300, 0x23ff), @@ -55,17 +44,25 @@ namespace UICatalog.Scenarios { CreateRadio("Miscellaneous Symbols", 0x2600, 0x26ff), CreateRadio("Dingbats", 0x2700, 0x27ff), CreateRadio("Braille", 0x2800, 0x28ff), - CreateRadio("Miscellaneous Symbols and Arrows", 0x2b00, 0x2bff), - CreateRadio("Alphabetic Presentation Forms", 0xFB00, 0xFb4f), - CreateRadio("Cuneiform Numbers and Punctuation", 0x12400, 0x1240f), + CreateRadio("Miscellaneous Symbols & Arrows", 0x2b00, 0x2bff), + CreateRadio("Alphabetic Pres. Forms", 0xFB00, 0xFb4f), + CreateRadio("Cuneiform Num. and Punct.", 0x12400, 0x1240f), CreateRadio("Chess Symbols", 0x1FA00, 0x1FA0f), CreateRadio("End", CharMap.MaxCodePointVal - 16, CharMap.MaxCodePointVal), }; + (ustring radioLabel, int start, int end) CreateRadio (ustring title, int start, int end) + { + return ($"{title} (U+{start:x5}-{end:x5})", start, end); + } + + Win.Add (_charMap); + var label = new Label ("Jump To Unicode Block:") { X = Pos.Right (_charMap) + 1, Y = Pos.Y (_charMap) }; + Win.Add (label); var jumpList = new RadioGroup (radioItems.Select (t => t.radioLabel).ToArray ()) { X = Pos.X (label), Y = Pos.Bottom (label), - Width = Dim.Fill (), + Width = radioItems.Max (r => r.radioLabel.Length) + 3, SelectedItem = 8 }; jumpList.SelectedItemChanged += (args) => { @@ -76,11 +73,9 @@ namespace UICatalog.Scenarios { jumpList.Refresh (); jumpList.SetFocus (); - } - public override void Run () - { - base.Run (); + _charMap.Width = Dim.Fill () - jumpList.Width; + } } @@ -98,97 +93,90 @@ namespace UICatalog.Scenarios { SetNeedsDisplay (); } } + int _start = 0x2500; - public const int H_SPACE = 2; - public const int V_SPACE = 2; + public const int COLUMN_WIDTH = 3; + public const int ROW_HEIGHT = 1; public static int MaxCodePointVal => 0x10FFFF; - // Row Header + space + (space + char + space) - public static int RowHeaderWidth => $"U+{MaxCodePointVal:x5}".Length; - public static int RowWidth => RowHeaderWidth + (H_SPACE * 16); + public static int RowLabelWidth => $"U+{MaxCodePointVal:x5}".Length; + public static int RowWidth => RowLabelWidth + (COLUMN_WIDTH * 16); public CharMap () { + ColorScheme = Colors.Dialog; + CanFocus = true; + ContentSize = new Size (CharMap.RowWidth, MaxCodePointVal / 16); ShowVerticalScrollIndicator = true; ShowHorizontalScrollIndicator = false; LayoutComplete += (args) => { - if (Bounds.Width <= RowWidth) { + if (Bounds.Width < RowWidth) { ShowHorizontalScrollIndicator = true; } else { ShowHorizontalScrollIndicator = false; + // Snap 1st column into view if it's been scrolled horizontally + ContentOffset = new Point (0, ContentOffset.Y); + SetNeedsDisplay (); } }; -#if DRAW_CONTENT - DrawContent += CharMap_DrawContent; -#endif + + AddCommand (Command.ScrollUp, () => { ScrollUp (1); return true; }); + AddCommand (Command.ScrollDown, () => { ScrollDown (1); return true; }); + AddCommand (Command.ScrollLeft, () => { ScrollLeft (1); return true; }); + AddCommand (Command.ScrollRight, () => { ScrollRight (1); return true; }); } private void CharMap_DrawContent (Rect viewport) { - //Rune ReplaceNonPrintables (Rune c) - //{ - // if (c < 0x20) { - // return new Rune (c + 0x2400); // U+25A1 □ WHITE SQUARE - // } else { - // return c; - // } - //} - - ContentSize = new Size (CharMap.RowWidth, MaxCodePointVal / 16 + Frame.Height - 1); - - for (int header = 0; header < 16; header++) { - Move (viewport.X + RowHeaderWidth + (header * H_SPACE), 0); - Driver.AddStr ($" {header:x} "); + var oldClip = Driver.Clip; + Driver.Clip = Frame; + // Redraw doesn't know about the scroll indicators, so if off, add one to height + if (!ShowHorizontalScrollIndicator) { + Driver.Clip = new Rect (Frame.X, Frame.Y, Frame.Width, Frame.Height + 1); } - for (int row = 0, y = 0; row < viewport.Height / 2 - 1; row++, y += V_SPACE) { - int val = (-viewport.Y + row) * 16; + Driver.SetAttribute (HasFocus ? ColorScheme.HotFocus : ColorScheme.Focus); + Move (0, 0); + Driver.AddStr (new string (' ', RowLabelWidth + 1)); + for (int hexDigit = 0; hexDigit < 16; hexDigit++) { + var x = ContentOffset.X + RowLabelWidth + (hexDigit * COLUMN_WIDTH); + if (x > RowLabelWidth - 2) { + Move (x, 0); + Driver.AddStr ($" {hexDigit:x} "); + } + } + //Move (RowWidth, 0); + //Driver.AddRune (' '); + + var firstColumnX = viewport.X + RowLabelWidth; + for (int row = -ContentOffset.Y, y = 0; row <= (-ContentOffset.Y) + (Bounds.Height / ROW_HEIGHT); row++, y += ROW_HEIGHT) { + int val = (row) * 16; + Driver.SetAttribute (GetNormalColor ()); + Move (firstColumnX, y + 1); + Driver.AddStr (new string (' ', 16 * COLUMN_WIDTH)); if (val < MaxCodePointVal) { - var rowLabel = $"U+{val / 16:x4}x"; - Move (0, y + 1); - Driver.AddStr (rowLabel); - var prevColWasWide = false; + Driver.SetAttribute (GetNormalColor ()); for (int col = 0; col < 16; col++) { - var rune = new Rune ((uint)((uint)(-viewport.Y + row) * 16 + col)); - Move (viewport.X + RowHeaderWidth + (col * H_SPACE) + (prevColWasWide ? 0 : 1), y + 1); - if (rune >= 0x00D800 && rune <= 0x00DFFF) { - if (col == 0) { - Driver.AddStr ("Reserved to surrogate pairs."); - } - continue; - } + var rune = new Rune ((uint)((uint)val + col)); + //if (rune >= 0x00D800 && rune <= 0x00DFFF) { + // if (col == 0) { + // Driver.AddStr ("Reserved for surrogate pairs."); + // } + // continue; + //} + Move (firstColumnX + (col * COLUMN_WIDTH) + 1, y + 1); Driver.AddRune (rune); - //prevColWasWide = Rune.ColumnWidth (rune) > 1; } + Move (0, y + 1); + Driver.SetAttribute (HasFocus ? ColorScheme.HotFocus : ColorScheme.Focus); + var rowLabel = $"U+{val / 16:x4}x "; + Driver.AddStr (rowLabel); } } - } -#if BASE_DRAW_CONTENT - public override void OnDrawContent (Rect viewport) - { - CharMap_DrawContent (viewport); - base.OnDrawContent (viewport); - } -#endif - - public override bool ProcessKey (KeyEvent kb) - { - if (kb.Key == Key.PageDown) { - ContentOffset = new Point (0, ContentOffset.Y - Bounds.Height / 2 + 1); - return true; - } - if (kb.Key == Key.PageUp) { - if (ContentOffset.Y + Bounds.Height / 2 - 1 < 0) { - ContentOffset = new Point (0, ContentOffset.Y + Bounds.Height / 2 - 1); - } else { - ContentOffset = Point.Empty; - } - return true; - } - return base.ProcessKey (kb); + Driver.Clip = oldClip; } protected override void Dispose (bool disposing) diff --git a/UnitTests/ViewTests.cs b/UnitTests/ViewTests.cs index e2a20e80b..28306f8cc 100644 --- a/UnitTests/ViewTests.cs +++ b/UnitTests/ViewTests.cs @@ -4062,5 +4062,23 @@ This is a tes Assert.False (view.IsKeyPress); Assert.True (view.IsKeyUp); } + + [Fact, AutoInitShutdown] + public void IsOverridden_False_IfNotOverriden () + { + var view = new DerivedView () { Text = "Derived View does not override MouseEvent or Redraw", Width = 10, Height = 10 }; + + Assert.False (View.IsOverridden (view, "MouseEvent")); + Assert.False (View.IsOverridden (view, "Redraw")); + } + + [Fact, AutoInitShutdown] + public void IsOverridden_True_IfOverriden () + { + var view = new ScrollBarView () { Text = "ScrollBarView overrides both MouseEvent and Redraw", Width = 10, Height = 10 }; + + Assert.True (View.IsOverridden (view, "MouseEvent")); + Assert.True (View.IsOverridden (view, "Redraw")); + } } }