[notifications] Hide close/configure buttons on mobile
Needs ReviewPublic

Authored by nicolasfella on Apr 2 2020, 9:11 PM.

Details

Reviewers
None
Group Reviewers
Plasma
Plasma: Mobile
Summary

The close button is not really needed since the notification can just be swiped away. Without the close button the configure button is a bit loneley so let's remove that too. It's still there in the history applet though

Diff Detail

Repository
R120 Plasma Workspace
Branch
close
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 24648
Build 24666: arc lint + arc unit
nicolasfella created this revision.Apr 2 2020, 9:11 PM
Restricted Application added a project: Plasma. · View Herald TranscriptApr 2 2020, 9:11 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
nicolasfella requested review of this revision.Apr 2 2020, 9:11 PM
broulik added a subscriber: broulik.Apr 2 2020, 9:13 PM

I think we also want to stop the animation/circle thing, too? Maybe shove all of it in a loader?

I think we also want to stop the animation/circle thing, too? Maybe shove all of it in a loader?

Isn't that done automatically since the circle thing is a child of the close button (which is invisible)?

mart added a subscriber: mart.Apr 6 2020, 9:21 AM

having just the circle anim and no closebutton would be fine i think.. at least, i would prefer if it would retain some sort of visual countdown