D20569: RFC: Use more compact OSD

Kai Uwe Broulik noreply at phabricator.kde.org
Mon Apr 15 10:13:49 BST 2019


broulik added inline comments.

INLINE COMMENTS

> davidedmundson wrote in OsdItem.qml:31
> Screen.desktopAvailableWidth is all screens. Do you mean that?
> 
> ---
> 
> Dialog should understand Layout attached properties of mainItem, so you can write the slightly more semantic:
> 
> Layout.max screen.width/2
> Layout.min: units * 15
> 
> Though it's Dialog, so who knows....

> Do you mean that?

Umm, just took that from the old code. What's the alternative? I guess I can just do `width` then without taking into acount panels

> Though it's Dialog, so who knows....

I tried that, it didn't work reliably :)

REPOSITORY
  R120 Plasma Workspace

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

To: broulik, #plasma, #vdg
Cc: davidedmundson, hein, Codezela, Fuchs, filipf, zzag, plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190415/915de62b/attachment-0001.html>


More information about the Plasma-devel mailing list