Bug 408669

Summary: Animation Curves Docker Frame Scrubbing Doesn't Update Visual Indicator
Product: [Applications] krita Reporter: T.R.O Nunes <3rton93>
Component: AnimationAssignee: Scott Petrovic <scottpetrovic>
Status: RESOLVED FIXED    
Severity: minor CC: eoinoneill1991, griffinvalley, scottpetrovic
Priority: NOR    
Version: 4.2.1   
Target Milestone: ---   
Platform: Microsoft Windows   
OS: Microsoft Windows   
Latest Commit: Version Fixed In:
Attachments: As you can see here the frame number keeps changing while scrubbing while the indicator doesnt move

Description T.R.O Nunes 2019-06-13 22:39:06 UTC
Created attachment 120853 [details]
As you can see here the frame number keeps changing while scrubbing while the indicator doesnt move

SUMMARY
In animation curves docker when you drag the top bar frame slider the current frame will change, however, the visual indicator does not get dragged but rather updates when editor is changed.


STEPS TO REPRODUCE
1. Open 'Animation' and 'Animation Curves' dockers
2. In 'Animation Curves' scrub the timeline above the curves
3. Change frame in 'Animation' docker 

OBSERVED RESULT
In 'Animation' docker we can see our frame change, however, in 'Animation Curves' docker our frame indicator does not change until frame is updated from other dockers.

EXPECTED RESULT
In 'Animation Curves' docker visual indicator scrubs with the frame number

SOFTWARE/OS VERSIONS
Windows: 10 PRO Ver 1803 Build 17134.829
Krita 4.2.1
ADDITIONAL INFORMATION
Comment 1 wolthera 2019-09-21 16:23:45 UTC
Yeah, I can confirm this here in 4.3 pre-alpha.
Comment 2 Scott Petrovic 2019-09-23 13:19:59 UTC
I started doing some fixes to it that are in this merge request:
https://invent.kde.org/kde/krita/merge_requests/56

The main blocker for this merge request right now, I think, is fixing the animation cache. The animation cache doesn't handle the opacity tweening stuff very well and shows old frames. We need to get that figured out first before this can be merged.
Comment 3 Eoin O'Neill 2020-03-03 22:14:20 UTC
Scott's PR has now been merged into master.