Re: Plasma Vault

2017-05-22 Thread Sebastian Kügler
On maandag 22 mei 2017 19:08:48 CEST Ivan Čukić wrote: > Thanks for this. I had problems with the docker image, so I succeeded > in reproducing the issue in VBox. > > I'll push the fix soon. Mind that it will still not work in Neon (old > version of encfs and no version of cryfs in main repos), bu

[kio-extras] [Bug 376344] cant write to smb shares which has write access

2017-05-22 Thread Bronson
https://bugs.kde.org/show_bug.cgi?id=376344 --- Comment #26 from Bronson --- (In reply to madcatx from comment #25) > Created attachment 105680 [details] > Allow write access to root samba share Id be happy to try this, but I dont know how to make the kio-extras package (https://github.com/KDE/k

[kio-extras] [Bug 376344] cant write to smb shares which has write access

2017-05-22 Thread bugzilla_noreply
https://bugs.kde.org/show_bug.cgi?id=376344 --- Comment #25 from madc...@atlas.cz --- Created attachment 105680 --> https://bugs.kde.org/attachment.cgi?id=105680&action=edit Allow write access to root samba share -- You are receiving this mail because: You are the assignee for the bug.

[kio-extras] [Bug 376344] cant write to smb shares which has write access

2017-05-22 Thread bugzilla_noreply
https://bugs.kde.org/show_bug.cgi?id=376344 --- Comment #24 from madc...@atlas.cz --- Getting any debug output from kio_smb has proven to be next to impossible so I took a bit of an educated guess. Can you guys try the attached patch, it seems to work for me. -- You are receiving this mail becau

D5932: KCM for controlling Night Color

2017-05-22 Thread Roman Gilg
subdiff added inline comments. INLINE COMMENTS > lueck wrote in CMakeLists.txt:2 > The kcm loads a translation catalog, but where is the Messages.sh extracting > the strings into the catalog "kcm_nightcolor"? Ah, that's what these mysterious Messages.sh files everywhere are for. ;) Is the Mess

D5932: KCM for controlling Night Color

2017-05-22 Thread Burkhard Lück
lueck added inline comments. INLINE COMMENTS > CMakeLists.txt:2 > +# KI18N Translation Domain for this library > +add_definitions(-DTRANSLATION_DOMAIN=\"kcm_nightcolor\") > + The kcm loads a translation catalog, but where is the Messages.sh extracting the strings into the catalog "kcm_nightcolo

D5927: improve positioning for highlight in systray

2017-05-22 Thread Michail Vourlakos
mvourlakos added a comment. @mart the last commit fixes the multirow behavior and also this topic (fixed issue of line positioning) F3760696: b1.png (fixed issue of multirow behavior from initial commit and behavior is identical to master)

D5927: improve positioning for highlight in systray

2017-05-22 Thread Michail Vourlakos
mvourlakos updated this revision to Diff 14766. mvourlakos added a comment. Updating https://phabricator.kde.org/D5927: improve positioning for highlight in systray --fixed the issue mentioned for multirow case REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE https://phabr

D5926: Remove filename from source/dest fields in job notification

2017-05-22 Thread Drosera Sprout
droserasprout added a comment. In https://phabricator.kde.org/D5926#111001, @davidedmundson wrote: > Why? This way it's easier to track progress of jobs with long paths IMO. REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D5926 To: droserasprout,

[Powerdevil] [Bug 350365] Battery monitor in tray randomly shows there is no battery

2017-05-22 Thread Tim Taylor
https://bugs.kde.org/show_bug.cgi?id=350365 --- Comment #17 from Tim Taylor --- I still have this problem with Kubuntu 17.04, running KDE Frameworks 5.31.0 and Plasma 5.9.4 :-( -- You are receiving this mail because: You are the assignee for the bug.

[kio-extras] [Bug 366795] Paste action disabled when copying a local file to smb:// or mtp://

2017-05-22 Thread bugzilla_noreply
https://bugs.kde.org/show_bug.cgi?id=366795 johan.claudebreunin...@gmail.com changed: What|Removed |Added CC||johan.claudebreuninger@gmai

Re: Plasma Vault

2017-05-22 Thread Ivan Čukić
Thanks for this. I had problems with the docker image, so I succeeded in reproducing the issue in VBox. I'll push the fix soon. Mind that it will still not work in Neon (old version of encfs and no version of cryfs in main repos), but at least it will print out the proper message. Cheers, Ivan

D5932: KCM for controlling Night Color

2017-05-22 Thread David Edmundson
davidedmundson added a comment. >> Make sure you test with --reverse, I think your activator checkbox will be broken > > You're right, but I'm just using the text attribute of QQC's CheckBox, so I'm not sure if I should write a workaround here or just wait for upstream to fix it. The vi

D5913: Fix fifteen puzzle solveability

2017-05-22 Thread Igor Poboiko
This revision was automatically updated to reflect the committed changes. Closed by commit R114:3516eda8107b: Fix fifteen puzzle solveability (authored by poboiko). REPOSITORY R114 Plasma Addons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D5913?vs=14678&id=14763 REVISION DETAIL

[Powerdevil] [Bug 378445] kscreenlocker_greet occasionally freezes in malloc in signal handler

2017-05-22 Thread bugzilla_noreply
https://bugs.kde.org/show_bug.cgi?id=378445 --- Comment #16 from r...@alum.mit.edu --- Happened again in charge_state_changed. This *may* have been at the time my laptop reached full charge but I have no logs. -- You are receiving this mail because: You are the assignee for the bug.

D5937: No longer build Plasmate by default

2017-05-22 Thread Friedrich W. H. Kossebau
kossebau retitled this revision from "No longer built Plasmate by default" to "No longer build Plasmate by default". REPOSITORY R118 Plasma SDK REVISION DETAIL https://phabricator.kde.org/D5937 To: kossebau, #plasma, sandroandrade, jriddell Cc: plasma-devel, ZrenBot, spstarr, progwolff, les

D5937: No longer built Plasmate by default

2017-05-22 Thread Friedrich W. H. Kossebau
kossebau added a comment. IMHO Plasmate should not be released for default build also with Plasma 5.10. Distribution packagers seems to just look if things built, but not test the functionality, at least when it comes to Plasmate. E.g. for openSUSE plasmate is part of the package plasma-sd

D5937: No longer built Plasmate by default

2017-05-22 Thread Friedrich W. H. Kossebau
kossebau created this revision. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY Does not work currently and needs a maintainer. REPOSITORY R118 Plasma SDK BRANCH dontbuildplasmatebydefault REVISION DETAIL https://

D5748: workaround for correct wayland positioning

2017-05-22 Thread Marco Martin
mart added a comment. In https://phabricator.kde.org/D5748#110388, @davidedmundson wrote: > edit, ignore that last comment. I hadn't seen it was a QPointer. > > Just my comment about checking on all other plaforms remains. i could check on wayland Qt 5.9, X Qt 5.8, both seemed

[kio-extras] [Bug 376344] cant write to smb shares which has write access

2017-05-22 Thread bugzilla_noreply
https://bugs.kde.org/show_bug.cgi?id=376344 madc...@atlas.cz changed: What|Removed |Added CC||madc...@atlas.cz --- Comment #23 from madc...

D5933: We should not forget the launchers that are not currently shown

2017-05-22 Thread Ivan Čukić
This revision was automatically updated to reflect the committed changes. Closed by commit R120:e8e2be90834b: We should not forget the launchers that are not currently shown (authored by ivan). REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D5933?vs=14

Re: Broken dependencies: plasma-vault

2017-05-22 Thread Ben Cooksley
On Sat, May 20, 2017 at 8:38 AM, Ivan Čukić wrote: > Seems I updated the wrong file (the one for qt4 which is strangely > called just dependency-data), thanks Marco for applying the changes to > proper files. Thanks Ivan and Marco for sorting that out. Aside from xdg-desktop-portal-kde all of Pla

D5933: We should not forget the launchers that are not currently shown

2017-05-22 Thread Eike Hein
hein accepted this revision. hein added a comment. This revision is now accepted and ready to land. Yay! REPOSITORY R120 Plasma Workspace BRANCH master REVISION DETAIL https://phabricator.kde.org/D5933 To: ivan, hein, mart Cc: plasma-devel, ZrenBot, spstarr, progwolff, lesliezhai, ali

Re: Plasma Vault

2017-05-22 Thread Jonathan Riddell
It happens with a Neon Docker image too http://embra.edinburghlinux.co.uk/~jr/tmp/vault.png https://community.kde.org/Neon/Docker follow ths instructions in the yellow boxes to get a dev unstable docker image running inside the docker image nested Plasma shell run apt install plasma-vault and

Re: Plasma Vault

2017-05-22 Thread Ivan Čukić
Do you have a VM or something I can log into like we did before so that I can try to see what is going on? (I'm using vault every day, and I haven't had any problems so far) Cheers, Ivan -- KDE, ivan.cu...@kde.org, http://cukic.co/ gpg key fingerprint: 292F 9B5C 5A1B 2A2F 9CF3 45DF C9C5 77AF

D5927: improve positioning for highlight in systray

2017-05-22 Thread Marco Martin
mart added a comment. In https://phabricator.kde.org/D5927#59, @mvourlakos wrote: > In https://phabricator.kde.org/D5927#40, @mart wrote: > > > this will break for multi-line systrays > > > makes sense, in multi line systray the active applet shouldnt be indicated correc

D5932: KCM for controlling Night Color

2017-05-22 Thread Marco Martin
mart added a comment. as far i seen, most qqc1 checkboxes around seem to have this problem REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D5932 To: subdiff, #plasma Cc: mart, davidedmundson, plasma-devel, ZrenBot, spstarr, progwolff, lesliezhai, ali-mohamed,

D5927: improve positioning for highlight in systray

2017-05-22 Thread Michail Vourlakos
mvourlakos added a comment. In https://phabricator.kde.org/D5927#40, @mart wrote: > this will break for multi-line systrays makes sense, in multi line systray the active applet shouldnt be indicated correctly... I can follow two paths, 1. stinguish the multi line cas

Re: Plasma Vault

2017-05-22 Thread Jonathan Riddell
On Mon, May 22, 2017 at 11:04:40AM +0200, Ivan Čukić wrote: > Do you have bustle to log the dbus events so that I can see what is happening? Here is the single dbus call that is made when I click Create New Vault button on the the widget embra.edinburghlinux.co.uk/~jr/tmp/vault-2017-05-22.bustl

D5932: KCM for controlling Night Color

2017-05-22 Thread Roman Gilg
subdiff added inline comments. INLINE COMMENTS > davidedmundson wrote in kcm_nightcolor.desktop:17 > FYI, adding translations here is a waste of time > > Scripty will ignore this when it extracts the text; then they'll be replaced > when it puts the translations back Ok, will omit it next time

D5932: KCM for controlling Night Color

2017-05-22 Thread Roman Gilg
subdiff updated this revision to Diff 14753. subdiff marked 4 inline comments as done. subdiff added a comment. Diff update: - Fixed inline comments - Fixed @mart's comment - Labels aligned to the right - Slight renaming of labels ("Sunrise/set ends" -> "and ends") > Make sure y

D5933: We should not forget the launchers that are not currently shown

2017-05-22 Thread Marco Martin
mart added a comment. +1 REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D5933 To: ivan, hein, mart Cc: plasma-devel, ZrenBot, spstarr, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, lukas

Re: [plasma-devel] Re: Plasma Vault

2017-05-22 Thread Ivan Čukić
Do you have bustle to log the dbus events so that I can see what is happening? Cheers, Ivan

D5932: KCM for controlling Night Color

2017-05-22 Thread Marco Martin
mart added a comment. small comment of the layout: align the labels (sunset begins etc) to the right instead of the left REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D5932 To: subdiff, #plasma Cc: mart, davidedmundson, plasma-devel, ZrenBot, spstarr, progwo

D5927: improve positioning for highlight in systray

2017-05-22 Thread Marco Martin
mart requested changes to this revision. mart added a comment. This revision now requires changes to proceed. can you make the panel taller and send a screenshot of how it looks then with different items highlighted? REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kd

D5927: improve positioning for highlight in systray

2017-05-22 Thread Marco Martin
mart added a comment. this will break for multi-line systrays REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D5927 To: mvourlakos, #plasma, mart, davidedmundson Cc: plasma-devel, ZrenBot, spstarr, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts,

D5913: Fix fifteen puzzle solveability

2017-05-22 Thread Marco Martin
mart accepted this revision. This revision is now accepted and ready to land. REPOSITORY R114 Plasma Addons REVISION DETAIL https://phabricator.kde.org/D5913 To: poboiko, #plasma, mart Cc: plasma-devel, davidedmundson, ZrenBot, spstarr, progwolff, lesliezhai, ali-mohamed, jensreuterberg, ab

D5918: set transientparent for menus

2017-05-22 Thread Marco Martin
mart added a comment. In https://phabricator.kde.org/D5918#110833, @davidedmundson wrote: > Please test under x. no noticeable difference in x REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D5918 To: mart, #plasma, davidedmundson Cc: d

D5918: set transientparent for menus

2017-05-22 Thread Marco Martin
This revision was automatically updated to reflect the committed changes. Closed by commit R242:5ba1bed5823b: set transientparent for menus (authored by mart). REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D5918?vs=14692&id=14752 REVISION D

D5933: We should not forget the launchers that are not currently shown

2017-05-22 Thread Ivan Čukić
ivan added a comment. This is meant to fix the https://bugs.kde.org/show_bug.cgi?id=379414 Currently, the ordering of pinned tasks is not independent between activities - that would require more intrusive changes not really desired this late in the release cycle (changes to the configura

D5933: We should not forget the launchers that are not currently shown

2017-05-22 Thread Ivan Čukić
ivan created this revision. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY The pinned launchers that are assigned to other activities will have a -1 as the order. We should not forget them when saving to the configura

D5932: KCM for controlling Night Color

2017-05-22 Thread David Edmundson
davidedmundson added a comment. looks mostly good. Make sure you test with --reverse, I think your activator checkbox will be broken INLINE COMMENTS > kcm_nightcolor.desktop:17 > +Name=Night Color > +Name[de]=Nachtfarbe > +Comment=Adjust color temperatur at night to reduce eye strain