ngraham added inline comments.


> sharvey wrote in SettingsBase.cpp:82
> So should I just run the "preferred size" (1024x760) through the `boundedTo` 
> function for odd-sized screens? In other words, ditch the scale factor 
> completely? As in:
>   const QSize screenSize = 
> (QGuiApplication::primaryScreen()->availableSize()*0.9);
>   const QSize targetSize = QSize(1024, 700);
>   return targetSize.boundedTo(screenSize);
> I thought I'd ask before updating the diff for the 19th time.  :-)

Confirmed. This works fine for me, so you can just remove the `factor` part 
entirely. Sorry for sending you down the wrong path!

  R124 System Settings


To: sharvey, ngraham, mart, davidedmundson
Cc: davidedmundson, cfeck, zzag, plasma-devel, ragreen, Pitel, ZrenBot, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

Reply via email to