D12872: ScalableTest, add "scalable" plasma-browser-integration

Maximiliano Curia noreply at phabricator.kde.org
Mon May 14 11:50:29 UTC 2018


maximilianocuria created this revision.
Restricted Application added a project: Frameworks.
Restricted Application added a subscriber: kde-frameworks-devel.
maximilianocuria requested review of this revision.

REVISION SUMMARY
  With the introduction of the plasma-browser-integration icon (Add
  plasma-browser-integration system tray icon, 47b8e71a <https://phabricator.kde.org/R266:47b8e71a42a33a942c19ac4d2b596b5ee244476d>) the ScalableTest started
  failing [1]. The test seems to expect a "larger" icon (greater than 48)
  available to consider that there is a scalable icon. I'm not
  sure if this test is still useful, as all the icons are now in svg format, and
  while they might be designed for a low resolution, they are "scalable".
  
  This patch simply adds symlinks to the 48 and 64 res, in order to make the
  test pass.
  
  [1]: https://build.kde.org/job/Frameworks%20breeze-icons%20kf5-qt5%20SUSEQt5.10/81/

TEST PLAN
  #frameworks <https://phabricator.kde.org/tag/frameworks/>, faure, andreaska, andreask

REPOSITORY
  R266 Breeze Icons

BRANCH
  scalable_pbi (branched from master)

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

AFFECTED FILES
  icons-dark/apps/48/plasma-browser-integration.svg
  icons-dark/apps/64/plasma-browser-integration.svg
  icons/apps/48/plasma-browser-integration.svg
  icons/apps/64/plasma-browser-integration.svg

To: maximilianocuria
Cc: kde-frameworks-devel, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180514/2ce19c20/attachment.html>


More information about the Kde-frameworks-devel mailing list