The cursor for fly and walk navigation modes is now themable under user preferences.
Don't you have to also change the python file for users to set the color?
follow the existent style (no extra white line), and leave a , in the end if it works.
cpack(0) needs to leave, it was doing glColor3ub(0,0,0)
same here, see above note
leave this pad for a datablock in the future, I would recommend adding the view_navigation_cursor replacing another pad. (the variables are kind of grouped by themes here). no big deal though
description could be: "View Navigation Cursor"
Isn't char 8bits? Why does that structure need padding to be 8 aligned if it is all chars?
I put it here because it made sense to me next to camera. In the UI, I also put this after camera (and in all code files).
Not really keen to add this, there are a few places in blender where a default color is used for the 3d view...
- Camera guides (use wire color)
- 3D cursor crosshair (hard coded to black)
- Fly mode (hard coded to black)
Think it may be better to have a more general color which is used for viewport hints then adding a theme color for each.
So, I should also add options for '3D Cursor Crosshair' and 'Camera Guides'.
Add a 'Viewport Hints' option, and the other 3 inherit from it? This is what I understood, but I didn't know the themes had 'inheritance'.
Add a 'Viewport Hints' that affects the 3 options temporarily until the specific options are done and ready?
There is no way I can make this changes, other changes, and not get fired from my day job in time for 2.70 release. There is a user waiting for this with some upvotes: http://blender.stackexchange.com/questions/7122/how-can-i-change-the-color-of-fly-walk-mode-crosshair
Should this get in as is, and the rest added when I can, or its only going to enter after 2.70 anyway?
Hi @Inês Almeida (brita_) Campbell meant to have all 3 components using the same UI theme settings (in your words, to have them inherit the same property).
I agree that this makes more sense in the end.
It's not a lot of work from where you left, but if you think you won't be able to do it I can step in. It would be a shame though, since you are that close.
That said, users are always waiting for this or that. Not really a determining factor in my opinion. Specially since they can always get a nice build.blender.org Blender after the commit is in 'master'.
Add theme option 'Viewport Hints'
The options affects 3D Cursor crosshair, View Navigation Cursor and Camera Guides. Camera Guides are 'Composition Guides' and 'Safe Areas'. 'Sensor' was not
considered because it had a different color.
In general, I think this is fine.
Was thinking of other areas this could apply to the dashed line when transforming/scaling between the mouse cursor and the center point.
but can't find any other areas this might be used.
I'd like to apply this after 2.70 release.
note, am not sure of using name hints its a bit vague, but also not sure of better name, perhaps viewport_overlay_lines ?