Bug 417843 - First item in grid view is always selected in new GHNS dialog
Summary: First item in grid view is always selected in new GHNS dialog
Status: RESOLVED FIXED
Alias: None
Product: frameworks-knewstuff
Classification: Frameworks and Libraries
Component: general (show other bugs)
Version: unspecified
Platform: Other Linux
: NOR minor
Target Milestone: ---
Assignee: Alexander Lohnau
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-02-18 20:24 UTC by simon
Modified: 2020-08-01 23:10 UTC (History)
6 users (show)

See Also:
Latest Commit:
Version Fixed In: 5.73
Sentry Crash Report:


Attachments
Screenshot (1.08 MB, image/png)
2020-02-18 20:27 UTC, simon
Details

Note You need to log in before you can comment on or make changes to this bug.
Description simon 2020-02-18 20:24:01 UTC
SUMMARY

On new tiled overview of wallpapers to install there is a hover effect in the form of a blue outline. For the first wallpaper (top-left) the hover effect never disappears. 

STEPS TO REPRODUCE
1. Right-click - Configure Desktop
2. Click "Download New Wallpaper"
3. Move cursor over suggested wallpapers

OBSERVED RESULT
Hover effect showing for top-left wallpaper and whichever other one the cursor moves to.

EXPECTED RESULT
Hover effect for top-left wallpaper should only show if the cursor is indeed on the same wallpaper. 

SOFTWARE/OS VERSIONS

Linux/KDE Plasma: Kubuntu 19.10
KDE Plasma Version: 5.18


ADDITIONAL INFORMATION
Comment 1 simon 2020-02-18 20:27:33 UTC
Created attachment 126142 [details]
Screenshot
Comment 2 Nate Graham 2020-02-18 22:50:34 UTC
Confirmed.
Comment 3 Dan Leinir Turthra Jensen 2020-02-19 08:56:18 UTC
If by top-left you mean the bright blue focus rectangle, that's the keyboard focus rectangle (which there is a problem with, as the view is not focused by default as it ought to be). If you're talking about there being two hover effect rectangles the way the two top right ones have in your screenshot, i can't seem to provoke that at all and will need some more information on how you're doing that.
Comment 4 simon 2020-02-19 12:13:43 UTC
Hi Dan

Thanks for the follow-up. Correct, I refer to bright blue focus rectanlge. I would expect only one rectangle to be visible at any given moment like it is on web content. Alternatively keyboard and focus focus should be visually differentiated. 

Right now a user looks at two blue focus rectangles which is confusing. 

Please ignore the screenshot as it was taken while moving from one tile to another.
Comment 5 Bug Janitor Service 2020-03-05 04:33:13 UTC
Dear Bug Submitter,

This bug has been in NEEDSINFO status with no change for at least
15 days. Please provide the requested information as soon as
possible and set the bug status as REPORTED. Due to regular bug
tracker maintenance, if the bug is still in NEEDSINFO status with
no change in 30 days the bug will be closed as RESOLVED > WORKSFORME
due to lack of needed information.

For more information about our bug triaging procedures please read the
wiki located here:
https://community.kde.org/Guidelines_and_HOWTOs/Bug_triaging

If you have already provided the requested information, please
mark the bug as REPORTED so that the KDE team knows that the bug is
ready to be confirmed.

Thank you for helping us make KDE software even better for everyone!
Comment 6 Christoph Feck 2020-03-06 11:54:17 UTC
Requested information was added with comment 4; changing status for inspection.
Comment 7 Bug Janitor Service 2020-07-23 09:29:54 UTC
A possibly relevant merge request was started @ https://invent.kde.org/frameworks/knewstuff/-/merge_requests/19
Comment 8 Nate Graham 2020-07-23 17:14:23 UTC
Git commit 5e31f8505a9ed335e56eee4087f1435d993f97ca by Nate Graham, on behalf of Alexander Lohnau.
Committed on 23/07/2020 at 17:14.
Pushed by ngraham into branch 'master'.

Do not focus first element automatically
FIXED-IN: 5.73

There are currently no keyboard controls, apart from tab implemented.
Consequently we can remove this tiny bit of code, because it looks
really confusing if we have two items focused at the same time.

M  +4    -2    src/qtquick/qml/private/GridTileDelegate.qml

https://invent.kde.org/frameworks/knewstuff/commit/5e31f8505a9ed335e56eee4087f1435d993f97ca