Bug 408669 - Animation Curves Docker Frame Scrubbing Doesn't Update Visual Indicator
Summary: Animation Curves Docker Frame Scrubbing Doesn't Update Visual Indicator
Status: RESOLVED FIXED
Alias: None
Product: krita
Classification: Applications
Component: Animation (show other bugs)
Version: 4.2.1
Platform: Microsoft Windows Microsoft Windows
: NOR minor
Target Milestone: ---
Assignee: Scott Petrovic
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2019-06-13 22:39 UTC by T.R.O Nunes
Modified: 2020-03-03 22:14 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
As you can see here the frame number keeps changing while scrubbing while the indicator doesnt move (58.40 KB, image/gif)
2019-06-13 22:39 UTC, T.R.O Nunes
Details

Note You need to log in before you can comment on or make changes to this bug.
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.