[kwin] [Bug 441308] Activation feels too slow

2021-08-26 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=441308

Nate Graham  changed:

   What|Removed |Added

 Resolution|--- |FIXED
 Status|ASSIGNED|RESOLVED
  Latest Commit||https://invent.kde.org/plas
   ||ma/kwin/commit/205fb923cb8e
   ||1cbc0d26edc5a2ce37784ac32b4
   ||1
   Version Fixed In||5.23

--- Comment #9 from Nate Graham  ---
Git commit 205fb923cb8e1cbc0d26edc5a2ce37784ac32b41 by Nate Graham.
Committed on 26/08/2021 at 16:15.
Pushed by ngraham into branch 'master'.

[effects/overview] Speed up animation

The current duration is 300ms, which is inappropriate for two reasons:

1. It's too slow
2. It doesn't match one of the standard durations. The effect should
   ideally use those durations directly, but if this is not possible
   for technical reasons, it should use the effective value of one of
   those durations. Units.longDuration is 200ms which is the most
   appropriate one to use here.

Therefore, change the animation duration from 300ms to 200ms to match
the standard Units.longDuration duration.
FIXED-IN: 5.23

M  +1-1src/effects/overview/overvieweffect.cpp
M  +1-1src/effects/overview/overvieweffect.h

https://invent.kde.org/plasma/kwin/commit/205fb923cb8e1cbc0d26edc5a2ce37784ac32b41

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

[kwin] [Bug 441308] Activation feels too slow

2021-08-26 Thread Bug Janitor Service
https://bugs.kde.org/show_bug.cgi?id=441308

Bug Janitor Service  changed:

   What|Removed |Added

 Status|CONFIRMED   |ASSIGNED

--- Comment #8 from Bug Janitor Service  ---
A possibly relevant merge request was started @
https://invent.kde.org/plasma/kwin/-/merge_requests/1312

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

[kwin] [Bug 441308] Activation feels too slow

2021-08-24 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=441308

--- Comment #7 from Nate Graham  ---
I guess that would explain it. Any reason why?

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

[kwin] [Bug 441308] Activation feels too slow

2021-08-24 Thread David Edmundson
https://bugs.kde.org/show_bug.cgi?id=441308

--- Comment #6 from David Edmundson  ---
>Maybe it's just a bit too slow...

It is currently set to a much higher value than the old effect

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

[kwin] [Bug 441308] Activation feels too slow

2021-08-24 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=441308

Nate Graham  changed:

   What|Removed |Added

 Ever confirmed|0   |1
 Status|REPORTED|CONFIRMED

--- Comment #5 from Nate Graham  ---
Indeed. The duration is 300, which simply matches the existing hardcoded
duration of the old Present Windows effect. This should be using a standard
PlasmaCore.Units duration value, such as longDuration (200ms). Now it all makes
sense for why it feels slow since the ease-in+ease-out behavior is calibrated
for a fast overall duration, not a slow one. So the old 300ms with no ease-in
feels faster than 300ms with an ease-in+ease-out. 200ms with an
ease-in+ease-out feels a lot better in my testing.

I'll work on a patch.

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

[kwin] [Bug 441308] Activation feels too slow

2021-08-24 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=441308

Nate Graham  changed:

   What|Removed |Added

Summary|Animation curve makes it|Activation feels too slow
   |feel slow   |
   Severity|normal  |minor

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