[Notifications] Tighten layout
ClosedPublic

Authored by broulik on Dec 1 2016, 3:12 PM.

Details

Summary

Our notification popups are huge compared to other platforms, this tightens and streamlines the layout a bit.

The button sizes are much more dynamic now, allowing them to become smaller but also significantly wider. It always bothered me how, at least in German locale, most of the action buttons are elided.

Test Plan


Left is old, right is new

The spectacle icon in the notification is also somewhat redundant now that we have previews.

Diff Detail

Repository
R120 Plasma Workspace
Lint
Lint Skipped
Unit
Unit Tests Skipped
broulik updated this revision to Diff 8677.Dec 1 2016, 3:12 PM
broulik retitled this revision from to [Notifications] Tighten layout.
broulik updated this object.
broulik edited the test plan for this revision. (Show Details)
broulik added reviewers: Plasma, Plasma: Design.
broulik set the repository for this revision to R120 Plasma Workspace.
Restricted Application added a project: Plasma. · View Herald TranscriptDec 1 2016, 3:12 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
mart accepted this revision.Dec 5 2016, 9:31 AM
mart added a reviewer: mart.
mart added a subscriber: mart.
mart added inline comments.
applets/notifications/package/contents/ui/NotificationItem.qml
156

you should still make sure it's an integer number

This revision is now accepted and ready to land.Dec 5 2016, 9:31 AM
This revision was automatically updated to reflect the committed changes.