[Differential] [Commented On] D3616: [Lock Screen / Login] Add "reveal password button"

2016-12-07 Thread broulik (Kai Uwe Broulik)
broulik added a comment. One thing we perhaps should do is add a kiosk restriction to globally disable this button everywhere. REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D3616 EMAIL PREFERENCES

Re: Selecting a Plasma logo

2016-12-07 Thread Ken Vermette
I think this is likely the best and most expedient idea. Just list all the potential logotypes and have everyone score every logo on a scale. Whichever logo has the highest score becomes the Plasma logo. If there's a tie the VDG serves as tiebreaker. If everyone is O.K. with something like Google

Re: KDE 4 vs 5, regional settings and dolphin

2016-12-07 Thread René J . V . Bertin
Christoph Feck wrote: > http://www.ivoras.net/blog/tree/2010-12-20.the-wonderful-en_dk-locale.html Heh, more or less what I thought it was. The weird thing is that both Ireland and Malta are English-speaking EU countries that use the € and metric system. Anyway ... :)

[Differential] [Accepted] D3616: [Lock Screen / Login] Add "reveal password button"

2016-12-07 Thread Martin Gräßlin
graesslin accepted this revision. graesslin added a reviewer: graesslin. graesslin added a comment. This revision is now accepted and ready to land. +1 from KScreenlocker maintainer. If you also want VDG input, please wait :-) REPOSITORY R120 Plasma Workspace REVISION DETAIL

Re: Selecting a Plasma logo

2016-12-07 Thread Martin Gräßlin
Am 2016-12-07 18:04, schrieb Marco Martin: Hi Jens, Thanks to make this rolling again! and yes, is massively about time to decide something On Wednesday 07 December 2016 13:58:34 Jens Reuterberg wrote: #1 means that we with a high level of chance will go to the Plasma logo we had before this

[Differential] [Commented On] D3616: [Lock Screen / Login] Add "reveal password button"

2016-12-07 Thread cfeck (Christoph Feck)
cfeck added a comment. Using that argument, we would have to remove the 'reveal password' option from KPasswordDialog, too. REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D3616 EMAIL PREFERENCES

[Differential] [Commented On] D3616: [Lock Screen / Login] Add "reveal password button"

2016-12-07 Thread subdiff (Roman Gilg)
subdiff added a comment. On one hand: Yea, why not. On the other imagine the following situation: Two people sit in front of the PC, and one of them wants to login to the PC. After entering his password, he accidently hits the "show password" icon (since it's pretty near to the

Re: Selecting a Plasma logo

2016-12-07 Thread Marco Martin
Hi Jens, Thanks to make this rolling again! and yes, is massively about time to decide something On Wednesday 07 December 2016 13:58:34 Jens Reuterberg wrote: > #1 means that we with a high level of chance will go to the Plasma logo we > had before this started. There is zero criticism against it

Re: KDE 4 vs 5, regional settings and dolphin

2016-12-07 Thread Christoph Feck
On 07.12.2016 14:41, René J. V. Bertin wrote: Christoph Feck wrote: See bug 340982. KF5 applications do not use the customizable KLocale class, but use what Qt offers (QLocale), which we hoped would soon get support for customizations, but progress is slow, so no code has surfaced. I just

[Differential] [Updated] D3617: [Touchpad KCM] New KWin Wayland version

2016-12-07 Thread subdiff (Roman Gilg)
subdiff updated the summary for this revision. REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D3617 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: subdiff, #kwin, #plasma_on_wayland, #plasma, #vdg Cc: kwin, plasma-devel,

[Differential] [Updated] D3617: [Touchpad KCM] New KWin Wayland version

2016-12-07 Thread subdiff (Roman Gilg)
subdiff added a reviewer: VDG. REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D3617 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: subdiff, #kwin, #plasma_on_wayland, #plasma, #vdg Cc: kwin, plasma-devel, lesliezhai,

[Differential] [Request, 2,415 lines] D3617: [Touchpad KCM] New KWin Wayland version

2016-12-07 Thread subdiff (Roman Gilg)
subdiff created this revision. subdiff added reviewers: KWin, Plasma on Wayland, Plasma. subdiff added a subscriber: plasma-devel. subdiff set the repository for this revision to R119 Plasma Desktop. subdiff added a project: Plasma. Restricted Application edited projects, added Plasma on Wayland,

Re: status of kde/plasma kiosk framework in kf5

2016-12-07 Thread David Edmundson
> > > > i also kinda hacked my own secure environment where shell access is not > allowed by placing a .desktop file in .local/share/kservices5/ServiceMenus/ > that allows me to open a terminal in the current folder ^^ > > dolphin shouldn't allow this.. right? > > Konsole's desktop file has a key

Re: KaOS locale KCM (KDE 4 vs 5, regional settings and dolphin)

2016-12-07 Thread René J . V . Bertin
René J. V. Bertin wrote: > I just played with a full Plasma5 desktop (KaOS) and noticed that it has a > Regional Settings/Locale KCM I don't have on my own install, for as yet > unclear reasons. It seems it gives an additional step in the right direction Clear now:

Re: status of kde/plasma kiosk framework in kf5

2016-12-07 Thread Dennis Knorr
Am 07.12.2016 um 10:46 schrieb Kai Uwe Broulik: > Even if the entries still show up in the context menu when you restricted > them (which would be a bug you should report) kcmshell will still refuse to > open it, so it should be purely cosmetical then. > > I *think* the network editor, not

Re: Review Request 128038: [libtaskmanager] Stop highlighted window effect in group item

2016-12-07 Thread Anthony Fieroni
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128038/#review101325 --- You not against i plan to ship this patch next week. -

Re: KDE 4 vs 5, regional settings and dolphin

2016-12-07 Thread René J . V . Bertin
Christoph Feck wrote: > See bug 340982. > > KF5 applications do not use the customizable KLocale class, but use what > Qt offers (QLocale), which we hoped would soon get support for > customizations, but progress is slow, so no code has surfaced. I just played with a full Plasma5 desktop

Re: Selecting a Plasma logo

2016-12-07 Thread Jens Reuterberg
About half a year ago this thread started and so far no actual change has happened. The background is that the past logo for Plasma Desktop was disagreed upon by the Plasma developer team and they wanted a new one. Thomas created a competition of sorts which had a lot of suggestions in it.

plasma-nm build failure

2016-12-07 Thread Jonathan Riddell
plasma-nm has a build failure in stable today http://build.neon.kde.org/job/xenial_stable_plasma_plasma-nm_bin_amd64/56/console bug is https://bugs.kde.org/show_bug.cgi?id=363917 https://cgit.kde.org/plasma-nm.git/commit/?h=Plasma/5.8=a51e9c5c2a2c32ef8d2f354af33532cbc35bfa9e It doesn't seem to

[Differential] [Closed] D3603: Option to show percentage charge in the icon

2016-12-07 Thread mart (Marco Martin)
This revision was automatically updated to reflect the committed changes. Closed by commit R120:27ac0b760855: Option to show percentage charge in the icon (authored by mart). REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D3603?vs=8843=8844 REVISION

[Differential] [Updated, 177 lines] D3603: Option to show percentage charge in the icon

2016-12-07 Thread mart (Marco Martin)
mart updated this revision to Diff 8843. mart added a comment. - use anchors REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D3603?vs=8840=8843 BRANCH phab/batterypercent REVISION DETAIL https://phabricator.kde.org/D3603 AFFECTED FILES

[Differential] [Accepted] D3603: Option to show percentage charge in the icon

2016-12-07 Thread broulik (Kai Uwe Broulik)
broulik accepted this revision. broulik added a reviewer: broulik. broulik added inline comments. This revision is now accepted and ready to land. INLINE COMMENTS > BadgeOverlay.qml:31 > +id: badgeRect > +x: Qt.application.layoutDirection === Qt.RightToLeft ? 0 : > parent.width

[Differential] [Request, 2 lines] D3616: [Lock Screen / Login] Add "reveal password button"

2016-12-07 Thread broulik (Kai Uwe Broulik)
broulik created this revision. broulik added reviewers: Plasma, Plasma: Design. broulik set the repository for this revision to R120 Plasma Workspace. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY Allows to show the

[Differential] [Updated, 175 lines] D3603: Option to show percentage charge in the icon

2016-12-07 Thread mart (Marco Martin)
mart updated this revision to Diff 8840. mart added a comment. - add missing file REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D3603?vs=8802=8840 BRANCH phab/batterypercent REVISION DETAIL https://phabricator.kde.org/D3603 AFFECTED FILES

[Differential] [Commented On] D3603: Option to show percentage charge in the icon

2016-12-07 Thread mart (Marco Martin)
mart added a comment. F672027: Spectacle.T21326.png REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D3603 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: mart, #plasma Cc: broulik,

[Differential] [Commented On] D3603: Option to show percentage charge in the icon

2016-12-07 Thread broulik (Kai Uwe Broulik)
broulik added a comment. You forgot BadgeOverlay.qml INLINE COMMENTS > ConfigGeneral.qml:28 > + > +signal configurationChanged > + Unused > ConfigGeneral.qml:30-31 > + > +width: childrenRect.width > +height: childrenRect.height > +implicitWidth: pageColumn.implicitWidth

Re: status of kde/plasma kiosk framework in kf5

2016-12-07 Thread Marco Martin
On Wednesday 07 December 2016 10:46:29 Kai Uwe Broulik wrote: > Even if the entries still show up in the context menu when you restricted > them (which would be a bug you should report) kcmshell will still refuse to > open it, so it should be purely cosmetical then. > > I *think* the network

Re: status of kde/plasma kiosk framework in kf5

2016-12-07 Thread Kai Uwe Broulik
Hi Thomas, good to hear back from you! > - Menu for "Edit Applications"  in the launcher called "Anwendungsübersicht" > and "Anwendungsmenü" (its working in "Anwendungs-Starter") That was an oversight, I just uploaded a patch to fix this :) The others are just shortcuts to system settings

[Differential] [Request, 4 lines] D3615: [Kicker] Hide "Edit Applications..." context menu entry if system immutable

2016-12-07 Thread broulik (Kai Uwe Broulik)
broulik created this revision. broulik added reviewers: Plasma, hein. broulik set the repository for this revision to R119 Plasma Desktop. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY We don't want users in a