[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
  1. jonasschnelli commented at 3:48 PM on May 18, 2015: contributor

    Should solve #5908 Needs testing (especially in wallet disabled mode).

  2. 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.

  3. jonasschnelli force-pushed on May 19, 2015
  4. [QT] debug window "rpcconsole.ui" should have window flag to make sure a close button is present 5fa5d2441b
  5. jonasschnelli force-pushed on May 19, 2015
  6. 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
  7. 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::Window flag for the widget. This should be sufficient to make the debug window act as a normal window.

  8. 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.

  9. 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::Window flag for the console widget should anyway be the right thing.

  10. 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).

  11. jonasschnelli commented at 2:57 PM on May 19, 2015: contributor

    I'm closing this until #5908 turns out to be a real issue.

  12. jonasschnelli closed this on May 19, 2015

  13. 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.

  14. 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.

  15. MarcoFalke locked this on Sep 8, 2021

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-04-19 00:15 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me