Differential D24258

Forward error message from backend to ui
AbandonedPublic
Actions

Authored by nicolasfella on Sep 27 2019, 12:06 PM.

Details

Reviewers
None
Group Reviewers
Plasma
KWin
Summary

Saying that no backend was found is not accurate when there was a different error in the backend. The backend gives us an error message we can display instead.

It's not really more usefriendly, but gives us a glimpse at what actually might be wrong

Test Plan

I get a different error message now

Diff Detail

Repository
R104 KScreen
Branch
errormsg
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 17083
Build 17101: arc lint + arc unit
nicolasfella created this revision.Sep 27 2019, 12:06 PM
Restricted Application added a project: Plasma. · View Herald TranscriptSep 27 2019, 12:06 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
nicolasfella requested review of this revision.Sep 27 2019, 12:06 PM
nicolasfella edited the summary of this revision. (Show Details)Sep 27 2019, 12:09 PM
nicolasfella edited the test plan for this revision. (Show Details)
broulik added a subscriber: broulik.Sep 27 2019, 12:10 PM

Maybe at least show "Failed to contact screen management service: $msg"

ngraham added a subscriber: ngraham.Sep 27 2019, 12:41 PM

Heh, if that's a more accurate error message, I as a user would still have no clue what do to about it...

nicolasfella abandoned this revision.May 18 2020, 3:52 PM

Revision Contents
Changeset List

PathPackages
Mkcm/kcm.h (2 lines)
Mkcm/kcm.cpp (2 lines)
Mkcm/package/contents/ui/main.qml (6 lines)

Diff 66941

View Options

kcm/kcm.h

Loading...
View Options

kcm/kcm.cpp

Loading...
View Options

kcm/package/contents/ui/main.qml

Loading...