Don't show explicit "Update" button
AbandonedPublic

Authored by broulik on May 29 2019, 1:45 PM.

Details

Reviewers
None
Group Reviewers
Plasma
VDG
Summary

The entire notification popup is clickable already. It's a bit awkward since the cursor turns into a pointing hand cursor while traing to reach the "Update" button.

Test Plan

5.16?

Diff Detail

Repository
R134 Discover Software Store
Lint
Lint Skipped
Unit
Unit Tests Skipped
broulik created this revision.May 29 2019, 1:45 PM
Restricted Application added a project: Plasma. · View Herald TranscriptMay 29 2019, 1:45 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
broulik requested review of this revision.May 29 2019, 1:45 PM

Hmm, I don't think this would be an upgrade. Without a button there I don't think it's obvious that clicking on the pop-up will allow you to go perform updates. Sometimes it's best to be explicit.

Perhaps. But there's also a tray icon that appears simultaneously which people are trained to click.

broulik abandoned this revision.May 29 2019, 2:05 PM