----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122119/#review74209 -----------------------------------------------------------
The change looks off. Shouldn't this go to either the QQC theme or upstream? Changing from using Qt components to our own thing is kind of against what we've been working for. - Aleix Pol Gonzalez On Jan. 18, 2015, 12:36 a.m., Kai Uwe Broulik wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/122119/ > ----------------------------------------------------------- > > (Updated Jan. 18, 2015, 12:36 a.m.) > > > Review request for Plasma. > > > Repository: plasma-framework > > > Description > ------- > > This ports those two components to the new KQuickControls ones that features > a full context menu. > > Note that this requires Qt 5.4 (QtQuick.Controls 1.3), so might needs to be > postponed. > > > Diffs > ----- > > src/declarativeimports/plasmacomponents/qml/TextArea.qml 1fcbb5b > src/declarativeimports/plasmacomponents/qml/TextField.qml c027add > > Diff: https://git.reviewboard.kde.org/r/122119/diff/ > > > Testing > ------- > > Ran both tests in qmlscene and works as before with the new context menu. > > I noticed that in KRunner I get a widget-like context menu with shadows and > all whereas in plasmoids (kickoff and klipper) I get an out-of-place looking > menu without shadows and using Plasma highlight. This is probably it using > Plasma's MenuStyle rather than the desktop style. > > > Thanks, > > Kai Uwe Broulik > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
