Differential D6192

Set Containments to have focus within the view
ClosedPublic
Actions

Authored by davidedmundson on Jun 12 2017, 1:34 PM.

Details

Reviewers
broulik
Group Reviewers
Plasma
Commits
R242:b0e8ea29fab4: Set Containments to have focus within the view
Summary

Now that applets are their own focus scope (which makes sense) we've
also made containments their own focus scope.

We therefore need to also say the containment FocusScope has the focus
within the view.

Test Plan

Renamed something in folderview
Used cursor nav

Diff Detail

Repository
R242 Plasma Framework (Library)
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
davidedmundson created this revision.Jun 12 2017, 1:34 PM
Restricted Application added projects: Plasma, Frameworks. · View Herald TranscriptJun 12 2017, 1:34 PM
Restricted Application added subscribers: Frameworks, plasma-devel. · View Herald Transcript
broulik accepted this revision.Jun 12 2017, 2:18 PM
This revision is now accepted and ready to land.Jun 12 2017, 2:18 PM
broulik added a comment.Jun 12 2017, 2:19 PM

BUG: 381124

Closed by commit R242:b0e8ea29fab4: Set Containments to have focus within the view (authored by davidedmundson). · Explain WhyJun 12 2017, 2:22 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 15379

View Options

src/plasmaquick/containmentview.cpp

Loading...