----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119860/#review64978 -----------------------------------------------------------
src/plasmaquick/dialog.cpp <https://git.reviewboard.kde.org/r/119860/#comment45409> we're now doing this on every resize, is this a big overhead? src/plasmaquick/dialog.cpp <https://git.reviewboard.kde.org/r/119860/#comment45410> this is new. src/plasmaquick/dialog.cpp <https://git.reviewboard.kde.org/r/119860/#comment45407> delete it if you don't need it src/plasmaquick/dialog.cpp <https://git.reviewboard.kde.org/r/119860/#comment45411> syncToMainItemSize calls updateTheme at the end do we need to call it twice? if so why? src/plasmaquick/dialog.cpp <https://git.reviewboard.kde.org/r/119860/#comment45408> only do this if we're in componentCompleted - David Edmundson On Aug. 21, 2014, 1:03 p.m., Marco Martin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/119860/ > ----------------------------------------------------------- > > (Updated Aug. 21, 2014, 1:03 p.m.) > > > Review request for Plasma and Pier Luigi Fiorini. > > > Repository: plasma-framework > > > Description > ------- > > add the possibility to not have backgrounds in dialogs. > one ting i don't like that much is the fact that it pretty much forces to > either having perfect rectangular windows or to have composite for correct > looks, but that will be probably less important. > One thing that should be paid attention is that with no background is not > supported the kwin-side shadow, as well no blur as well, since we don't have > info for the shape, tough if needed in the future blur with just the full > rectangular shape may be added as a new flag in backgroundhints, those > possibilities of expansion is one reason it's an enum not a bool. > > The use case for it is for custom dialogs in Maui, and since Dialog already > manages the million edge cases of sizing and positioning that we had to fix, > the least painful route i think is to use them with the background disabled. > the framesvgitem is still there, but not having a svg loaded shouldn't take > memory for images neither in heap nor gpu (probably would be good as well to > set the flag itemhasnocontents on framesvgitem if imagepath is invalid, to be > really sure it will never even try to upload any texture) > > > Diffs > ----- > > src/plasmaquick/dialog.h 27b3b5a > src/plasmaquick/dialog.cpp 64fb4a9 > > Diff: https://git.reviewboard.kde.org/r/119860/diff/ > > > Testing > ------- > > > Thanks, > > Marco Martin > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
