[Differential] [Commented On] D2248: [kcm] Show output name in output preview

2016-07-21 Thread Sebastian Kügler
sebas added inline comments. INLINE COMMENTS > graesslin wrote in Output.qml:254 > this one looks confusing to my: why is the vendor compared to output.name? > Just from reading it, one doesn't assume that root.output.name is also the > vendor. This label may show the output name, if it does,

[Differential] [Commented On] D2248: [kcm] Show output name in output preview

2016-07-21 Thread Martin Gräßlin
graesslin added a comment. generally looks ok to me. INLINE COMMENTS > Output.qml:254 > id: label > -text: if (root.isCloneMode === true) { > -return ""; > - } else if (root.output.type != KScreenOut