Bug 233306

Summary: Display resolution does not revert to original size after dialog times out
Product: [Applications] systemsettings Reporter: Scott Kitterman <kde>
Component: generalAssignee: System Settings Bugs <sourtooth+ssbugs>
Status: RESOLVED DUPLICATE    
Severity: normal CC: andresbajotierra
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: openSUSE   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:

Description Scott Kitterman 2010-04-05 00:48:56 UTC
Version:            (using KDE 4.4.2)
OS:                Linux
Installed from:    openSUSE RPMs

At ~kdebase-workspace-4.4.2/kcontrol/randr/cpp +122 it says:
        KTimerDialog acceptDialog(15000, KTimerDialog::CountDown,
                                   0, "mainKTimerDialog", true,
                                   i18n("Confirm Display Setting Change"),
                                   KTimerDialog::Ok|KTimerDialog::Cancel,
                                   KTimerDialog::Cancel);
At ~kdebase-workspace-4.4.2/kcontrol/randr/ktimerdialog.h +67 it says:
    /**
      * Constructor for the standard mode where you must specify the main
      * widget with @ref setMainWidget() . See @see KDialog for further details.
      *
      * For the rest of the arguments, See @see KDialog .
      */
     explicit KTimerDialog( int msec, TimerStyle style=CountDown, QWidget *parent=0,
                            const char *name=0, bool modal=true,
                            const QString &caption=QString(),
                            int buttonMask=Ok|Apply|Cancel, ButtonCode defaultButton=Ok,
                            bool separator=false,
                            const KGuiItem &user1=KGuiItem(),
                            const KGuiItem &user2=KGuiItem(),
                            const KGuiItem &user3=KGuiItem() );
So that looks correct (it also confirms that cancel is the desired default). My guess is KTimerDialog is designed to default to cancel and pass this through to KDialog (which defaults to accept), but that something is getting lost in the transition or that the revert is requested, but fails for some reason. 

This works in the krandrtray systray application, but it's code is completely different.
Comment 1 Scott Kitterman 2010-04-05 00:53:13 UTC
4.3.5 has the same issue (Also Ubuntu packages).
Comment 2 Dario Andres 2010-04-05 01:59:56 UTC
This is being tracked at bug 222110. 
Regards

*** This bug has been marked as a duplicate of bug 222110 ***