You might consider using TH_HEADER_TEXT_HI for this instead of TH_TEXT_HI. The former is hardly ever used for anything. But the latter is used for a few things in that area. For example ANIM_channel_draw() uses that for selected channel text. marker_color_get() using it as well. Probably a few others as I didn't look too far. When there is overlap it could limit some design choices. Like you might want a red number but not want red for selected text elsewhere.

Another comment is more a curiosity. You are changing draw_current_frame() which looks to be the exact place if remember right. But there is also a very similar looking ANIM_draw_cfra_number() in editors\animation\anim_draw.c that has notes that it is for drawing current frame number too. I think it is deprecated code that is not called anywhere. I think this other one might have been forgotten when some of these animation and timeline editors were amalgamated.

In D7897#190804, @Harley Acheson (harley) wrote:

You might consider using TH_HEADER_TEXT_HI for this instead of TH_TEXT_HI. The former is hardly ever used for anything. But the latter is used for a few things in that area. For example ANIM_channel_draw() uses that for selected channel text. marker_color_get() using it as well. Probably a few others as I didn't look too far. When there is overlap it could limit some design choices. Like you might want a red number but not want red for selected text elsewhere.

That makes sense, I'll try using that. I agree that the "current frame indicator" is probably a special case.

Another comment is more a curiosity. You are changing draw_current_frame() which looks to be the exact place if remember right. But there is also a very similar looking ANIM_draw_cfra_number() in editors\animation\anim_draw.c that has notes that it is for drawing current frame number too. I think it is deprecated code that is not called anywhere. I think this other one might have been forgotten when some of these animation and timeline editors were amalgamated.

Haha, I noticed that too and made a patch to remove that code, it's totally dead (D7898).