For consistency with the generic version and because it makes more sense than
blocking in the dialog destructor (which is the only other alternative for the
dialogs without wxPD_AUTO_HIDE to work), we should block in Update() when the
maximal value is reached for dialogs without wxPD_AUTO_HIDE style until the
dialog is dismissed.
Document this behaviour even more clearly.
git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@65507
c3d73ce0-8a6f-49c7-b76d-
6d57e0e08775
Uncancelable = -1, // dialog can't be canceled
Canceled, // can be cancelled and, in fact, was
Continue, // can be cancelled but wasn't
Uncancelable = -1, // dialog can't be canceled
Canceled, // can be cancelled and, in fact, was
Continue, // can be cancelled but wasn't
- Finished // finished, waiting to be removed from screen
+ Finished, // finished, waiting to be removed from screen
+ Dismissed // was closed by user after finishing
Causes the progress dialog to disappear from screen as soon as the
maximum value of the progress meter has been reached.
If this style is not present, the dialog will become a modal dialog
Causes the progress dialog to disappear from screen as soon as the
maximum value of the progress meter has been reached.
If this style is not present, the dialog will become a modal dialog
- (see wxDialog::ShowModal) once the maximum value has been reached;
- this results in processing of pending events and may cause
- unwanted re-entrancies.
+ (see wxDialog::ShowModal) once the maximum value has been reached
+ and wait for the user to dismiss it.
@style{wxPD_SMOOTH}
Causes smooth progress of the gauge control (uses a wxGauge with the
@c wxGA_SMOOTH style).
@style{wxPD_SMOOTH}
Causes smooth progress of the gauge control (uses a wxGauge with the
@c wxGA_SMOOTH style).
dialog or ask the user for the confirmation and if the abort is not confirmed
the dialog may be resumed with Resume() function.
dialog or ask the user for the confirmation and if the abort is not confirmed
the dialog may be resumed with Resume() function.
+ If @a value is the maximum value for the dialog, the behaviour of the
+ function depends on whether @c wxPD_AUTO_HIDE was used when the dialog
+ was created. If it was, the dialog is hidden and the function returns
+ immediately. If it was not, the dialog becomes a modal dialog and waits
+ for the user to dismiss it, meaning that this function does not return
+ until this happens.
+
Notice that you may want to call Fit() to change the dialog size to
conform to the length of the new message if desired. The dialog does
not do this automatically.
Notice that you may want to call Fit() to change the dialog size to
conform to the length of the new message if desired. The dialog does
not do this automatically.
@param value
The new value of the progress meter. It should be less than or equal to
the maximum value given to the constructor.
@param value
The new value of the progress meter. It should be less than or equal to
the maximum value given to the constructor.
- See @c wxPD_AUTO_HIDE style for more info about the behaviour of
- wxProgressDialog when @a value is the maximum value given in the ctor.
@param newmsg
The new messages for the progress dialog text, if it is
empty (which is the default) the message is not changed.
@param newmsg
The new messages for the progress dialog text, if it is
empty (which is the default) the message is not changed.
#include "wx/msw/private/msgdlg.h"
#include "wx/progdlg.h"
#include "wx/msw/private/msgdlg.h"
#include "wx/progdlg.h"
using namespace wxMSWMessageDialog;
using namespace wxMSWMessageDialog;
+namespace
+{
+
+// A custom event loop which runs until the state of the dialog becomes
+// "Dismissed".
+class wxProgressDialogModalLoop : public wxEventLoop
+{
+public:
+ wxProgressDialogModalLoop(wxProgressDialogSharedData& data)
+ : m_data(data)
+ {
+ }
+
+protected:
+ virtual void OnNextIteration()
+ {
+ wxCriticalSectionLocker locker(m_data.m_cs);
+
+ if ( m_data.m_state == wxProgressDialog::Dismissed )
+ Exit();
+ }
+
+ wxProgressDialogSharedData& m_data;
+
+ wxDECLARE_NO_COPY_CLASS(wxProgressDialogModalLoop);
+};
+
// ============================================================================
// Helper functions
// ============================================================================
// ============================================================================
// Helper functions
// ============================================================================
// This function returns true if the progress dialog with the given style
// (combination of wxPD_XXX constants) needs the "Close" button and this button
// only, i.e. not a "Cancel" one.
// This function returns true if the progress dialog with the given style
// (combination of wxPD_XXX constants) needs the "Close" button and this button
// only, i.e. not a "Cancel" one.
#ifdef wxHAS_MSW_TASKDIALOG
if ( HasNativeProgressDialog() )
{
#ifdef wxHAS_MSW_TASKDIALOG
if ( HasNativeProgressDialog() )
{
- wxCriticalSectionLocker locker(m_sharedData->m_cs);
+ {
+ wxCriticalSectionLocker locker(m_sharedData->m_cs);
- // Do nothing in canceled state.
- if ( !DoNativeBeforeUpdate(skip) )
- return false;
+ // Do nothing in canceled state.
+ if ( !DoNativeBeforeUpdate(skip) )
+ return false;
- wxASSERT_MSG( value <= m_maximum, wxT("invalid progress value") );
+ wxASSERT_MSG( value <= m_maximum, wxT("invalid progress value") );
- m_sharedData->m_value = value;
- m_sharedData->m_notifications |= wxSPDD_VALUE_CHANGED;
+ m_sharedData->m_value = value;
+ m_sharedData->m_notifications |= wxSPDD_VALUE_CHANGED;
- if ( !newmsg.empty() )
- {
- m_message = newmsg;
- m_sharedData->m_message = newmsg;
- m_sharedData->m_notifications |= wxSPDD_MESSAGE_CHANGED;
- }
+ if ( !newmsg.empty() )
+ {
+ m_message = newmsg;
+ m_sharedData->m_message = newmsg;
+ m_sharedData->m_notifications |= wxSPDD_MESSAGE_CHANGED;
+ }
- if ( m_sharedData->m_progressBarMarquee )
- {
- m_sharedData->m_progressBarMarquee = false;
- m_sharedData->m_notifications |= wxSPDD_PBMARQUEE_CHANGED;
- }
+ if ( m_sharedData->m_progressBarMarquee )
+ {
+ m_sharedData->m_progressBarMarquee = false;
+ m_sharedData->m_notifications |= wxSPDD_PBMARQUEE_CHANGED;
+ }
- UpdateExpandedInformation( value );
+ UpdateExpandedInformation( value );
+
+ // If we didn't just reach the finish, all we have to do is to
+ // return true if the dialog wasn't cancelled and false otherwise.
+ if ( value != m_maximum || m_state == Finished )
+ return m_sharedData->m_state != Canceled;
- // Has the progress bar finished?
- if ( value == m_maximum )
- {
- if ( m_state == Finished )
- return true;
+ // On finishing, the dialog without wxPD_AUTO_HIDE style becomes a
+ // modal one meaning that we must block here until the user
+ // dismisses it.
m_state = Finished;
m_sharedData->m_state = Finished;
m_sharedData->m_notifications |= wxSPDD_FINISHED;
m_state = Finished;
m_sharedData->m_state = Finished;
m_sharedData->m_notifications |= wxSPDD_FINISHED;
- if( !HasPDFlag(wxPD_AUTO_HIDE) && newmsg.empty() )
+ if ( !HasPDFlag(wxPD_AUTO_HIDE) && newmsg.empty() )
{
// Provide the finishing message if the application didn't.
m_message = _("Done.");
m_sharedData->m_message = m_message;
m_sharedData->m_notifications |= wxSPDD_MESSAGE_CHANGED;
}
{
// Provide the finishing message if the application didn't.
m_message = _("Done.");
m_sharedData->m_message = m_message;
m_sharedData->m_notifications |= wxSPDD_MESSAGE_CHANGED;
}
- }
-
- return m_sharedData->m_state != Canceled;
+ } // unlock m_sharedData->m_cs
+
+ // We only get here when we need to wait for the dialog to terminate so
+ // do just this by running a custom event loop until the dialog is
+ // dismissed.
+ wxProgressDialogModalLoop loop(*m_sharedData);
+ loop.Run();
+ return true;
}
#endif // wxHAS_MSW_TASKDIALOG
}
#endif // wxHAS_MSW_TASKDIALOG
if ( FAILED(hr) )
wxLogApiError( "TaskDialogIndirect", hr );
if ( FAILED(hr) )
wxLogApiError( "TaskDialogIndirect", hr );
+ // If the main thread is waiting for us to exit inside the event loop in
+ // Update(), wake it up so that it checks our status again.
+ wxWakeUpIdle();
+
case IDCANCEL:
if ( sharedData->m_state == wxProgressDialog::Finished )
{
case IDCANCEL:
if ( sharedData->m_state == wxProgressDialog::Finished )
{
+ // If the main thread is waiting for us, tell it that
+ // we're gone (and if it doesn't wait, it's harmless).
+ sharedData->m_state = wxProgressDialog::Dismissed;
+
+ // Let Windows close the dialog.