El diumenge, 8 de juny del 2025, a les 15:01:06 (Hora d’estiu d’Europa central), Stefano Crocco va escriure: > Hello to everyone, > I was investigating why in Konqueror middle-clicking on a bookmark in the > bookmarks menu doesn't open the bookmark in a new tab like it used to do a > while ago
Do you know when it broke more or less? Looking at the code it seems it has been broken since KF 5? Cheers, Albert > but opens it in the current tab and I found something which could > be a bug in KBookmarks, a bug in Qt or a bug in Konqueror itself. > > What Konqueror does is subclassing KBookmarkOwner and reimplementing > KBookmarkOwner::openBookmark(). In this function, it checks the contents of > the Qt::MouseButtons argument to determine if the action was triggered by a > left click or middle click, then decide whether to open the bookmark in the > current tab or a new tab. The condition is: > > if (mb & Qt::MiddleButton) { > ... > } else { > ... > } > > where mb is the Qt::MouseButtons argument. > > The problem is that, regardless of whether I left-click or middle-click on a > bookmark in the bookmark men, mb is always Qt::NoButton, which triggers the > "else" clause, making the bookmark always open in the current tab. This > seems to happen because QGuiApplication::mouseButtons() when called by a > slot connected to the triggered() signal of an action in a menu returns > Qt::NoButton. > > I was going to create a bug report for KBookmarks, but then I thought that > maybe the Konqueror code (which wasn't written by me) is using the > Qt::MouseButton argument in a wrong way. I've found a work around for this > issue (see [1]) but I still like to create a bug report for it, if it's > indeed a bug. Could you please give me any hint about how to proceed? Is > this likely to be a bug in QGuiApplication, a bug in KBookmarks or a bug in > Konqueror? > > Thanks in advance > > Stefano > > [1]: https://invent.kde.org/network/konqueror/-/merge_requests/426