D27424: Add missing cursors for Chromium-based apps

2020-02-18 Thread Guilherme Silva
guilhermesilva added a comment. Absolutely! Thank you very much. REPOSITORY R31 Breeze REVISION DETAIL https://phabricator.kde.org/D27424 To: guilhermesilva, #breeze, broulik, ngraham Cc: ngraham, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas,

D27424: Add missing cursors for Chromium-based apps

2020-02-17 Thread Nathaniel Graham
ngraham added a comment. Landed for Plasma 5.18.1! Very nice patch. May it be the first of many! :) REPOSITORY R31 Breeze REVISION DETAIL https://phabricator.kde.org/D27424 To: guilhermesilva, #breeze, broulik, ngraham Cc: ngraham, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack

D27424: Add missing cursors for Chromium-based apps

2020-02-17 Thread Nathaniel Graham
This revision was automatically updated to reflect the committed changes. Closed by commit R31:3944c5f74d04: Add missing cursors for Chromium-based apps (authored by guilhermesilva, committed by ngraham). REPOSITORY R31 Breeze CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D27424?vs=7

D27424: Add missing cursors for Chromium-based apps

2020-02-17 Thread Kai Uwe Broulik
broulik accepted this revision. broulik added a comment. Cool, thanks! I was totally oblivious of the fact the page had three columns :) This can go into 5.18 I'd say REPOSITORY R31 Breeze BRANCH breeze-symlinks (branched from master) REVISION DETAIL https://phabricator.kde.org/D

D27424: Add missing cursors for Chromium-based apps

2020-02-15 Thread Guilherme Silva
guilhermesilva added a comment. It wouldn't call it tough, but it wasn't that easy either... Hahah. Thanks for pointing me to the right direction too! REPOSITORY R31 Breeze BRANCH breeze-symlinks (branched from master) REVISION DETAIL https://phabricator.kde.org/D27424 To: guilhe

D27424: Add missing cursors for Chromium-based apps

2020-02-15 Thread Nathaniel Graham
ngraham accepted this revision. ngraham added a comment. This revision is now accepted and ready to land. Thanks for the patch! Hope the process wasn't too tough to get through for the first time. LGTM! @broulik, does this look good to you too? REPOSITORY R31 Breeze BRANCH breeze-sy

D27424: Add missing cursors for Chromium-based apps

2020-02-15 Thread Guilherme Silva
guilhermesilva created this revision. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. guilhermesilva requested review of this revision. REVISION SUMMARY The Cursor Specification from FreeDesktop doesn't even mention these two, but Chromium-based apps search for them usi