[systemsettings] [Bug 392050] Applications > Launch Feedback > Cursor: effect does not save when changed

2018-03-28 Thread Kai Uwe Broulik
https://bugs.kde.org/show_bug.cgi?id=392050

Kai Uwe Broulik  changed:

   What|Removed |Added

 CC||e.t...@laposte.net

--- Comment #19 from Kai Uwe Broulik  ---
*** Bug 392435 has been marked as a duplicate of this bug. ***

-- 
You are receiving this mail because:
You are watching all bug changes.

[systemsettings] [Bug 392050] Applications > Launch Feedback > Cursor: effect does not save when changed

2018-03-20 Thread Eike Hein
https://bugs.kde.org/show_bug.cgi?id=392050

Eike Hein  changed:

   What|Removed |Added

 Status|CONFIRMED   |RESOLVED
  Latest Commit||https://commits.kde.org/pla
   ||sma-desktop/8af970fd49f3ca3
   ||3450cf3ccd81856694a0dc752
 Resolution|--- |FIXED

--- Comment #18 from Eike Hein  ---
Git commit 8af970fd49f3ca33450cf3ccd81856694a0dc752 by Eike Hein, on behalf of
Scott Harvey.
Committed on 21/03/2018 at 02:37.
Pushed by hein into branch 'master'.

KCM Launch Feedback not saving cursor settings

Reviewers: #plasma, hein, davidedmundson, ngraham

Reviewed By: #plasma, hein

Subscribers: davidedmundson, plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D11522

M  +5-4kcms/launch/package/contents/ui/main.qml

https://commits.kde.org/plasma-desktop/8af970fd49f3ca33450cf3ccd81856694a0dc752

-- 
You are receiving this mail because:
You are watching all bug changes.

[systemsettings] [Bug 392050] Applications > Launch Feedback > Cursor: effect does not save when changed

2018-03-20 Thread Scott Harvey
https://bugs.kde.org/show_bug.cgi?id=392050

--- Comment #17 from Scott Harvey  ---
(In reply to Eike Hein from comment #8)
> > Changes do take effect after logging out and back in. A simple restart of 
> > plasmashell isn't sufficient.
> 
> That means KWin isn't watching launchrc for changes.

I can retract this portion, now that klaunchrc is being updated. KWin appears
to catch it and make a change. Although "blink" doesn't seem to blink - it's
static instead, at least on my system. I *can* change from "none" to "bounce"
and KWin correctly makes the change.

-- 
You are receiving this mail because:
You are watching all bug changes.

[systemsettings] [Bug 392050] Applications > Launch Feedback > Cursor: effect does not save when changed

2018-03-20 Thread Scott Harvey
https://bugs.kde.org/show_bug.cgi?id=392050

--- Comment #16 from Scott Harvey  ---
Is my first test getting your website to serve that .txt file instead of
redirecting to your logo page?  ;-)

-- 
You are receiving this mail because:
You are watching all bug changes.

[systemsettings] [Bug 392050] Applications > Launch Feedback > Cursor: effect does not save when changed

2018-03-20 Thread Eike Hein
https://bugs.kde.org/show_bug.cgi?id=392050

--- Comment #15 from Eike Hein  ---
We all were at some point :D

-- 
You are receiving this mail because:
You are watching all bug changes.

[systemsettings] [Bug 392050] Applications > Launch Feedback > Cursor: effect does not save when changed

2018-03-20 Thread Scott Harvey
https://bugs.kde.org/show_bug.cgi?id=392050

--- Comment #14 from Scott Harvey  ---
You're a champ. Thanks for helping a rookie.

-- 
You are receiving this mail because:
You are watching all bug changes.

[systemsettings] [Bug 392050] Applications > Launch Feedback > Cursor: effect does not save when changed

2018-03-20 Thread Eike Hein
https://bugs.kde.org/show_bug.cgi?id=392050

--- Comment #13 from Eike Hein  ---
Best way to go is:

a) Clone plasma-desktop
b) When you run cmake set a custom install prefix somewhere in your $HOME
c) Before you run "make" and "make install" in the build dir, cd into
kcms/launch to only build that one module instead of all of plasma-desktop
d) Modify https://www.eikehein.com/kdedev.txt to reflect the custom prefix you
personally chose and stick it as somethingsomething.sh into your
/etc/profile.d/ (or whatever you prefer to set env vars for your session) - go
over this file carefully, as some of the baked-in paths there might be
different on your distro or hw
e) Log out of Plasma and back in

System Settings will now take your self-built module over the system-level one.

-- 
You are receiving this mail because:
You are watching all bug changes.

[systemsettings] [Bug 392050] Applications > Launch Feedback > Cursor: effect does not save when changed

2018-03-20 Thread Scott Harvey
https://bugs.kde.org/show_bug.cgi?id=392050

--- Comment #12 from Scott Harvey  ---
Yep, I got that far.

I'm more puzzled about how to recompile just this one KCM module and stick it
back into my system. Without destroying things, of course.

This is my first take on modifying a live part of Plasma, as opposed to one of
the applications.

-- 
You are receiving this mail because:
You are watching all bug changes.

[systemsettings] [Bug 392050] Applications > Launch Feedback > Cursor: effect does not save when changed

2018-03-20 Thread Eike Hein
https://bugs.kde.org/show_bug.cgi?id=392050

--- Comment #11 from Eike Hein  ---
I can give you a general low-down of how it works, or is supposed to: In
response to clicking around in the UI, the QML code in main.qml sets the new
setting value in the C++ backend object (written down in launchfeedback.cpp).
The C++ code receives the new value, and then updateNeedsSaving() compares it
against the saved value and figures out whether it needs to mark the System
Settings module as "needs saving". While a module "needs saving" the Apply
button is enabled.

Most likely there's a logic bug in updateNeedsSaving().

-- 
You are receiving this mail because:
You are watching all bug changes.

[systemsettings] [Bug 392050] Applications > Launch Feedback > Cursor: effect does not save when changed

2018-03-20 Thread Scott Harvey
https://bugs.kde.org/show_bug.cgi?id=392050

--- Comment #10 from Scott Harvey  ---
(In reply to Eike Hein from comment #9)
> Scott, do you still want to fix the KCM code for the Apply button?

I'll give it a try. Sorry I didn't work on it more yesterday, been sick. Need
to sort out how to test this on a living, breathing Plasma installation. Any
brief advice? (Like I said, been sick - logic circuits are running slow.)

-- 
You are receiving this mail because:
You are watching all bug changes.

[systemsettings] [Bug 392050] Applications > Launch Feedback > Cursor: effect does not save when changed

2018-03-20 Thread Eike Hein
https://bugs.kde.org/show_bug.cgi?id=392050

--- Comment #8 from Eike Hein  ---
> Changes do take effect after logging out and back in. A simple restart of 
> plasmashell isn't sufficient.

That means KWin isn't watching launchrc for changes.

-- 
You are receiving this mail because:
You are watching all bug changes.

[systemsettings] [Bug 392050] Applications > Launch Feedback > Cursor: effect does not save when changed

2018-03-20 Thread Eike Hein
https://bugs.kde.org/show_bug.cgi?id=392050

--- Comment #9 from Eike Hein  ---
Scott, do you still want to fix the KCM code for the Apply button?

-- 
You are receiving this mail because:
You are watching all bug changes.

[systemsettings] [Bug 392050] Applications > Launch Feedback > Cursor: effect does not save when changed

2018-03-19 Thread Scott Harvey
https://bugs.kde.org/show_bug.cgi?id=392050

--- Comment #7 from Scott Harvey  ---
Changes do take effect after logging out and back in. A simple restart of
plasmashell isn't sufficient.

-- 
You are receiving this mail because:
You are watching all bug changes.

[systemsettings] [Bug 392050] Applications > Launch Feedback > Cursor: effect does not save when changed

2018-03-19 Thread Scott Harvey
https://bugs.kde.org/show_bug.cgi?id=392050

--- Comment #6 from Scott Harvey  ---
The problem appears to run deeper than a non-saving dialog. The bouncing cursor
doesn't seem to obey what's set in $HOME/.config/klaunchrc, even if set
manually (which I did).

No matter what values are in klaunchrc, the cursor still shows a bouncing icon. 

In the screencast, you can see that I've manually changed the values to get "No
Feedback" as the active option. But when I launch Handbrake (which takes long
enough to start), I still get a bouncing icon cursor.

Input, please?

-- 
You are receiving this mail because:
You are watching all bug changes.

[systemsettings] [Bug 392050] Applications > Launch Feedback > Cursor: effect does not save when changed

2018-03-19 Thread Scott Harvey
https://bugs.kde.org/show_bug.cgi?id=392050

--- Comment #5 from Scott Harvey  ---
Created attachment 111513
  --> https://bugs.kde.org/attachment.cgi?id=111513&action=edit
Screencast of bouncing cursor regardless of klaunchrc

-- 
You are receiving this mail because:
You are watching all bug changes.

[systemsettings] [Bug 392050] Applications > Launch Feedback > Cursor: effect does not save when changed

2018-03-19 Thread Scott Harvey
https://bugs.kde.org/show_bug.cgi?id=392050

--- Comment #4 from Scott Harvey  ---
Thanks, Nate.

The code, like all good code, appears straightforward enough. I'll see if I can
chase it down. It might not be as simple as I'd like... after all, this was
written by the famous (yet helpful!) Sho  :)  I don't think many bugs escape
his keyboard.

-- 
You are receiving this mail because:
You are watching all bug changes.

[systemsettings] [Bug 392050] Applications > Launch Feedback > Cursor: effect does not save when changed

2018-03-19 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=392050

--- Comment #3 from Nate Graham  ---
If you want to have a go, that would be excellent! The code is in
plasma-desktop:

https://cgit.kde.org/plasma-desktop.git/tree/kcms/launch

-- 
You are receiving this mail because:
You are watching all bug changes.

[systemsettings] [Bug 392050] Applications > Launch Feedback > Cursor: effect does not save when changed

2018-03-19 Thread Scott Harvey
https://bugs.kde.org/show_bug.cgi?id=392050

--- Comment #2 from Scott Harvey  ---
This might be something I'm able to fix... just as soon as I can find out where
it lives. (*grumbles at impressive scope and scale of project)

-- 
You are receiving this mail because:
You are watching all bug changes.

[systemsettings] [Bug 392050] Applications > Launch Feedback > Cursor: effect does not save when changed

2018-03-19 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=392050

Nate Graham  changed:

   What|Removed |Added

 Ever confirmed|0   |1
 Status|UNCONFIRMED |CONFIRMED
   Assignee|plasma-b...@kde.org |h...@kde.org
 CC||h...@kde.org

--- Comment #1 from Nate Graham  ---
Can confirm. After changing the Cursor setting, the "Apply" button does not
take effect.

Changing any other setting *does* result in the Apply button being activated,
and the change can be saved.

Even if I change another setting to activate the Apply button, and then change
the Cursor setting, the new Cursor setting is still not saved after hitting the
Apply button.

Looks like this was likely fallout from the recent QML port. Eike?

-- 
You are receiving this mail because:
You are watching all bug changes.

[systemsettings] [Bug 392050] Applications > Launch Feedback > Cursor: effect does not save when changed

2018-03-19 Thread Scott Harvey
https://bugs.kde.org/show_bug.cgi?id=392050

Scott Harvey  changed:

   What|Removed |Added

 CC||n...@kde.org

-- 
You are receiving this mail because:
You are watching all bug changes.