Author Topic: Dooble releases-Qt6  (Read 332978 times)

Dave Yeo

  • Hero Member
  • *****
  • Posts: 5865
  • Karma: +155/-1
Re: Dooble releases-Qt6
« Reply #240 on: May 02, 2026, 05:55:11 am »
Here's the commit that changed the behaviour,
Code: [Select]
From 8504587e4ed3d5c137ab5b8e148d79f0944aec03 Mon Sep 17 00:00:00 2001
From: textbrowser <textbrowser@gmail.com>
Date: Thu, 12 Feb 2026 08:37:19 -0500
Subject: [PATCH] New item.

---
 Documentation/ReleaseNotes.html |  1 +
 Source/dooble_main.cc           | 15 +++++++--------
 2 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/Documentation/ReleaseNotes.html b/Documentation/ReleaseNotes.html
index e6565814..293d5da7 100644
--- a/Documentation/ReleaseNotes.html
+++ b/Documentation/ReleaseNotes.html
@@ -34,6 +34,7 @@
  <h3><b>Dooble 2026.02.15</b></h3>
  Welcome to the <b>2026.02.15</b> release of Dooble. Corrections and new features are detailed below.
  <ul>
+   <li>Interpret command-line arguments as local files first and then as remote locations. See 2025.11.25 item.</li>
    <li>New Settings -> Web -> Local Content Can Access File URLs.</li>
    <li>Sleepy-deepy timers on panel closings.</li>
  </ul>
diff --git a/Source/dooble_main.cc b/Source/dooble_main.cc
index 350de004..9decaeb6 100644
--- a/Source/dooble_main.cc
+++ b/Source/dooble_main.cc
@@ -185,17 +185,16 @@ int main(int argc, char *argv[])
    i += 1;
  else
    {
-     auto url(QUrl::fromUserInput(argv[i]));
+     QFileInfo const file_info(argv[i]);
+     QUrl url;
 
-     if(url.isValid() == false)
+     if(file_info.isReadable())
+       url = QUrl::fromUserInput(file_info.absoluteFilePath());
+     else
        {
- QFileInfo const file_info(argv[i]);
-
- if(file_info.isReadable())
-   url = QUrl::fromUserInput(file_info.absoluteFilePath());
+ url = QUrl::fromUserInput(argv[i]);
+ url.setScheme("https");
        }
-     else if(url.scheme() == "http")
-       url.setScheme("https");
 
      if(dooble_ui_utilities::allowed_url_scheme(url))
        urls << url;
--
2.50.1

I think that the problem has been user error, namely passing a bad parameter to Dooble which Dooble tries to display as an URL after displaying as a file fails.
So if you want to disable-gpu, use the environment, same with other Chromium flags

Rich Walsh

  • Sr. Member
  • ****
  • Posts: 427
  • Karma: +30/-0
  • ONU! (OS/2 is NOT Unix!)
Re: Dooble releases-Qt6
« Reply #241 on: May 02, 2026, 08:19:23 am »
If I remember previous discussion about the Del key correctly it is a bug in our port of qt.

Not quite... Delete works fine in Qt proper (e.g. try it in the address bar). Where it doesn't work is in WebEngine which has its own keyboard handler for the text fields it creates (i.e. in forms).

Paul Smedley

  • Hero Member
  • *****
  • Posts: 2561
  • Karma: +207/-0
Re: Dooble releases-Qt6
« Reply #242 on: May 02, 2026, 11:26:34 am »
If I remember previous discussion about the Del key correctly it is a bug in our port of qt.

Not quite... Delete works fine in Qt proper (e.g. try it in the address bar). Where it doesn't work is in WebEngine which has its own keyboard handler for the text fields it creates (i.e. in forms).

Possibly/probably due to stubbed code in https://github.com/psmedley/qt6-webengine-os2/blob/main/src/3rdparty/chromium/ui/events/os2/keyboard_hook_os2.cc

Windows code is in https://github.com/psmedley/qt6-webengine-os2/blob/main/src/3rdparty/chromium/ui/events/win/keyboard_hook_win_base.cc and probs other files in https://github.com/psmedley/qt6-webengine-os2/blob/main/src/3rdparty/chromium/ui/events/win