Uploaded image for project: 'Qt'
  1. Qt
  2. QTBUG-6018

QDialog as a delegate woes

    XMLWordPrintable

Details

    • Bug
    • Resolution: Done
    • P4: Low
    • 5.9.0 Beta 2
    • 4.5.3
    • Widgets: Itemviews
    • None
    • All
    • 8b1d9d308b1f31e1b9c817e0d91e14baa3b76d6b

    Description

      Using QDialog as a delegate is problematic because QDialog::done(int) calls hide() before setResult(), therefore the result cannot be inspected by the Delegate's setModelData() method.

      QDialog::done(int) should invoke setResult() before calling hide().

      Attachments

        Issue Links

          is duplicated by

          Bug - A problem which impairs or prevents the functions of the product. QTBUG-12156 CLONE -QDialog as a delegate woes

          • P4: Low - Would be nice to fix, but it's not very important.
          • Closed

          Bug - A problem which impairs or prevents the functions of the product. QTBUG-14430 CLONE - QDialog as a delegate woes

          • P4: Low - Would be nice to fix, but it's not very important.
          • Closed
          relates to

          Bug - A problem which impairs or prevents the functions of the product. QTBUG-118226 Rejected QMessageBox emits Accepted Signal

          • P1: Critical - Urgent and Important, will STOP the release if matched with set FixVersion field. This includes regressions from the last version that are not edge cases; Data loss; Build issues; Flaky tests; All but the most unlikely crashes/lockups/hanging; Serious usability issues and embarrassing bugs & Bugs critical to a deliverable. This is the highest possible priority for requirements.
          • Closed
          For Gerrit Dashboard: QTBUG-6018
          # Subject Branch Project Status CR V
          147871,6 Fix result handling in QDialog::done 5.8 qt/qtbase Status: MERGED +2 0

          Activity

            People

              sgaist Samuel Gaist
              fbolduc Francis Bolduc
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Gerrit Reviews

                  There are no open Gerrit changes