From 82e064dc29f510018df6b1dab661ac2535b88eb7 Mon Sep 17 00:00:00 2001
From: RisaDev <151885272+RisaDev@users.noreply.github.com>
Date: Sun, 4 Feb 2024 22:32:51 +0300
Subject: [PATCH] Clipboard data warning

---
 .../Tabs/Templates/TemplateFileSystemSelector.cs |  1 -
 .../MainWindow/Tabs/Templates/TemplatePanel.cs   | 16 +++++++++++++---
 2 files changed, 13 insertions(+), 4 deletions(-)

diff --git a/CustomizePlus/UI/Windows/MainWindow/Tabs/Templates/TemplateFileSystemSelector.cs b/CustomizePlus/UI/Windows/MainWindow/Tabs/Templates/TemplateFileSystemSelector.cs
index aedc471..ed4dcdd 100644
--- a/CustomizePlus/UI/Windows/MainWindow/Tabs/Templates/TemplateFileSystemSelector.cs
+++ b/CustomizePlus/UI/Windows/MainWindow/Tabs/Templates/TemplateFileSystemSelector.cs
@@ -95,7 +95,6 @@ public class TemplateFileSystemSelector : FileSystemSelector<Template, TemplateS
         _popupSystem = popupSystem;
 
         _popupSystem.RegisterPopup("template_editor_active_warn", "You need to stop bone editing before doing this action"/*, false, new Vector2(5, 12)*/);
-        _popupSystem.RegisterPopup("clipboard_data_not_longterm", "Warning: clipboard data is not designed to be used as long-term way of storing your templates.\nCompatibility of clipboard data between different Customize+ is not guaranteed."/*, false, new Vector2(5, 12)*/);
         _popupSystem.RegisterPopup("clipboard_data_unsupported_version", "Clipboard data you are trying to use cannot be used in this version of Customize+.");
 
         _templateChangedEvent.Subscribe(OnTemplateChange, TemplateChanged.Priority.TemplateFileSystemSelector);
diff --git a/CustomizePlus/UI/Windows/MainWindow/Tabs/Templates/TemplatePanel.cs b/CustomizePlus/UI/Windows/MainWindow/Tabs/Templates/TemplatePanel.cs
index e7da62e..84525c1 100644
--- a/CustomizePlus/UI/Windows/MainWindow/Tabs/Templates/TemplatePanel.cs
+++ b/CustomizePlus/UI/Windows/MainWindow/Tabs/Templates/TemplatePanel.cs
@@ -14,6 +14,7 @@ using CustomizePlus.Templates;
 using CustomizePlus.Configuration.Data;
 using CustomizePlus.Core.Helpers;
 using CustomizePlus.Templates.Data;
+using OtterGui.Log;
 
 namespace CustomizePlus.UI.Windows.MainWindow.Tabs.Templates;
 
@@ -25,6 +26,8 @@ public class TemplatePanel
     private readonly BoneEditorPanel _boneEditor;
     private readonly PluginConfiguration _configuration;
     private readonly MessageService _messageService;
+    private readonly PopupSystem _popupSystem;
+    private readonly Logger _logger;
 
     private string? _newName;
     private Template? _changedTemplate;
@@ -38,7 +41,9 @@ public class TemplatePanel
         GameStateService gameStateService,
         BoneEditorPanel boneEditor,
         PluginConfiguration configuration,
-        MessageService messageService)
+        MessageService messageService,
+        PopupSystem popupSystem,
+        Logger logger)
     {
         _selector = selector;
         _manager = manager;
@@ -46,6 +51,10 @@ public class TemplatePanel
         _boneEditor = boneEditor;
         _configuration = configuration;
         _messageService = messageService;
+        _popupSystem = popupSystem;
+        _logger = logger;
+
+        _popupSystem.RegisterPopup("clipboard_data_not_longterm", "Warning: clipboard data is not designed to be used as long-term way of storing your templates.\nCompatibility of copied data between different Customize+ versions is not guaranteed.", true, new Vector2(5, 10));
     }
 
     public void Draw()
@@ -219,11 +228,12 @@ public class TemplatePanel
         try
         {
             Clipboard.SetText(Base64Helper.ExportToBase64(_selector.Selected!, Constants.ConfigurationVersion));
+            _popupSystem.ShowPopup("clipboard_data_not_longterm");
         }
         catch (Exception ex)
         {
-            _messageService.NotificationMessage(ex, $"Could not copy {_selector.Selected!.Name} data to clipboard.",
-                $"Could not copy data from template {_selector.Selected!.UniqueId} to clipboard", NotificationType.Error, false);
+            _logger.Error($"Could not copy data from template {_selector.Selected!.UniqueId} to clipboard: {ex}");
+            _popupSystem.ShowPopup("action_error");
         }
     }
 }