D21361: [Notifications] Remove "Notifications" heading
Kai Uwe Broulik
noreply at phabricator.kde.org
Thu May 23 09:21:25 BST 2019
broulik created this revision.
broulik added reviewers: Plasma, VDG.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
broulik requested review of this revision.
REVISION SUMMARY
It is redundant with the title added by system tray. Even when added standalone it is kind of obvious that this is notifications.
The clear button is moved next to the configure button.
TEST PLAN
5.16 branch
F6843694: Screenshot_20190523_101859.png <https://phabricator.kde.org/F6843694>
Clear button is aligned with "configure" button on notifications, hence the `spacing: 0`
No notifications text still shows up fine
F6843695: Screenshot_20190523_101945.png <https://phabricator.kde.org/F6843695>
REPOSITORY
R120 Plasma Workspace
REVISION DETAIL
https://phabricator.kde.org/D21361
AFFECTED FILES
applets/notifications/package/contents/ui/FullRepresentation.qml
To: broulik, #plasma, #vdg
Cc: plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190523/89bbeea4/attachment.html>
More information about the Plasma-devel
mailing list