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