plusui-native-core 0.1.42 → 0.1.44

package/Core/CMakeLists.txt

@@ -24,6 +24,7 @@ add_library(plusui STATIC
     Features/Bindings/CustomBindings/custom_bindings.cpp
     Features/App/app.cpp
     Features/Window/window.cpp
+    Features/FileDrop/filedrop.cpp
     Features/Browser/browser.cpp
     Features/Display/display.cpp
     Features/Tray/tray.cpp

package/Core/Features/Window/window.cpp

@@ -767,6 +767,12 @@ Window Window::create(void *windowHandle, const WindowConfig &config) {
   Window win;
   win.pImpl->config = config;
 
+  // Pure native file-drop mode: when native FileDrop is enabled,
+  // fully disable browser/WebView drag-drop handling.
+  if (win.pImpl->config.enableFileDrop) {
+    win.pImpl->config.disableWebviewDragDrop = true;
+  }
+
 #ifdef _WIN32
   HWND hwnd = static_cast<HWND>(windowHandle);
 
@@ -829,24 +835,31 @@ Window Window::create(void *windowHandle, const WindowConfig &config) {
                         // and allows the native FileDrop API to handle them instead
                         if (pImpl->config.disableWebviewDragDrop) {
                           std::string disableDragDropScript = R"(
-                              (function() {
-                                  // Prevent default drag and drop behavior on document/window
-                                  ['dragenter', 'dragover', 'dragleave', 'drop'].forEach(function(eventName) {
-                                      document.addEventListener(eventName, function(e) {
-                                          // Only prevent default on document.body or document.documentElement
-                                          // to allow custom drop zones to still work
-                                          if (e.target === document.body || e.target === document.documentElement || e.target === document) {
-                                              e.preventDefault();
-                                              e.stopPropagation();
-                                          }
-                                      }, true);
-
-                                      window.addEventListener(eventName, function(e) {
-                                          e.preventDefault();
-                                          e.stopPropagation();
-                                      }, true);
-                                  });
-                              })();
+                            (function() {
+                              if (window.__plusui_nativeFileDropOnly) return;
+                              window.__plusui_nativeFileDropOnly = true;
+
+                              var block = function(e) {
+                                if (!e) return false;
+                                e.preventDefault();
+                                e.stopPropagation();
+                                if (typeof e.stopImmediatePropagation === 'function') {
+                                  e.stopImmediatePropagation();
+                                }
+                                if (e.dataTransfer) {
+                                  try { e.dataTransfer.dropEffect = 'none'; } catch (_) {}
+                                }
+                                return false;
+                              };
+
+                              window.__plusui_dragDropEvents = ['dragenter', 'dragover', 'dragleave', 'drop'];
+                              window.__plusui_dragDropBlocker = block;
+
+                              window.__plusui_dragDropEvents.forEach(function(eventName) {
+                                window.addEventListener(eventName, block, true);
+                                document.addEventListener(eventName, block, true);
+                              });
+                            })();
                           )";
                           pImpl->webview->AddScriptToExecuteOnDocumentCreated(
                               std::wstring(disableDragDropScript.begin(),
@@ -1175,17 +1188,21 @@ Window Window::create(void *windowHandle, const WindowConfig &config) {
   // Disable webview drag & drop behavior to allow native FileDrop API
   if (win.pImpl->config.disableWebviewDragDrop) {
     NSString *disableDragDropScript = @"(function() {"
-      "['dragenter', 'dragover', 'dragleave', 'drop'].forEach(function(eventName) {"
-        "document.addEventListener(eventName, function(e) {"
-          "if (e.target === document.body || e.target === document.documentElement || e.target === document) {"
-            "e.preventDefault();"
-            "e.stopPropagation();"
-          "}"
-        "}, true);"
-        "window.addEventListener(eventName, function(e) {"
-          "e.preventDefault();"
-          "e.stopPropagation();"
-        "}, true);"
+      "if (window.__plusui_nativeFileDropOnly) return;"
+      "window.__plusui_nativeFileDropOnly = true;"
+      "var block = function(e) {"
+        "if (!e) return false;"
+        "e.preventDefault();"
+        "e.stopPropagation();"
+        "if (typeof e.stopImmediatePropagation === 'function') e.stopImmediatePropagation();"
+        "if (e.dataTransfer) { try { e.dataTransfer.dropEffect = 'none'; } catch (_) {} }"
+        "return false;"
+      "};"
+      "window.__plusui_dragDropEvents = ['dragenter','dragover','dragleave','drop'];"
+      "window.__plusui_dragDropBlocker = block;"
+      "window.__plusui_dragDropEvents.forEach(function(eventName) {"
+        "window.addEventListener(eventName, block, true);"
+        "document.addEventListener(eventName, block, true);"
       "});"
     "})();";
 
@@ -1299,29 +1316,6 @@ void Window::navigate(const std::string &url) {
     webkit_web_view_load_uri(pImpl->gtkWebView, url.c_str());
   }
 #endif
-
-  // Apply scrollbar hiding CSS if configured
-  if (!pImpl->config.scrollbars) {
-    std::string hideScrollbars = kHideScrollbarsScript;
-    // Schedule this to run after a short delay to ensure DOM is ready
-#ifdef _WIN32
-    if (pImpl->ready && pImpl->webview) {
-      pImpl->webview->ExecuteScript(
-          std::wstring(hideScrollbars.begin(), hideScrollbars.end()).c_str(),
-          nullptr);
-    }
-#elif defined(__APPLE__)
-    if (pImpl->wkWebView) {
-      NSString *js = [NSString stringWithUTF8String:hideScrollbars.c_str()];
-      [pImpl->wkWebView evaluateJavaScript:js completionHandler:nil];
-    }
-#else
-    if (pImpl->gtkWebView) {
-      webkit_web_view_run_javascript(pImpl->gtkWebView, hideScrollbars.c_str(),
-                                     nullptr, nullptr, nullptr);
-    }
-#endif
-  }
 }
 
 void Window::loadURL(const std::string &url) {
@@ -1654,30 +1648,47 @@ void Window::setWebviewDragDropEnabled(bool enabled) {
     // Enable webview drag-drop by removing our prevention handlers
     script = R"(
       (function() {
-        if (window.__plusui_dragDropDisabled) {
-          delete window.__plusui_dragDropDisabled;
+        var events = window.__plusui_dragDropEvents || ['dragenter', 'dragover', 'dragleave', 'drop'];
+        var blocker = window.__plusui_dragDropBlocker;
+        if (blocker) {
+          events.forEach(function(eventName) {
+            window.removeEventListener(eventName, blocker, true);
+            document.removeEventListener(eventName, blocker, true);
+          });
         }
+        delete window.__plusui_dragDropBlocker;
+        delete window.__plusui_dragDropEvents;
+        delete window.__plusui_dragDropDisabled;
+        delete window.__plusui_nativeFileDropOnly;
       })();
     )";
   } else {
     // Disable webview drag-drop by preventing default behavior
     script = R"(
       (function() {
-        if (window.__plusui_dragDropDisabled) return; // Already disabled
+        if (window.__plusui_nativeFileDropOnly) return;
+        window.__plusui_nativeFileDropOnly = true;
         window.__plusui_dragDropDisabled = true;
 
-        ['dragenter', 'dragover', 'dragleave', 'drop'].forEach(function(eventName) {
-          document.addEventListener(eventName, function(e) {
-            if (e.target === document.body || e.target === document.documentElement || e.target === document) {
-              e.preventDefault();
-              e.stopPropagation();
-            }
-          }, true);
+        var block = function(e) {
+          if (!e) return false;
+          e.preventDefault();
+          e.stopPropagation();
+          if (typeof e.stopImmediatePropagation === 'function') {
+            e.stopImmediatePropagation();
+          }
+          if (e.dataTransfer) {
+            try { e.dataTransfer.dropEffect = 'none'; } catch (_) {}
+          }
+          return false;
+        };
+
+        window.__plusui_dragDropEvents = ['dragenter', 'dragover', 'dragleave', 'drop'];
+        window.__plusui_dragDropBlocker = block;
 
-          window.addEventListener(eventName, function(e) {
-            e.preventDefault();
-            e.stopPropagation();
-          }, true);
+        window.__plusui_dragDropEvents.forEach(function(eventName) {
+          window.addEventListener(eventName, block, true);
+          document.addEventListener(eventName, block, true);
         });
       })();
     )";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "plusui-native-core",
-  "version": "0.1.42",
+  "version": "0.1.44",
   "description": "PlusUI Core framework (frontend + backend implementations)",
   "type": "module",
   "files": [