[Bug 554948] Re: Display settings change defaults to keep, not revert

[Scott Kitterman]

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.  This needs someone who
does KDE/C++ programming to fix.

** Changed in: kdebase-workspace (Ubuntu Lucid)
       Status: In Progress => Triaged

** Changed in: kdebase-workspace (Ubuntu Lucid)
     Assignee: Scott Kitterman (kitterman) => (unassigned)

-- 
Display settings change defaults to keep, not revert
https://bugs.launchpad.net/bugs/554948
You received this bug notification because you are a member of Kubuntu
Bugs, which is subscribed to kdebase-workspace in ubuntu.