Differential D7368

[KACLEditWidget] Allow double clicking to edit entry
ClosedPublic
Actions

Authored by broulik on Aug 17 2017, 2:56 PM.

Details

Reviewers
dfaure
Commits
R241:1d480054428d: [KACLEditWidget] Allow double clicking to edit entry
Summary

This is less cumbersome than having to select an item and then clicking "Edit Entry"

Test Plan
  • Verified that double-clicking one of the built-in ones (Owner, Group, Mask) does not yield the dialog
  • Verified that double-clicking one of the check marks does not yield the dialog to avoid conflicting interaction

Diff Detail

Repository
R241 KIO
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
broulik created this revision.Aug 17 2017, 2:56 PM
Restricted Application added a project: Frameworks. · View Herald TranscriptAug 17 2017, 2:56 PM
Restricted Application added a subscriber: Frameworks. · View Herald Transcript
dfaure accepted this revision.Aug 23 2017, 8:31 PM
dfaure added inline comments.
src/widgets/kacleditwidget.cpp
951

typo: permission

This revision is now accepted and ready to land.Aug 23 2017, 8:31 PM
Closed by commit R241:1d480054428d: [KACLEditWidget] Allow double clicking to edit entry (authored by broulik). · Explain WhyAug 24 2017, 8:26 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 18654

View Options

src/widgets/kacleditwidget.cpp

Loading...
View Options

src/widgets/kacleditwidget_p.h

Loading...