Bug 416478

Summary:	SDDM theme: the password box is tiny
Product:	[Plasma] plasmashell	Reporter:	Filip Fila <filipfila.kde>
Component:	Theme - Breeze	Assignee:	visual-design
Status:	RESOLVED FIXED
Severity:	normal	CC:	davestechshop, kde, nate, plasma-bugs
Priority:	NOR
Version:	5.17.90
Target Milestone:	1.0
Platform:	Other
OS:	Linux
See Also:	https://bugs.kde.org/show_bug.cgi?id=410810
Latest Commit:		Version Fixed In:	5.23
Sentry Crash Report:

Description Filip Fila 2020-01-19 20:47:06 UTC

Forwarding this bug report: https://github.com/sddm/sddm/issues/1177

I could never reproduce it IRL, but these are the steps to reproduce it in a testing environment.

1. git clone the plasma-workspace repo
2. sddm-greeter --test-mode --theme plasma-workspace/sddm-theme


The password box is tiny. Why? Well because we overhauled the font sizes and now rely on the font size to be specified in 'theme.conf'. The password box is resized based on the font size. Obviously the non-installed repo SDDM theme doesn't have theme.conf generated yet but it proves what causes this:

When the theme cannot access the 'theme.conf font' size value, it will probably use 1pt as the font size, thus the small password box. We can easily test this by editing Login.qml and changing the password box's font size to something sane. The password box is back to its normal size.

But this bug shouldn't be happening. The Breeze SDDM theme should create a theme.conf file with the entry "fontSize=10" specified. Why doesn't it do that in some cases, I'm not sure. Packaging issue maybe?

Comment 1 Nate Graham 2020-01-19 21:34:58 UTC

I suppose we could fix this in the theme itself by setting a minimum for the font size?

Should probably investigate and fix the SDDM config issue regardless, though.

Comment 2 Filip Fila 2020-01-19 21:54:03 UTC

Yeah I'm hoping to hear more from MountainX in regards to:

- does their /usr/share/sddm/themes/breeze/theme.conf file exist?
- does it have 'fontSize=10' specified inside?

It really sounds like some sort of a glitch in the installation process.

Honestly the whole theme.conf shebang is just a messy workaround for the fact that SDDM defaults to 9pt fonts so the best solution would be if we could switch this to 10pt somewhere else. I asked David about this once and he said SDDM just takes some default Qt size, but I don't know where to change this value.

Comment 3 MountainX 2020-01-20 00:02:47 UTC

This info is very helpful! I appreciate understanding the issue now.

- does their /usr/share/sddm/themes/breeze/theme.conf file exist?

Yes.

- does it have 'fontSize=10' specified inside?

No.

Happy to provide more info.

Comment 4 Filip Fila 2020-01-20 11:16:15 UTC

(In reply to MountainX from comment #3)
> This info is very helpful! I appreciate understanding the issue now.
> 
> - does their /usr/share/sddm/themes/breeze/theme.conf file exist?
> 
> Yes.
> 
> - does it have 'fontSize=10' specified inside?
> 
> No.
> 
> Happy to provide more info.

I just removed 'fontSize=10' from my theme.conf and reinstalled plasma-workspace. The entry reappeared in the file so it can't be that the package doesn't know how to write a new entry into an existing file.

I'm on Manjaro so the packaging should be exactly the same as in Arch.

Can you try reinstalling plasma-workspace and see if that adds the fontSize line?

Comment 5 MountainX 2020-11-29 05:13:05 UTC

FYI I stopped using the breeze theme for sddm. (I switched to sugar-dark.) However, if this issue has been fixed, I am interested in switching back.

Comment 6 Nate Graham 2021-10-01 16:28:53 UTC

This should be fixed now in Plasma 5.23, if not 5.22.