Add theming for keyframes in the timeline
Firstly, thanks for taking the initiative on this :)
So far it all looks good (I'll have to double check the DNA alignment). A few comments though:
- For the keyframes originating from the scene, you could use UI_ThemeColorShade to get a darker shade (like in the past)
- In resources.c, you also need to add some version patching code n init_userdef_do_versions() to set the new colours, or else in all old/saved configs, those colours will only appear black.
- Have you considered just reusing the keyframe theme colours (for the dopesheet) instead? They might not be initialised for the timeline though...
Thanks for reviewing so quickly :)
- Cool idea
- The problem I see in this is that since keyframes in the timeline are very thin lines, they need a highly contrasting color to stay visible. For instance, graph theme uses a dark grey for dope keyframes which works fine, but in the timeline they blend with the grid and background.
Would prefer to check adding color options to the User Preferences with the UI-team. Some weeks ago we made decision to not allow any new color options until we've found a proper solution for the mess in the theme settings (was communicated quite badly though).
Don't think it's worth reverting, just wanted to mention this ;)
Edit: @Diego Gangl (januz), don't want to sound ungrateful, I'm glad you worked on this, we just have to keep things organized