[Powerdevil] [Bug 350676] Need to press key twice to decrease brightness

2015-09-15 Thread Weng Xuetian
https://bugs.kde.org/show_bug.cgi?id=350676 --- Comment #16 from Weng Xuetian --- (In reply to Igor Poboiko from comment #15) > (In reply to Weng Xuetian from comment #14) > > FYI, https://git.reviewboard.kde.org/r/125182/ > What's the status of that patch? > That check was there from very begin

[Powerdevil] [Bug 350676] Need to press key twice to decrease brightness

2015-09-15 Thread Igor Poboiko
https://bugs.kde.org/show_bug.cgi?id=350676 --- Comment #15 from Igor Poboiko --- (In reply to Weng Xuetian from comment #12) > Disable vsync helps me to workaround the issue, you may try that to see if > it works for you.. No, unfortunately it didn't. (In reply to Weng Xuetian from comment #14)

[Powerdevil] [Bug 350676] Need to press key twice to decrease brightness

2015-09-11 Thread Weng Xuetian
https://bugs.kde.org/show_bug.cgi?id=350676 --- Comment #14 from Weng Xuetian --- (In reply to Igor Poboiko from comment #11) > BTW, another issue related to animation routine: all keypresses that are > happening during animation are ignored (due to the very same check it didn't > update the cach

[Powerdevil] [Bug 350676] Need to press key twice to decrease brightness

2015-09-11 Thread Weng Xuetian
https://bugs.kde.org/show_bug.cgi?id=350676 Weng Xuetian changed: What|Removed |Added Resolution|--- |FIXED Version Fixed In|

[Powerdevil] [Bug 350676] Need to press key twice to decrease brightness

2015-09-11 Thread Weng Xuetian
https://bugs.kde.org/show_bug.cgi?id=350676 --- Comment #12 from Weng Xuetian --- (In reply to Kai Uwe Broulik from comment #10) > Could it be that you're using EGL? Does the highlight in eg. Kickoff also > only update every second item? If so, that's a driver issue. So glad to learn that here a

[Powerdevil] [Bug 350676] Need to press key twice to decrease brightness

2015-09-11 Thread Igor Poboiko
https://bugs.kde.org/show_bug.cgi?id=350676 --- Comment #11 from Igor Poboiko --- (In reply to Kai Uwe Broulik from comment #10) > Could it be that you're using EGL? Does the highlight in eg. Kickoff also > only update every second item? If so, that's a driver issue. Yeah, same in kickoff. Thank

[Powerdevil] [Bug 350676] Need to press key twice to decrease brightness

2015-09-11 Thread Kai Uwe Broulik
https://bugs.kde.org/show_bug.cgi?id=350676 --- Comment #10 from Kai Uwe Broulik --- Could it be that you're using EGL? Does the highlight in eg. Kickoff also only update every second item? If so, that's a driver issue. -- You are receiving this mail because: You are the assignee for the bug. _

[Powerdevil] [Bug 350676] Need to press key twice to decrease brightness

2015-09-11 Thread Igor Poboiko
https://bugs.kde.org/show_bug.cgi?id=350676 --- Comment #9 from Igor Poboiko --- Funny, although after fixing this brightness gets updated correctly, OSD doesn't. It still gets updated each second keypress, but seems like it is another issue somewhere in OSD/QML code. Somehow ProgressBar value (w

[Powerdevil] [Bug 350676] Need to press key twice to decrease brightness

2015-09-11 Thread Weng Xuetian
https://bugs.kde.org/show_bug.cgi?id=350676 --- Comment #8 from Weng Xuetian --- https://git.reviewboard.kde.org/r/125156/ -- You are receiving this mail because: You are the assignee for the bug. ___ Plasma-devel mailing list Plasma-devel@kde.org htt

[Powerdevil] [Bug 350676] Need to press key twice to decrease brightness

2015-09-11 Thread Weng Xuetian
https://bugs.kde.org/show_bug.cgi?id=350676 --- Comment #7 from Weng Xuetian --- Well, imho, such animation is more or less the standard behavior in real world (Windows, Android, ...). Though from reading our implementation there are definitely some issue here. stop will not reset the progress,

[Powerdevil] [Bug 350676] Need to press key twice to decrease brightness

2015-09-11 Thread Igor Poboiko
https://bugs.kde.org/show_bug.cgi?id=350676 --- Comment #6 from Igor Poboiko --- Created attachment 94505 --> https://bugs.kde.org/attachment.cgi?id=94505&action=edit patch that removes QPropertyAnimation I would prefer removing animation routine at all. I mean, why do we need it? It just make

[Powerdevil] [Bug 350676] Need to press key twice to decrease brightness

2015-09-11 Thread Igor Poboiko
https://bugs.kde.org/show_bug.cgi?id=350676 --- Comment #5 from Igor Poboiko --- I did some investigation on this problem by adding few debug lines, and I nailed it. What is happening here is following: 1. User presses DecreaseBrightness button first time. - UPower backend starts an QPropertyAni

[Powerdevil] [Bug 350676] Need to press key twice to decrease brightness

2015-09-10 Thread Weng Xuetian
https://bugs.kde.org/show_bug.cgi?id=350676 Weng Xuetian changed: What|Removed |Added CC||wen...@gmail.com --- Comment #4 from Weng Xuetia

[Powerdevil] [Bug 350676] Need to press key twice to decrease brightness

2015-09-08 Thread Bhushan Shah
https://bugs.kde.org/show_bug.cgi?id=350676 Bhushan Shah changed: What|Removed |Added CC||igor.pobo...@gmail.com --- Comment #3 from Bhush

[Powerdevil] [Bug 350676] Need to press key twice to decrease brightness

2015-08-27 Thread Martin Klapetek
https://bugs.kde.org/show_bug.cgi?id=350676 --- Comment #2 from Martin Klapetek --- Max Screen Brightness for me says 100. -- You are receiving this mail because: You are the assignee for the bug. ___ Plasma-devel mailing list Plasma-devel@kde.org htt

[Powerdevil] [Bug 350676] Need to press key twice to decrease brightness

2015-08-27 Thread Martin Klapetek
https://bugs.kde.org/show_bug.cgi?id=350676 Martin Klapetek changed: What|Removed |Added CC||mklape...@kde.org --- Comment #1 from Martin

[Powerdevil] [Bug 350676] Need to press key twice to decrease brightness

2015-07-27 Thread Kai Uwe Broulik
https://bugs.kde.org/show_bug.cgi?id=350676 Kai Uwe Broulik changed: What|Removed |Added CC||k...@privat.broulik.de Ever confirmed|0