Skip to content

GitLab

  • Menu
Projects Groups Snippets
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • P Plasma Workspace
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 19
    • Issues 19
    • List
    • Boards
    • Service Desk
    • Milestones
  • Bugzilla
    • Bugzilla
  • Merge requests 111
    • Merge requests 111
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Releases
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Plasma
  • Plasma Workspace
  • Merge requests
  • !768

Revert "[lookandfeel] Fix wake existing screensaver mode with key presses"

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Nate Graham requested to merge ngraham/plasma-workspace:fix-LTS-lockscreen into Plasma/5.18 Apr 03, 2021
  • Overview 2
  • Commits 1
  • Changes 2

This reverts commit 13057013.

That commit set the cursorVisible: propety to true. However that property only exists in the QtQuickControls 2 TextField, not the QQC1 version. We ported the lock screen to PlasmaComponents3 (which internally uses QQC2) after Plasma 5.18, meaning that in the LTS version, we still have a QQC1 TextField so this property does not exist. Trying to set it causes a runtime error and breaks the lock screen.

This went un-detected because, being a runtime error, it did not cause a build failure and writing an integration test for this kind of thing is difficult. Also apparently no KDE developers are actually using the LTS Plasma version (guilty as charged). At least one of these things is probably going to have to change if we want our Plasma LTS promise to mean something. However that's a discussion for another time of course. :)

BUG: 435233 FIXED-IN: 5.18.7.1

Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: fix-LTS-lockscreen