Bug 202566

Summary: Fix Plasma::Corona::popupPosition to align the dialog top edge with the given position
Product: [Unmaintained] plasma4 Reporter: Steve Gilberd <steve>
Component: widget-folderviewAssignee: Ignat Semenov <i.semenov.kde>
Status: RESOLVED UNMAINTAINED    
Severity: normal CC: aseigo, asraniel, caionnew, kde, lasse.liehu, milovan.krivokapic, notmart, steve
Priority: NOR    
Version: 4.7.3   
Target Milestone: ---   
Platform: Compiled Sources   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:
Attachments: Example screenshot

Description Steve Gilberd 2009-08-05 05:35:32 UTC 
Version:            (using KDE 4.3.0)
Compiler:          gcc (Gentoo 4.3.3-r2 p1.2, pie-10.1.5) 4.3.3 x85_64-pc-linux-gnu
OS:                Linux
Installed from:    Compiled From Sources

When launching a folder view plasmoid hosted by a panel aligned along a vertical screen edge, the folder view height is restricted to the height of the panel.

The correct behavior would be to allow it to take on any height, as long as it is no higher than the screen.

This bug is a regression - it worked perfectly in 4.2.4, where the folder view height would scale to accommodate all the items in the folder.
Comment 1 Steve Gilberd 2009-08-05 07:05:40 UTC 
Created attachment 35851 [details]
Example screenshot
Comment 2 Aaron J. Seigo 2009-08-05 07:55:43 UTC 
hm. neat bug. btw, thanks for filing these and testing 4.3.0. you should consider grabbing the RC releases in future (e.g. for 4.4)  because you seem to be pretty good at finding and clearly reporting issues. when we find out about these issues sooner, we can often fix them for the .0 release :)
Comment 3 Steve Gilberd 2009-08-05 08:24:12 UTC 
Will definitely consider it, subject to having sufficient free machine time to compile them - KDE isn't exactly a lightweight when it comes to compiling an entire release!
Comment 4 Marco Martin 2009-08-05 17:39:25 UTC 
yup, it would be the correct behaviour indeed, however the popup would look cutted, hmm.
well i suppose it would be the least annoying behaviour anyways...
Comment 5 Steve Gilberd 2009-09-11 01:02:19 UTC 
Still present in 4.3.1.
Comment 6 Lasse Liehu 2010-01-03 21:17:54 UTC 
Still present in KDE SC 4.4 beta 2, Qt 4.6.0.
Comment 7 Nicolas L. 2010-06-07 10:31:16 UTC 
still valid on kde 4.4.4
Comment 8 disabled account 2010-06-08 03:30:53 UTC 
Still present in 4.5 beta 1, however the popup can go higher than the panel height, it just can not go lower, so it is just an issue for short panels on the top of a screen.
Comment 9 Milovan 2011-12-03 13:54:16 UTC 
4.7.3 confirmed. 
When listing down, its height is limited by the panel size. 
When listing up, it goes till the upper screen edge.
Comment 10 Ignat Semenov 2012-06-06 07:04:55 UTC 
This is in Plasma::Corona::popupPosition()

It needs to be changed to align the top of the returned rect with the top of the given position. Currently, this is not always the case, sometimes it overshoots, as you can see with e.g. this bug.
Comment 11 Nate Graham 2018-06-08 19:33:28 UTC 
Hello!

This bug report was filed for KDE Plasma 4, which reached end-of-support status in August 2015. KDE Plasma 5's desktop shell has been almost completely rewritten for better performance and usability, so it is likely that this bug is already resolved in Plasma 5.

Accordingly, we hope you understand why we must close this bug report. If the issue described  here is still present in KDE Plasma 5.12 or later, please feel free to open a new ticket in the "plasmashell" product after reading https://community.kde.org/Get_Involved/Bug_Reporting

If you would like to get involved in KDE's bug triaging effort so that future mass bug closes like this are less likely, please read https://community.kde.org/Get_Involved#Bug_Triaging

Thanks for your understanding!

Nate Graham