Re: Review Request: Plasmate: Make Publisher able to remove the packages which have been installed with the cmake method

2012-11-23 Thread Antonis Tsiapaliokas
On Nov. 23, 2012, 12:06 a.m., Aaron J. Seigo wrote: why would Publish do the exact opposite and uninstall something? shouldn't this be a different function? in which case is it needed / desired to even do this from plasmate? Yes, you have right. The Publish should only install things..

Re: Review Request: Use static KNS3::DownloadDialog in PlasmoidPackage

2012-11-23 Thread Jan Grulich
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107413/ --- (Updated Nov. 22, 2012, 7:08 p.m.) Review request for kdelibs and Plasma.

Re: Review Request: Use static KNS3::DownloadDialog in PlasmoidPackage

2012-11-23 Thread Jan Grulich
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107413/ --- (Updated Nov. 23, 2012, 9:39 a.m.) Review request for kdelibs and Plasma.

Re: [RFC] New (QML) Desktop Containment

2012-11-23 Thread Aaron J. Seigo
On Thursday, November 22, 2012 17:11:56 Sebastian Kügler wrote: So for many workflows, we rely quite heavily on the desktop being unlocked. indeed. locking the desktop introduces a separation between functions. axiomatically, it removes functionality. that's the point of it. and when removing

Review Request: plasmoid currentappconrol - max width of dropdown

2012-11-23 Thread Greg T
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107433/ --- Review request for Plasma. Description --- the drop down list gets

Re: Review Request: plasmoid currentappconrol - max width of dropdown

2012-11-23 Thread Greg T
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107433/ --- (Updated Nov. 23, 2012, 10:27 a.m.) Review request for Plasma.

Re: Review Request: Use static KNS3::DownloadDialog in PlasmoidPackage

2012-11-23 Thread Aaron J. Seigo
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107413/#review22419 --- Ship it! Ship It! - Aaron J. Seigo On Nov. 23, 2012, 11:12

Re: Review Request: Use static KNS3::DownloadDialog in PlasmoidPackage

2012-11-23 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107413/#review22420 --- This review has been submitted with commit

Re: Review Request: Use static KNS3::DownloadDialog in PlasmoidPackage

2012-11-23 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107413/#review22421 --- This review has been submitted with commit

Re: Review Request: Add the plasmagik icon into the plasmate's repo

2012-11-23 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107419/#review22429 --- This review has been submitted with commit

Re: Review Request: hide save and save as actions in every language

2012-11-23 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107418/#review22431 --- This review has been submitted with commit

Re: Review Request: Plasmate: Make Publisher able to remove the packages which have been installed with the cmake method

2012-11-23 Thread Aaron J. Seigo
On Nov. 23, 2012, 12:06 a.m., Aaron J. Seigo wrote: why would Publish do the exact opposite and uninstall something? shouldn't this be a different function? in which case is it needed / desired to even do this from plasmate? Antonis Tsiapaliokas wrote: Yes, you have right. The

Re: Review Request: plasmoid currentappconrol - max width of dropdown

2012-11-23 Thread Aaron J. Seigo
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107433/#review22435 ---

Re: Review Request: make the DeviceNotifier Plasmoid scale with high-res screens

2012-11-23 Thread Michael Zanetti
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107428/ --- (Updated Nov. 23, 2012, 6:15 p.m.) Review request for Plasma and Kai Uwe

Re: Review Request: make the DeviceNotifier Plasmoid scale with high-res screens

2012-11-23 Thread Michael Zanetti
On Nov. 23, 2012, 1:03 p.m., Kai Uwe Broulik wrote: plasma/generic/applets/devicenotifier/package/contents/ui/DeviceItem.qml, line 129 http://git.reviewboard.kde.org/r/107428/diff/1/?file=95873#file95873line129 Do two lines fit in? Maybe use elide: Text.ElideRight

Re: Review Request: make the DeviceNotifier Plasmoid scale with high-res screens

2012-11-23 Thread Michael Zanetti
On Nov. 23, 2012, 1:03 p.m., Kai Uwe Broulik wrote: plasma/generic/applets/devicenotifier/package/contents/ui/DeviceItem.qml, line 129 http://git.reviewboard.kde.org/r/107428/diff/1/?file=95873#file95873line129 Do two lines fit in? Maybe use elide: Text.ElideRight

Re: Review Request: make the DeviceNotifier Plasmoid scale with high-res screens

2012-11-23 Thread Michael Zanetti
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107428/ --- (Updated Nov. 23, 2012, 7:32 p.m.) Review request for Plasma and Kai Uwe

Re: Review Request: plasmoid currentappconrol - max width of dropdown

2012-11-23 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107433/#review22447 --- This review has been submitted with commit

Re: Review Request: plasmoid currentappconrol - max width of dropdown

2012-11-23 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107433/#review22448 --- This review has been submitted with commit

Re: Review Request: make the DeviceNotifier Plasmoid scale with high-res screens

2012-11-23 Thread Michael Zanetti
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107428/ --- (Updated Nov. 23, 2012, 10:11 p.m.) Review request for Plasma and Kai Uwe

[KDE Bugtracking System] REMINDER: current Plasma regressions

2012-11-23 Thread bugzilla_noreply
Please find below a list of the current regressions reported for Plasma. This is a weekly reminder. This search was scheduled by myr...@kde.org. Plasma regressions -- Bug 290018: https://bugs.kde.org/show_bug.cgi?id=290018 Priority: NOR Severity: normal Platform: Ubuntu