From 2f66aa9b07986a721124c9e738d6daf28446a7e7 Mon Sep 17 00:00:00 2001 From: Adriaan de Groot Date: Fri, 9 Mar 2018 11:24:41 -0500 Subject: [PATCH] [plasmalnf] Change to IgnoreAspectRatio It is the distro's responsibility to produce screenshots that look good; previously I chose to preserve the aspect ratio on the grounds that this would keep the look of the screenshot even if the distro had done one in a weird size. This makes the screenshot part of the LNF selection look weird, though, since then you get blank parts. Switch to ignoring the aspect ration; distro's should produce screenshots in a 12x8 (i.e. 3:2) aspect ratio, preferrably at least 120x80 pixels -- but keep in mind hiDPI and the default font sizes of the distro, which may make other sizes look better. (this follows discussion with BlueStar Linux) --- src/modules/plasmalnf/ThemeWidget.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/modules/plasmalnf/ThemeWidget.cpp b/src/modules/plasmalnf/ThemeWidget.cpp index f2a038030..0dd73dbd3 100644 --- a/src/modules/plasmalnf/ThemeWidget.cpp +++ b/src/modules/plasmalnf/ThemeWidget.cpp @@ -58,7 +58,7 @@ ThemeWidget::ThemeWidget(const ThemeInfo& info, QWidget* parent) image.fill( QColor( QRgb( hash_color ) ) ); } - image = image.scaled( image_size, Qt::KeepAspectRatio, Qt::SmoothTransformation ); + image = image.scaled( image_size, Qt::IgnoreAspectRatio, Qt::SmoothTransformation ); QLabel* image_label = new QLabel( this ); image_label->setPixmap( image );