Bug 367621

Summary: "Get new widgets" button overlaps scroll bar when search result is empty
Product: [Plasma] plasmashell Reporter: Patrick Silva <bugseforuns>
Component: Widget ExplorerAssignee: Plasma Bugs List <plasma-bugs>
Status: RESOLVED FIXED    
Severity: minor CC: kde, notmart
Priority: NOR    
Version: 5.7.3   
Target Milestone: 1.0   
Platform: Arch Linux   
OS: Linux   
URL: http://imgbox.com/Ok5Iio2K
Latest Commit: Version Fixed In:

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/