- fix: can't paste contents copied from Clion/IDEA/PyCharm

NEWS.md

@@ -3,6 +3,7 @@ Red Panda C++ Version 1.1.1
   - change: swap position of problem case's output and expected input controls
   - enhancement: when problem case panel is positioned at right, problem case's input, output and expected controls is layouted vertically
   - enhancement: add ignore spaces checkbox in problem cases panel
+  - fix: can't paste contents copied from Clion/IDEA/PyCharm
 
 Red Panda C++ Version 1.1.0
   - enhancement: when ctrl+mouse cursor hovered an identifier or header name, use underline to highlight it

RedPandaIDE/mainwindow.cpp

@@ -4852,7 +4852,13 @@ void MainWindow::on_actionPaste_triggered()
     const QMimeData* data = clipboard->mimeData();
     if (!data)
         return;
-    if (data->hasUrls()) {
+    if (data->hasText()) {
+        Editor * editor = mEditorList->getEditor();
+        if (editor != NULL ) {
+            editor->pasteFromClipboard();
+            editor->activate();
+        }
+    } else if (data->hasUrls()) {
         QStringList filesToOpen;
         foreach (const QUrl& url, data->urls()) {
             QString s = url.toLocalFile();
@@ -4862,12 +4868,6 @@ void MainWindow::on_actionPaste_triggered()
         }
         if (!filesToOpen.isEmpty())
             openFiles(filesToOpen);
-    } else {
-        Editor * editor = mEditorList->getEditor();
-        if (editor != NULL ) {
-            editor->pasteFromClipboard();
-            editor->activate();
-        }
     }
 }
 

RedPandaIDE/qsynedit/SynEdit.cpp

@@ -38,6 +38,8 @@
 #include <QDrag>
 #include <QMimeData>
 #include <QDesktopWidget>
+#include <QTextEdit>
+#include <QMimeData>
 
 SynEdit::SynEdit(QWidget *parent) : QAbstractScrollArea(parent),
     mDropped(false)
@@ -3016,7 +3018,8 @@ void SynEdit::doPasteFromClipboard()
     if (mReadOnly)
         return;
     QClipboard* clipboard = QGuiApplication::clipboard();
-    if (clipboard->text().isEmpty())
+    QString text = clipboard->text();
+    if (text.isEmpty())
         return;
     doOnPaintTransient(SynTransientType::ttBefore);
     mUndoList->BeginBlock();
@@ -3031,46 +3034,44 @@ void SynEdit::doPasteFromClipboard()
                       SynSelectionMode::smNormal);
         mUndoList->EndBlock();
     });
-    if (!clipboard->text().isEmpty()) {
+    mUndoList->AddChange(
+                SynChangeReason::crPasteBegin,
+                blockBegin(),
+                blockEnd(),
+                "",
+                SynSelectionMode::smNormal);
+    AddPasteEndMarker = true;
+    if (selAvail()) {
         mUndoList->AddChange(
-                    SynChangeReason::crPasteBegin,
+                    SynChangeReason::crDelete,
-                    blockBegin(),
+                    mBlockBegin,
-                    blockEnd(),
+                    mBlockEnd,
-                    "",
+                    selText(),
-                    SynSelectionMode::smNormal);
+                    mActiveSelectionMode);
-        AddPasteEndMarker = true;
+    }
-        if (selAvail()) {
-            mUndoList->AddChange(
-                        SynChangeReason::crDelete,
-                        mBlockBegin,
-                        mBlockEnd,
-                        selText(),
-                        mActiveSelectionMode);
-        }
 //        } else if (!colSelAvail())
 //            setActiveSelectionMode(selectionMode());
-        BufferCoord vStartOfBlock = blockBegin();
+    BufferCoord vStartOfBlock = blockBegin();
-        BufferCoord vEndOfBlock = blockEnd();
+    BufferCoord vEndOfBlock = blockEnd();
-        mBlockBegin = vStartOfBlock;
+    mBlockBegin = vStartOfBlock;
-        mBlockEnd = vEndOfBlock;
+    mBlockEnd = vEndOfBlock;
-        setSelTextPrimitive(clipboard->text());
+    setSelTextPrimitive(text);
-        if (mActiveSelectionMode == SynSelectionMode::smColumn) {
+    if (mActiveSelectionMode == SynSelectionMode::smColumn) {
-            mUndoList->AddChange(
+        mUndoList->AddChange(
-                            SynChangeReason::crPaste,
+                        SynChangeReason::crPaste,
-                            blockBegin(),
+                        blockBegin(),
-                            blockEnd(),
+                        blockEnd(),
-                            selText(),
+                        selText(),
-                            mActiveSelectionMode);
+                        mActiveSelectionMode);
-        } else {
+    } else {
 //            setBlockBegin(vStartOfBlock);
 //            setBlockEnd(caretXY());
-            mUndoList->AddChange(
+        mUndoList->AddChange(
-                            SynChangeReason::crPaste,
+                        SynChangeReason::crPaste,
-                            vStartOfBlock,
+                        vStartOfBlock,
-                            blockEnd(),
+                        blockEnd(),
-                            selText(),
+                        selText(),
-                            mActiveSelectionMode);
+                        mActiveSelectionMode);
-        }
     }
 }