Differential D24806

IncidenceEditor: allow double-click to edit an alarm
ClosedPublic
Actions

Authored by winterz on Oct 20 2019, 3:43 PM.

Details

Reviewers
mlaurent
Commits
R78:dd6795cec401: incidencealarm.cpp - allow double-click to edit an alarm
Summary

https://bugs.kde.org/show_bug.cgi?id=412242

User should be able start the alarm editor dialog when double-clicking its description item

Test Plan

tried it.

Diff Detail

Repository
R78 PIM: Incidence Editor
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
winterz created this revision.Oct 20 2019, 3:43 PM
Restricted Application added a project: KDE PIM. · View Herald TranscriptOct 20 2019, 3:43 PM
Restricted Application added a subscriber: kde-pim. · View Herald Transcript
winterz requested review of this revision.Oct 20 2019, 3:43 PM
mlaurent requested changes to this revision.Oct 20 2019, 4:18 PM
mlaurent added inline comments.
src/incidencealarm.cpp
52

connect(mUi->mAlarmList, &QListWidget::itemDoubleClicked, this, &IncidenceAlarm::editCurrentAlarm); no ?

This revision now requires changes to proceed.Oct 20 2019, 4:18 PM
winterz updated this revision to Diff 68388.Oct 20 2019, 7:47 PM
mlaurent accepted this revision.Oct 20 2019, 8:39 PM
This revision is now accepted and ready to land.Oct 20 2019, 8:39 PM
Closed by commit R78:dd6795cec401: incidencealarm.cpp - allow double-click to edit an alarm (authored by winterz). · Explain WhyOct 20 2019, 8:48 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Msrc/incidencealarm.cpp (2 lines)

Diff 68391

View Options

src/incidencealarm.cpp

Loading...