----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126356/#review90832 -----------------------------------------------------------
I'm not so sure if plasmoids started from desktop and panel have to be equal (if at all than I'd care about the caption). For what reason, familiarity? And the final size in the last picture (clipboardMinimumSize.png) is way too small to suit user's needs. That is here to browse through and read the content. For example 'dd if=/dev/zero of=/dev/s...' thrills only the bravest. The size of clipboard-new.png is okay. The current size for the panel based plasmoid is reasonable and similar to other plasmoids like the notification. And it fits the general visual style of Breeze. But that's only a two cent option. (BTW: I read in several postings that Breeze becomes very breezy with too large marigins and spaces.) Whether What I rather would expect is that a desktop plugin shows the list permanently and has means for resizing. Actually what is shown in the last screenshot but with a better default size. Took me some time to figure out how to achieve this (was aware of the option only because of the screenshot): One has to long click the icon! And there is no way back. tl;dr: (sorry for all my considerations; writing down all thoughts while testing is somewhat verbose) * same height is okay but not really necessary * default size should be large enough to read the labels * users should be able to minimize as much as possible, so yes to the patch if it's not the default - Heiko Tietze On Jan. 9, 2016, 9:54 p.m., andreas kainz wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/126356/ > ----------------------------------------------------------- > > (Updated Jan. 9, 2016, 9:54 p.m.) > > > Review request for Plasma and Heiko Tietze. > > > Repository: plasma-workspace > > > Description > ------- > > if you use clipboard on the desktop the minimum window height is heigher than > you use clipboard from the panel (drop down window). As the user can define > the width and height for the desktop plasmoid the minimum allowed height > should be at least the same than the drop down window in the panel. so I > change the minimumHeig to 16. > > > Diffs > ----- > > applets/clipboard/contents/ui/clipboard.qml 97230c1 > > Diff: https://git.reviewboard.kde.org/r/126356/diff/ > > > Testing > ------- > > > File Attachments > ---------------- > > old before with 26 min height > > https://git.reviewboard.kde.org/media/uploaded/files/2015/12/15/cdc4e222-a954-4f92-816f-521f35aab7c7__clipboard-alt_.png > new after with 16 min height > > https://git.reviewboard.kde.org/media/uploaded/files/2015/12/15/50773744-a980-4091-8ef2-7eec516cb597__clipboard-new_.png > screenshot > > https://git.reviewboard.kde.org/media/uploaded/files/2016/01/09/566b420b-df44-44d4-86b3-b9f2d441aed8__clipboardMinimumSize.png > > > Thanks, > > andreas kainz > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
