Bug 314905

Summary:	Lockout plasmoid QueryDialog is cropped
Product:	[Unmaintained] plasma4	Reporter:	Anthony Vital <anthony.vital>
Component:	qml-components	Assignee:	Plasma Bugs List <plasma-bugs-null>
Status:	RESOLVED DUPLICATE
Severity:	normal	CC:	alexis, azrdev, franz.trischberger, fxrh, Markus.Elfring, stupor_scurvy343
Priority:	NOR
Version First Reported In:	4.10.0
Target Milestone:	---
Platform:	Arch Linux
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:
Attachments:	Screenshot1 Screenshot2

Description Anthony Vital 2013-02-11 14:44:51 UTC

The lockout plasmoid confirmation dialog for sleeping is not big enough, making the yes/no options barely visible.
The bug is reproducible in plasmoidviewer, although it works fine after playing a little bit with the size of the plasmoidviewer window.

Reproducible: Always

Steps to Reproduce:
1. Add the lockout plasmoid to the desktop/panel.
2. Configure to show the Sleep (suspend to RAM) button.
3. Click the button.
Actual Results:  
Confirmation dialog is cropped, making it hard to see the Yes/No options.

Expected Results:  
Confirmation dialog is the right size, showing all the options in the dialog.

Comment 1 Anthony Vital 2013-02-11 14:45:36 UTC

Created attachment 77135 [details]
Screenshot1

Comment 2 Anthony Vital 2013-02-11 14:46:20 UTC

Created attachment 77136 [details]
Screenshot2

Comment 3 Alexis 2013-02-18 20:11:49 UTC

I have the same bug kde with 4.10.00 from kubuntu ppa (it is an upgraded version of kde, not a clean install). French locale too.

Comment 4 Markus Elfring 2013-02-20 10:58:01 UTC

I wonder also why the buttons are incompletely displayed by the dialogue which appears after I have selected the function "Hibernate (suspend to disk)" for my system.

Comment 5 Felix Rohrbach 2013-03-03 15:44:51 UTC

Doesn't work with english locale, too. Looks like it is a problem with the qml component. Or is it necessary to explicitly set the height for that component?

Comment 6 Franz Trischberger 2013-03-29 16:08:01 UTC

Seems to be a problem in the new Air theme. I just deleted /usr/share/apps/desktopthemes/default and placed there the "default" theme from a 4.9.5 installation, which fixed those size issues.
QueryDialog does some calculations based on "theme.defaultFont" - no idea where that comes from.
Also realised that with the new Air-theme that dialog is "too wide" (just some pixels) an get's resized to a certain smaller width. That wraps the label around which makes it higher pushing the buttons down. After the View inside the dialog get's resized the whole button should get adjusted, too - but it doesn't.

The easiest way to fix this would be to find out what changed in the "default" (Air) theme between 4.9.5 and 4.10.0 and change it back. But I think this bug will strike back so fixing the missing adjustment of size would be even better.

Comment 7 Franz Trischberger 2013-03-30 07:53:01 UTC

It just indirectly depends on the theme (margins, (size of) shadows etc.), issue happens with every theme.
The problem only happens with plasmoids on the desktop, not the panel.
Furthermore the plasmoid must be lower then the minimum height of the Dialog - this issue should be visible with every plasmoid popping up a Plasma::Dialog.

Comment 8 Felix Rohrbach 2013-03-31 17:16:35 UTC


*** This bug has been marked as a duplicate of bug 313943 ***