ndavis added a comment.
In D21630#475501 <https://phabricator.kde.org/D21630#475501>, @aacid wrote: > **wow** Not even one minute to review things > > I don't particularly like this, input-mouse is an icon that exists in most icon themes > extra/adwaita-icon-theme > extra/breeze-icons > extra/efl > extra/gnome-icon-theme > extra/gnome-icon-theme-symbolic > extra/gnome-themes-extra > extra/oxygen-icons > extra/oxygen-icons-svg > extra/qt5-doc > extra/xf86-input-mouse > extra/xfce4-power-manager > community/arc-icon-theme > community/elementary-icon-theme > community/faba-icon-theme > community/faenza-icon-theme > community/lxinput > community/lxinput-gtk3 > community/mate-icon-theme > community/mate-icon-theme-faenza > community/mate-themes > community/moka-icon-theme > community/papirus-icon-theme > > transform-browse exists only in breeze-icons > > but 🤷♂️ i guess Sorry, I'll wait longer in the future, but input-mouse is still wrong. This simply doesn't mean the same thing as "grabber" or "browse": https://raw.githubusercontent.com/GNOME/adwaita-icon-theme/master/Adwaita/22x22/legacy/input-mouse.png However, if you feel this commit should be reverted, my feelings won't be hurt. If you know of another common icon that works better than input-mouse, perhaps we can use that instead? REPOSITORY R223 Okular REVISION DETAIL https://phabricator.kde.org/D21630 To: ndavis, #okular, #vdg, ngraham Cc: aacid, ngraham, okular-devel, joaonetto, tfella, darcyshen
