D21471: [Notifications] Remove space between pause/stop and details button

Kai Uwe Broulik noreply at phabricator.kde.org
Wed May 29 08:18:14 BST 2019


broulik created this revision.
broulik added reviewers: Plasma, VDG, hein.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
broulik requested review of this revision.

REVISION SUMMARY
  This was meant to provide some separation but since the arrow icon has so much padding around it anyway and more importantly, it breaks alignment in the list, remove it.

TEST PLAN
  Before
  F6857677: Screenshot_20190529_091333.png <https://phabricator.kde.org/F6857677>
  After
  F6857678: Screenshot_20190529_091405.png <https://phabricator.kde.org/F6857678>
  --ocd
  Note how the stop icon (square) is aligned with the configure icon now
  After in the popu also looks fine
  F6857683: Screenshot_20190529_091758.png <https://phabricator.kde.org/F6857683>

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D21471

AFFECTED FILES
  applets/notifications/package/contents/ui/JobItem.qml

To: broulik, #plasma, #vdg, hein
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/20190529/e72c7e53/attachment.html>


More information about the Plasma-devel mailing list