[Bf-committers] [Bf-blender-cvs] SVN commit: /data/svn/bf-blender [22408] branches/blender2.5/blender/source /blender/editors/space_time/time_header.c: Edited timeline header layout slightly.

[Joshua Leung]

Hi,

I haven't checked on this visual appearance of this yet (not at home atm),
but just a few notes about this first.

On Thu, Aug 13, 2009 at 3:55 AM, William Reynish <william at reynish.com>wrote:

> Revision: 22408
>
> http://projects.blender.org/plugins/scmsvn/viewcvs.php?view=rev&root=bf-blender&revision=22408
> Author:   billrey
> Date:     2009-08-12 17:55:15 +0200 (Wed, 12 Aug 2009)
>
> Log Message:
> -----------
> Edited timeline header layout slightly.
>
> -Made the 'PR' button not align itself to the start/end frame fields, as
> this makes for a nicer visual rounding of those two numbers.

Hmm... the main reason that this was aligned to those buttons was to show
the connection of that setting with the frame range. Otherwise, it'd look
more like a random button that maybe did something else.


> -Made current frame indicator longer, to allow for display of higher frame
> numbers

Nice :)


> -Removed the auto-keyframe dropdown menu. This exists as a preference, and
> doesn't need to take up space here. This is not something you're likely to
> change much after all.

Ahem! That menu was for the auto-keying mode used for the current scene,
which is the current default. The user-prefs version was really quite nasty,
and was really only used for initialising the scene one on new scenes
(unless the stupid toolsettings are still not saved yet). The timeline
header for now was the only place you could set this.


> -Made the keying sets menu stay put in the UI. It was moving about
> depending on what options are selected to the left of it. This sort of
> moving around should be avoided as much as possible.

Right.



> -Swapped Add/Delete keyframe buttons. It's more intuitive to have the add
> key button first.

For consistency, I'd recommend that you check on the Outliner header too ;)