Created attachment 118054 [details] screenshot STEPS TO REPRODUCE 1. open Discover while some update from distro repositories is available 2. click "Update" in the side bar 3. click on some update available 4. hover over "Vendor" link 5. click "More info" button to open the app description 6. hover over "Vendor" link in the app description page OBSERVED RESULT normal cursor is visible in the steps 4 and 6, see my screenshot EXPECTED RESULT hand shaped cursor appears in the steps 4 and 6 SOFTWARE/OS VERSIONS Operating System: Arch Linux KDE Plasma Version: 5.15.0 KDE Frameworks Version: 5.55.0 Qt Version: 5.12.1
Hmm... It would seem some of the link locations do not use LinkButtons. That'll certainly need fixing, thank you for spotting this one :) If you come across others, do give us a poke and we'll get right on that!
Git commit 917819ac26f320c9a8aaec2d871b54ceb978398d by Dan Leinir Turthra Jensen. Committed on 19/02/2019 at 09:53. Pushed by leinir into branch 'master'. Add (and use) a LinkLabel component The component, which is deliberately very simple. just adds the functionality to Label of ensuring we have a hand cursor when hovering over a link in the Label component. This is required due to missing functionality in both Text and QCC2's Label components, which have no mouse cursor changing abilities. There is a Qt bug regarding the issue, mentioned in this new component's code, but so far it looks unlikely to get a "real" fix anytime soon, so, for less headachiness, this component (which simply takes code we already had in one place and applies it slightly more widely and generically)... Differential Revision: https://phabricator.kde.org/D19118 M +1 -9 discover/qml/ApplicationPage.qml A +36 -0 discover/qml/LinkLabel.qml [License: BSD] M +1 -1 discover/qml/ReviewDelegate.qml M +1 -1 discover/qml/UpdatesPage.qml M +1 -0 discover/resources.qrc https://commits.kde.org/discover/917819ac26f320c9a8aaec2d871b54ceb978398d