[Lock Screen] When pressing Return just trigger button
AbandonedPublic

Authored by broulik on Oct 6 2016, 8:20 AM.

Details

Reviewers
None
Group Reviewers
Plasma
Summary

When pressing Return on the "New Session" screen, it would still try to unlock the session as the text field still has focus.
Instead of duplicating some logic there, just "click" the button which will do the right thing in both cases.

Test Plan

Locked screen, typed password, pressed Return, session unlocked.
Locked screen, choose "New Session", pressed Return, new session would be created (previously it would try to unlock, or do nothing if you didn't type anything before)

Diff Detail

Repository
R120 Plasma Workspace
Lint
Lint Skipped
Unit
Unit Tests Skipped
broulik updated this revision to Diff 7134.Oct 6 2016, 8:20 AM
broulik retitled this revision from to [Lock Screen] When pressing Return just trigger button.
broulik updated this object.
broulik edited the test plan for this revision. (Show Details)
broulik added a reviewer: Plasma.
broulik set the repository for this revision to R120 Plasma Workspace.
Restricted Application added a project: Plasma. · View Herald TranscriptOct 6 2016, 8:20 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
mart added a subscriber: mart.Oct 6 2016, 9:08 AM

hmm is a bit dangerous? should the handler of button clicked be a function callable also from there instead?

broulik abandoned this revision.Oct 10 2016, 8:15 PM

Superseded by D2976