Should solve #5908 Needs testing (especially in wallet disabled mode).
[QT] debug window "rpcconsole.ui" should have Qt::Window flag #6155
pull jonasschnelli wants to merge 1 commits into bitcoin:master from jonasschnelli:2015/05/qt_console_window_close changing 1 files +3 −0-
jonasschnelli commented at 3:48 PM on May 18, 2015: contributor
-
laanwj commented at 5:00 PM on May 18, 2015: member
It should not be a dialog, just a window. A dialog implies a modal user interaction flow, whereas the debug window is just an auxiliary window.
- jonasschnelli force-pushed on May 19, 2015
-
[QT] debug window "rpcconsole.ui" should have window flag to make sure a close button is present 5fa5d2441b
- jonasschnelli force-pushed on May 19, 2015
- jonasschnelli renamed this:
[QT] debug window "rpcconsole.ui" should be inherited from QDialog
[QT] debug window "rpcconsole.ui" should have Qt::Window flag
on May 19, 2015 -
jonasschnelli commented at 11:57 AM on May 19, 2015: contributor
@laanwj: QWindow is actually a non QWidget subclass and would require some adaptation to make it work as it is now.
Now i went for the most easy solution: adding a
Qt::Windowflag for the widget. This should be sufficient to make the debug window act as a normal window. -
laanwj commented at 2:51 PM on May 19, 2015: member
Right. That's fine.
From http://doc.qt.io/qt-5/qwindow.html : "An application will typically use QWidget or QQuickView for its UI, and not QWindow directly"
QWidget is the correct type to use.
-
jonasschnelli commented at 2:53 PM on May 19, 2015: contributor
But i'm not sure if this is even required. I can't reproduce #5908. But adding the
Qt::Windowflag for the console widget should anyway be the right thing. -
laanwj commented at 2:55 PM on May 19, 2015: member
It should get added automatically when there is no parent widget. Forcing the flag on will likely give problems when the debug console is embedded in the main window (as when running in Node mode).
-
jonasschnelli commented at 2:57 PM on May 19, 2015: contributor
I'm closing this until #5908 turns out to be a real issue.
- jonasschnelli closed this on May 19, 2015
-
laanwj commented at 3:00 PM on May 19, 2015: member
We could add the flag conditionally, I think it can be passed to the constructor. But I agree we should first be sure it's the issue.
-
jonasschnelli commented at 7:05 AM on May 20, 2015: contributor
Regarding #5908, this is probably a Qt4.x "bug" (or inconsistency). I think the PR could make sense for users/package-maintainer who build/ship bitcoin-qt with Qt4.X. But it first needs testing over a Qt4 build.
- MarcoFalke locked this on Sep 8, 2021