Bug 202566 - Fix Plasma::Corona::popupPosition to align the dialog top edge with the given position
Summary: Fix Plasma::Corona::popupPosition to align the dialog top edge with the given...
Status: RESOLVED UNMAINTAINED
Alias: None
Product: plasma4
Classification: Plasma
Component: widget-folderview (show other bugs)
Version: 4.7.3
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: Ignat Semenov
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-08-05 05:35 UTC by Steve Gilberd
Modified: 2018-06-08 19:33 UTC (History)
8 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
Example screenshot (68.83 KB, image/jpeg)
2009-08-05 07:05 UTC, Steve Gilberd
Details

Note You need to log in before you can comment on or make changes to this bug.
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