D24479: [KCM] Make multi-screen draggability more obvious

[David Edmundson]

davidedmundson added inline comments.

INLINE COMMENTS

> davidedmundson wrote in Output.qml:273
> > Or the KCM re-creating the outputModel which would be weird.
> 
> It would, but:
> 
>   connect (m_config.get(), &ConfigHandler::outputModelChanged,
>            this, &KCMKScreen::outputModelChanged);
> 
> which gets emitted in
> 
>   ConfigHandler::setConfig(KScreen::ConfigPtr config)  
> 
> A new screen would be a new config.
> 
> I think output model is the same object, but it would retrigger this evaluation.

This still needs fixing. Regardless of whether it happens to work now or not.

> Screen.qml:56
> +            top: parent.top
> +            horizontalCenter: parent.horizontalCenter
> +            margins: units.smallSpacing

It's better to set the left + right anchors and then setting the text alignment to be horizontally centered.

That way we will wrap or elide if the translated string is too long rather than overflow.

REPOSITORY
  R104 KScreen

REVISION DETAIL
  https://phabricator.kde.org/D24479

To: ngraham, romangg, #plasma, #vdg
Cc: GB_2, davidedmundson, broulik, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20191018/86af4239/attachment.html>