Moved OnLinkClicked event to MarkdownTextBlock.

CommunityToolkit · Nov 22, 2024 · 02d27e4 · 02d27e4
1 parent a5ac5d9
commit 02d27e4
Show file tree

Hide file tree

Showing 8 changed files with 21 additions and 15 deletions.
diff --git a/components/MarkdownTextBlock/samples/MarkdownTextBlockCustomSample.xaml b/components/MarkdownTextBlock/samples/MarkdownTextBlockCustomSample.xaml
@@ -1,4 +1,4 @@
-<!--  Licensed to the .NET Foundation under one or more agreements. The .NET Foundation licenses this file to you under the MIT license. See the LICENSE file in the project root for more information.  -->
+<!--  Licensed to the .NET Foundation under one or more agreements. The .NET Foundation licenses this file to you under the MIT license. See the LICENSE file in the project root for more information.  -->
 <Page x:Class="MarkdownTextBlockExperiment.Samples.MarkdownTextBlockCustomSample"
       xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
       xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
@@ -25,7 +25,8 @@
                 <ColumnDefinition Width="*" />
                 <ColumnDefinition Width="*" />
             </Grid.ColumnDefinitions>
-            <controls:MarkdownTextBlock Grid.Column="0"
+            <controls:MarkdownTextBlock x:Name="MarkdownTextBlock1"
+                                        Grid.Column="0"
                                         Config="{x:Bind LiveMarkdownConfig, Mode=OneTime}"
                                         Text="{x:Bind MarkdownTextBox.Text, Mode=OneWay}" />
             <TextBox x:Name="MarkdownTextBox"
@@ -37,7 +38,8 @@
                    FontSize="16"
                    FontWeight="Bold"
                    Text="Built-in Sample" />
-        <controls:MarkdownTextBlock Grid.Row="3"
+        <controls:MarkdownTextBlock x:Name="MarkdownTextBlock2"
+                                    Grid.Row="3"
                                     Config="{x:Bind MarkdownConfig, Mode=OneTime}"
                                     Text="{x:Bind Text, Mode=OneTime}" />
     </Grid>

diff --git a/components/MarkdownTextBlock/samples/MarkdownTextBlockCustomSample.xaml.cs b/components/MarkdownTextBlock/samples/MarkdownTextBlockCustomSample.xaml.cs
@@ -604,14 +604,14 @@ public MarkdownTextBlockCustomSample()
     {
         this.InitializeComponent();
         _config = new MarkdownConfig();
-        _config.OnLinkClicked += this._config_OnLinkClicked;
         _liveConfig = new MarkdownConfig();
-        _liveConfig.OnLinkClicked += this._config_OnLinkClicked;
         _text = _markdown;
         MarkdownTextBox.Text = "# Hello World\n\n";
+        MarkdownTextBlock1.OnLinkClicked += MarkdownTextBlock_OnLinkClicked;
+        MarkdownTextBlock2.OnLinkClicked += MarkdownTextBlock_OnLinkClicked;
     }
 
-    private void _config_OnLinkClicked(object? sender, LinkClickedEventArgs e)
+    private void MarkdownTextBlock_OnLinkClicked(object? sender, LinkClickedEventArgs e)
     {
         Debug.WriteLine($"Link Clicked: {e.Uri}");
     }

diff --git a/components/MarkdownTextBlock/src/HtmlWriter.cs b/components/MarkdownTextBlock/src/HtmlWriter.cs
@@ -36,7 +36,7 @@ public static void WriteHtml(WinUIRenderer renderer, HtmlNodeCollection nodes)
                         var myHyperlinkButton = new MyHyperlinkButton(node, renderer.Config.BaseUrl);
                         myHyperlinkButton.ClickEvent += (sender, e) =>
                         {
-                            renderer.Config.RaiseLinkClickedEvent(renderer, ((HyperlinkButton)sender).NavigateUri);
+                            renderer.MarkdownTextBlock.RaiseLinkClickedEvent(((HyperlinkButton)sender).NavigateUri);
                         };
                         hyperLink = myHyperlinkButton;
                     }
@@ -45,7 +45,7 @@ public static void WriteHtml(WinUIRenderer renderer, HtmlNodeCollection nodes)
                         var myHyperlink = new MyHyperlink(node, renderer.Config.BaseUrl);
                         myHyperlink.ClickEvent += (sender, e) =>
                         {
-                            renderer.Config.RaiseLinkClickedEvent(renderer, sender.NavigateUri);
+                            renderer.MarkdownTextBlock.RaiseLinkClickedEvent(sender.NavigateUri);
                         };
                         hyperLink = myHyperlink;
                     }

diff --git a/components/MarkdownTextBlock/src/MarkdownConfig.cs b/components/MarkdownTextBlock/src/MarkdownConfig.cs
@@ -12,8 +12,6 @@ public record MarkdownConfig
     public IImageProvider? ImageProvider { get; set; }
     public ISVGRenderer? SVGRenderer { get; set; }
     public MarkdownThemes Themes { get; set; } = MarkdownThemes.Default;
-    public event EventHandler<LinkClickedEventArgs>? OnLinkClicked;
-    internal void RaiseLinkClickedEvent(object? sender, Uri uri) => OnLinkClicked?.Invoke(sender, new LinkClickedEventArgs(uri));
 
     public static MarkdownConfig Default = new();
 }
diff --git a/components/MarkdownTextBlock/src/MarkdownTextBlock.xaml.cs b/components/MarkdownTextBlock/src/MarkdownTextBlock.xaml.cs
@@ -42,6 +42,10 @@ public string Text
         set => SetValue(TextProperty, value);
     }
 
+    public event EventHandler<LinkClickedEventArgs>? OnLinkClicked;
+
+    internal void RaiseLinkClickedEvent(Uri uri) => OnLinkClicked?.Invoke(this, new LinkClickedEventArgs(uri));
+
     private static void OnConfigChanged(DependencyObject d, DependencyPropertyChangedEventArgs e)
     {
         if (d is MarkdownTextBlock self && e.NewValue != null)
@@ -110,7 +114,7 @@ private void Build()
         {
             if (_renderer == null)
             {
-                _renderer = new WinUIRenderer(_document, Config);
+                _renderer = new WinUIRenderer(_document, Config, this);
             }
             _pipeline.Setup(_renderer);
             ApplyText(false);

diff --git a/components/MarkdownTextBlock/src/Renderers/ObjectRenderers/Inlines/AutoLinkInlineRenderer.cs b/components/MarkdownTextBlock/src/Renderers/ObjectRenderers/Inlines/AutoLinkInlineRenderer.cs
@@ -28,7 +28,7 @@ protected override void Write(WinUIRenderer renderer, AutolinkInline link)
         var autolink = new MyAutolinkInline(link);
         autolink.ClickEvent += (sender, e) =>
         {
-            renderer.Config.RaiseLinkClickedEvent(renderer, sender.NavigateUri);
+            renderer.MarkdownTextBlock.RaiseLinkClickedEvent(sender.NavigateUri);
         };
 
         renderer.Push(autolink);

diff --git a/components/MarkdownTextBlock/src/Renderers/ObjectRenderers/Inlines/LinkInlineRenderer.cs b/components/MarkdownTextBlock/src/Renderers/ObjectRenderers/Inlines/LinkInlineRenderer.cs
@@ -33,7 +33,7 @@ protected override void Write(WinUIRenderer renderer, LinkInline link)
                 var myHyperlinkButton = new MyHyperlinkButton(link, renderer.Config.BaseUrl);
                 myHyperlinkButton.ClickEvent += (sender, e) =>
                 {
-                    renderer.Config.RaiseLinkClickedEvent(renderer, ((HyperlinkButton)sender).NavigateUri);
+                    renderer.MarkdownTextBlock.RaiseLinkClickedEvent(((HyperlinkButton)sender).NavigateUri);
                 };
                 renderer.Push(myHyperlinkButton);
             }
@@ -42,7 +42,7 @@ protected override void Write(WinUIRenderer renderer, LinkInline link)
                 var hyperlink = new MyHyperlink(link, renderer.Config.BaseUrl);
                 hyperlink.ClickEvent += (sender, e) =>
                 {
-                    renderer.Config.RaiseLinkClickedEvent(renderer, sender.NavigateUri);
+                    renderer.MarkdownTextBlock.RaiseLinkClickedEvent(sender.NavigateUri);
                 };
 
                 renderer.Push(hyperlink);

diff --git a/components/MarkdownTextBlock/src/Renderers/WinUIRenderer.cs b/components/MarkdownTextBlock/src/Renderers/WinUIRenderer.cs
@@ -23,11 +23,13 @@ public MarkdownConfig Config
         get => _config;
         set => _config = value;
     }
+    public MarkdownTextBlock MarkdownTextBlock { get; }
 
-    public WinUIRenderer(MyFlowDocument document, MarkdownConfig config)
+    public WinUIRenderer(MyFlowDocument document, MarkdownConfig config, MarkdownTextBlock markdownTextBlock)
     {
         _buffer = new char[1024];
         Config = config;
+        MarkdownTextBlock = markdownTextBlock;
         FlowDocument = document;
         // set style
         _stack.Push(FlowDocument);