Bug 367621 - "Get new widgets" button overlaps scroll bar when search result is empty
Summary: "Get new widgets" button overlaps scroll bar when search result is empty
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Widget Explorer (show other bugs)
Version: 5.7.3
Platform: Arch Linux Linux
: NOR minor
Target Milestone: 1.0
Assignee: Plasma Bugs List
URL: http://imgbox.com/Ok5Iio2K
Keywords:
Depends on:
Blocks:
 
Reported: 2016-08-20 21:39 UTC by Patrick Silva
Modified: 2017-05-15 16:26 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Patrick Silva 2016-08-20 21:39:44 UTC 
"Get new widget" button overlaps scroll bar when search result is empty

Reproducible: Always

Steps to Reproduce:
1.open widget explorer
2.type any random text in the search field
3.

Actual Results:  
"Get new widgets" button overlaps scroll bar. See the screenshot link.

Expected Results:  
"Get new widgets" button does NOT overlap scroll bar. See the screenshot link.
Comment 1 Marco Martin 2016-09-26 16:10:29 UTC 
this looks like it's a ScrollView bug, it seems to size the scrollbar wrong when there are no item (and the scrollbar is set as always shown (this is done to work around another bug)
Comment 2 Marco Martin 2016-09-26 17:00:02 UTC 
from Qt's qtquickcontrols/src/controls/ScrollViewStyle.qml:
property int extent: Math.max(minimumHandleLength, __styleData.horizontal ?
                                          (flickableItem ? flickableItem.width/flickableItem.contentWidth : 0 ) * bg.width :
                                          (flickableItem ? flickableItem.height/flickableItem.contentHeight : 0) * bg.height)
giving a divide by zero when flickableItem.contentHeight is 0
Comment 3 Kai Uwe Broulik 2017-01-05 15:49:55 UTC 
Fixed in Qt 5.8
Comment 4 Patrick Silva 2017-03-11 13:48:24 UTC 
Plasma 5.9.3 is built against qt 5.8 on Arch but this bug is still present.
Comment 5 Christoph Feck 2017-03-18 19:08:17 UTC 
I can confirm the bug is still present when running with Qt 5.8.0.
Comment 6 Patrick Silva 2017-03-20 01:55:29 UTC 
Reopening as it's not fixed.
Comment 7 Kai Uwe Broulik 2017-03-20 09:38:37 UTC 
Weird, my commit is in Qt 5.8 branch before 5.8 tagging [1].

[1] https://code.qt.io/cgit/qt/qtquickcontrols.git/commit/?id=8a223fc04e003d93c63dc3c1786da115dd70c7bd
Comment 8 Patrick Silva 2017-05-15 16:26:39 UTC 
Solved on Arch running plasma 5.10 beta. \o/