D26726: [Breeze Cursors] Add some more cursor names

[Nathaniel Graham]

ngraham added a comment.


  In D26726#610446 <https://phabricator.kde.org/D26726#610446>, @guilhermesilva wrote:
  
  > Hello,
  >
  > I'm the person the reported this issue <https://bugs.chromium.org/p/chromium/issues/detail?id=1038603> to the Chromium team. The fix they made will probably land only on Chromium 81 or later, but a lot of Electron apps will still rely on older versions of Chromium for a long time, where this missing cursors issue will still be present, unless they upgrade to Plasma 5.18.
  >
  > Speaking of which, I just upgraded to Plasma 5.18 myself, and it looks incredible, by the way. And this fix of yours is already present there too, which is great. However, I believe you unintentionally missed two CSS cursors: `grab`, and `grabbing`, which should be linked to `openhand` and `closedhand`, respectively. The rest of cursors are fine and can be tested here:
  >
  > http://elektronotdienst-nuernberg.de/bugs/cursor.html
  >
  > So, it'd be great if you added those too.
  >
  > Thanks!
  
  
  Great, I'm glad you're happy with it!
  
  As you can see here, the change is pretty simple. Would you like to try your hand at submitting a patch to add the two missing cursors, modeled on this one?

REPOSITORY
  R31 Breeze

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

To: broulik, #vdg, ngraham
Cc: guilhermesilva, davidedmundson, ngraham, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20200214/afa2298e/attachment.html>