griefith/test2
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

UI: Theme: Move curve handle properties in common

Browse Source 
Continuing #140360, after 39c066ee53

Moving curve handle colors & size from space editors into common theme properties.

They are now shared by 3D Viewport, Image Editor, Graph Editor, and Movie Clip Editor,
instead of each of them having their own properties.

(Video in PR)

---

Details:
- NURB U/V lines, active spline, and last selected point colors were defined
in `rna_def_userdef_theme_spaces_curves` but were optional and only called by
3D viewport. Since no other editor calls that function anymore, I removed it and
moved those properties directly inside `rna_def_userdef_theme_space_view3d`.
- In Image Editor & Movie Clip Editor (Mask mode), curves don't ever show
selection colors, and Vector handles are generally never visible, that is
bug/missing in Blender in general, not a result of this PR.
- Handle vertex size were included in Dope Sheet and NLA, where handles don't
exist. Now as a side effect they're removed as well.

Pull Request: https://projects.blender.org/blender/blender/pulls/143762
This commit is contained in:
Nika Kutsniashvili
2025-08-15 21:40:56 +02:00
committed by Nika Kutsniashvili
parent a767671c56
commit 7d2b024151
8 changed files with 227 additions and 249 deletions
Download Patch File Download Diff File Expand all files Collapse all files

62
scripts/presets/interface_theme/Blender_Light.xml
View File

@@ -398,6 +398,24 @@
>
</ThemeCommonAnim>
</anim>
<curves>
<ThemeCommonCurves
handle_free="#000000"
handle_sel_free="#000000"
handle_auto="#909000"
handle_sel_auto="#f0ff40"
handle_vect="#409030"
handle_sel_vect="#40c030"
handle_align="#803060"
handle_sel_align="#f090a0"
handle_auto_clamped="#994030"
handle_sel_auto_clamped="#f0af90"
handle_vertex="#000000"
handle_vertex_select="#ff8500"
handle_vertex_size="5"
>
</ThemeCommonCurves>
</curves>
</ThemeCommon>
</common>
<view_3d>
@@ -445,14 +463,6 @@
nurb_sel_uline="#f0ff40"
nurb_sel_vline="#f090a0"
act_spline="#db2512"
handle_free="#000000"
handle_auto="#909000"
handle_vect="#409030"
handle_sel_vect="#40c030"
handle_align="#803060"
handle_sel_free="#000000"
handle_sel_auto="#f0ff40"
handle_sel_align="#f090a0"
lastsel_point="#ffffff"
extra_edge_len="#ff6149"
extra_edge_angle="#cccc00"
@@ -519,20 +529,7 @@
vertex_size="6"
vertex_bevel="#000000"
vertex_unreferenced="#000000"
handle_free="#000000"
handle_auto="#909000"
handle_vect="#409030"
handle_sel_vect="#40c030"
handle_align="#803060"
handle_sel_free="#000000"
handle_sel_auto="#f0ff40"
handle_sel_align="#f090a0"
handle_auto_clamped="#994030"
handle_sel_auto_clamped="#f0af90"
lastsel_point="#ffffff"
handle_vertex="#000000"
handle_vertex_select="#ff8500"
handle_vertex_size="5"
>
<space>
<ThemeSpaceGeneric
@@ -731,17 +728,6 @@
uv_shadow="#707070ff"
metadatabg="#000000"
metadatatext="#ffffff"
handle_free="#000000"
handle_auto="#909000"
handle_align="#803060"
handle_sel_free="#000000"
handle_sel_auto="#f0ff40"
handle_sel_align="#f090a0"
handle_auto_clamped="#000000"
handle_sel_auto_clamped="#000000"
handle_vertex="#000000"
handle_vertex_select="#ffff00"
handle_vertex_size="5"
paint_curve_handle="#7fff7f7f"
paint_curve_pivot="#ff7f7f7f"
>
@@ -1067,17 +1053,7 @@
strips_selected="#ff8c00"
metadatabg="#000000"
metadatatext="#ffffff"
handle_free="#000000"
handle_auto="#909000"
handle_align="#803060"
handle_sel_free="#000000"
handle_sel_auto="#f0ff40"
handle_sel_align="#f090a0"
handle_auto_clamped="#000000"
handle_sel_auto_clamped="#000000"
handle_vertex="#000000"
handle_vertex_select="#ffff00"
handle_vertex_size="5"
preview_range="#a14d0066"
>
<space>
<ThemeSpaceGeneric