4 files changed, 31 insertions, 12 deletions
diff --git a/src/dialogs/qquickdialog.cpp b/src/dialogs/qquickdialog.cpp
index ebe2c7b01..ab3d9200f 100644
--- a/src/dialogs/qquickdialog.cpp
+++ b/src/dialogs/qquickdialog.cpp
@@ -73,7 +73,9 @@ QT_BEGIN_NAMESPACE
 /*!
     \qmlsignal Dialog::accepted()
 
-    This signal is emitted by \l accept().
+    This signal is emitted when the user has pressed any button which has the
+    \l {QMessageBox::}{AcceptRole}: \gui OK, \gui Open, \gui Save,
+    \gui {Save All}, \gui Retry or \gui Ignore.
 
     The corresponding handler is \c onAccepted.
 */
@@ -81,7 +83,9 @@ QT_BEGIN_NAMESPACE
 /*!
     \qmlsignal Dialog::rejected()
 
-    This signal is emitted by \l reject().
+    This signal is emitted when the user has dismissed the dialog, by closing
+    the dialog window, by pressing a \gui Cancel, \gui Close or \gui Abort
+    button on the dialog, or by pressing the back button or the escape key.
 
     The corresponding handler is \c onRejected.
 */
@@ -152,6 +156,12 @@ QT_BEGIN_NAMESPACE
     \sa modality
 */
 
+/*! \qmlproperty StandardButton Dialog::clickedButton
+
+    This property holds the button pressed by the user. Its value is
+    one of the flags set for the standardButtons property.
+*/
+
 /*!
     \qmlproperty Qt::WindowModality Dialog::modality
 
@@ -294,12 +304,6 @@ void QQuickDialog::setStandardButtons(StandardButtons buttons)
 }
 
 /*!
-    \qmlproperty bool Dialog::visible
-
-    This property holds whether the dialog is visible. By default this is false.
-*/
-
-/*!
     \qmlproperty QObject Dialog::contentItem
 
     The QML object which implements the dialog contents. Should be an \l Item.
diff --git a/src/dialogs/qquickplatformcolordialog.cpp b/src/dialogs/qquickplatformcolordialog.cpp
index f62833119..f48c4ab75 100644
--- a/src/dialogs/qquickplatformcolordialog.cpp
+++ b/src/dialogs/qquickplatformcolordialog.cpp
@@ -185,13 +185,18 @@ QPlatformColorDialogHelper *QQuickPlatformColorDialog::helper()
     containing the dialog's parent Item, modal with respect to the whole
     application, or non-modal.
 
-    By default it is \l NonModal.
+    By default it is \c Qt.NonModal.
 
     Modality does not mean that there are any blocking calls to wait for the
     dialog to be accepted or rejected; it's only that the user will be
     prevented from interacting with the parent window and/or the application
     windows at the same time.
 
+    You probably need to write an onAccepted handler if you wish to change a
+    color after the user has pressed the OK button, or an
+    onCurrentColorChanged handler if you wish to react to every change the
+    user makes while the dialog is open.
+
     On MacOS the color dialog is only allowed to be non-modal.
 */
 
diff --git a/src/dialogs/qquickplatformfontdialog.cpp b/src/dialogs/qquickplatformfontdialog.cpp
index faf873f4e..1744cb846 100644
--- a/src/dialogs/qquickplatformfontdialog.cpp
+++ b/src/dialogs/qquickplatformfontdialog.cpp
@@ -187,13 +187,17 @@ QPlatformFontDialogHelper *QQuickPlatformFontDialog::helper()
     containing the dialog's parent Item, modal with respect to the whole
     application, or non-modal.
 
-    By default it is \l WindowModal.
+    By default it is \c Qt.WindowModal.
 
     Modality does not mean that there are any blocking calls to wait for the
     dialog to be accepted or rejected; it's only that the user will be
     prevented from interacting with the parent window and/or the application
-    windows at the same time. You probably need to write an onAccepted handler
-    to actually load or save the chosen file.
+    windows at the same time.
+
+    You probably need to write an onAccepted handler if you wish to change a
+    font after the user has pressed the OK button, or an onCurrentFontChanged
+    handler if you wish to react to every change the user makes while the
+    dialog is open.
 */
 
 /*!
diff --git a/src/dialogs/qquickplatformmessagedialog.cpp b/src/dialogs/qquickplatformmessagedialog.cpp
index 07665fffe..c7b13c7ce 100644
--- a/src/dialogs/qquickplatformmessagedialog.cpp
+++ b/src/dialogs/qquickplatformmessagedialog.cpp
@@ -171,6 +171,12 @@ QT_BEGIN_NAMESPACE
     The corresponding handler is \c onReset.
 */
 
+/*! \qmlproperty StandardButton MessageDialog::clickedButton
+
+    This property holds the button pressed by the user. Its value is
+    one of the flags set for the standardButtons property.
+*/
+
 /*!
     \class QQuickPlatformMessageDialog
     \inmodule QtQuick.Dialogs