Anim: Theme confusion with 'Pose Bone' label #110874

Closed
opened 2023-08-07 08:00:57 +02:00 by sanek122005 · 4 comments

Blender Version
Broken: 3.6.1

Short description of error

Edit Mode:

  1. Unselected head/tails should use "Wire Edit" instead of "Vertex".
  2. Selected head/tails should use "Edge Select" instead of "Wire".

Pose Mode:

  1. Unselected outline color currently uses "Wire", which is also used for a lot of other things. Would be nice to have a separate "Bone Outline" color, although this would only affect un-colored bones.
  2. "Bone Pose" theme color name is inaccurate, should be "Bone Pose Selected" instead.
**Blender Version** Broken: 3.6.1 **Short description of error** Edit Mode: 1. Unselected head/tails should use "Wire Edit" instead of "Vertex". 2. Selected head/tails should use "Edge Select" instead of "Wire". <video src="/attachments/f78c01df-0ff5-4393-bbb6-e3c3250ac50c" title="2023-08-07 08-56-49.mp4" controls></video> Pose Mode: 1. Unselected outline color currently uses "Wire", which is also used for a lot of other things. Would be nice to have a separate "Bone Outline" color, although this would only affect un-colored bones. 2. "Bone Pose" theme color name is inaccurate, should be "Bone Pose Selected" instead. <video src="/attachments/138a14e4-fece-4006-88f9-1713df8f5ff6" title="2023-08-07 08-58-49.mp4" controls></video>
sanek122005 added the
Type
Report
Priority
Normal
Status
Needs Triage
labels 2023-08-07 08:00:58 +02:00
Member

The Head/Tail of bone uses color of vertex, including when selecting state. There should be a separate parameter for the color of the Head/Tail itself, and the color when they are selected

Hi, thanks for the report. This is more of a request than a bug. Existing edge/wire theme colors are used for edit and pose bones: https://projects.blender.org/blender/blender/src/branch/main/source/blender/draw/intern/draw_common.cc#L105

  UI_GetThemeColor4fv(TH_BONE_POSE, gb->color_bone_pose);
  UI_GetThemeColor4fv(TH_BONE_POSE_ACTIVE, gb->color_bone_pose_active);
  UI_GetThemeColorShade4fv(TH_EDGE_SELECT, 60, gb->color_bone_active);
  UI_GetThemeColorShade4fv(TH_EDGE_SELECT, -20, gb->color_bone_select);
  UI_GetThemeColorBlendShade4fv(TH_WIRE, TH_BONE_POSE, 0.15f, 0, gb->color_bone_pose_active_unsel);

  • Outline of bone body uses Wire edit color, outline when bone is selected don`t have parameter
  • bone outline when don`t select, uses Wire color, it's inconvenient

Yes, if nothing is selected, default wire color is used, see : get_pchan_color_wire


"Pose bone" - the confusing parameter name... it should be called "Pose bone Selected"

Agree, this should be renamed. 😅

Forwarding to A&R devs for the decision.

> The Head/Tail of bone uses color of vertex, including when selecting state. There should be a separate parameter for the color of the Head/Tail itself, and the color when they are selected Hi, thanks for the report. This is more of a request than a bug. Existing edge/wire theme colors are used for edit and pose bones: https://projects.blender.org/blender/blender/src/branch/main/source/blender/draw/intern/draw_common.cc#L105 ``` UI_GetThemeColor4fv(TH_BONE_POSE, gb->color_bone_pose); UI_GetThemeColor4fv(TH_BONE_POSE_ACTIVE, gb->color_bone_pose_active); UI_GetThemeColorShade4fv(TH_EDGE_SELECT, 60, gb->color_bone_active); UI_GetThemeColorShade4fv(TH_EDGE_SELECT, -20, gb->color_bone_select); UI_GetThemeColorBlendShade4fv(TH_WIRE, TH_BONE_POSE, 0.15f, 0, gb->color_bone_pose_active_unsel); ``` - - - > - Outline of bone body uses Wire edit color, outline when bone is selected don`t have parameter > - bone outline when don`t select, uses Wire color, it's inconvenient Yes, if nothing is selected, default wire color is used, see : [get_pchan_color_wire]( https://projects.blender.org/blender/blender/src/branch/main/source/blender/draw/engines/overlay/overlay_armature.cc#L1189) - - - > "Pose bone" - the confusing parameter name... it should be called "Pose bone Selected" Agree, this should be renamed. 😅 Forwarding to A&R devs for the decision.

this is being worked on here: #112635: Proposal: New Bone Color Presets

this is being worked on here: [#112635: Proposal: New Bone Color Presets](https://projects.blender.org/blender/blender/issues/112635)
Member

I think the color preset changes won't really affect these issues, these are more to do with other theme settings, some of which are relevant for edit bones or un-colored bones.

The task text is a bit unclear, but the videos are helpful. I dove in to try and better understand and describe any issues.

Here's what each relevant theme setting seems to be doing:

  • Bone Solid: Color of polygons of un-colored bones (including custom shapes, including edit and object mode)
  • Bone Pose Active: Outline of active un-colored pose bone
  • Bone Pose: Outline of selected un-colored pose bone
  • Wire: Outline of un-selected un-colored pose bones (pose mode only; Affects bone body, tail, and head. Makes sense.)
  • Wire Edit: Outline of un-selected un-colored edit bones (edit mode only; Affects body, does not affect tail or head. That seems like a bug.)
  • Vertex: Outline of unselected edit bones' heads and tails. (both colored and un-colored)
  • Edge Select: Outline of selected and active edit bones' body, but not heads and tails.
  • Vertex Select: Outline of selected and active edit bones' heads and tails, but not body.

Conclusions:

  • Weird: Un-selected edit bone heads and tails should use Wire Edit instead of Vertex.
  • Weird: Selected and Active edit bones' heads and tails should use Edge Select instead of Vertex Select.
  • Bad name: Bone Pose should indeed be renamed to Bone Pose Selected
  • Bad name: Theme settings called Wire should be renamed Edge for consistency, the fact that these words are used interchangably here is super confusing.
  • Overloaded: Wire being used for both mesh edges and bone outlines is arguably affecting too many things and should be split up into a separate Bone Outline color. Note that this is only used for un-colored bones anyways, though.
  • Note: The fact that the heads and tails of colored edit bones don't use the bone color is likely intentional, otherwise there can be cases where there's no visual distinction between pose and edit mode, which would be pretty bad. That said, this not the most rock solid design either, but it's low priority and it does the job for now.

File with some bones to play around: bone_theme_weirdness.blend

I've taken the liberty to update the task text to clarify the issues that I believe were originally being reported. I would say these are rather minor issues, perhaps papercuts?

I think the color preset changes won't really affect these issues, these are more to do with other theme settings, some of which are relevant for edit bones or un-colored bones. The task text is a bit unclear, but the videos are helpful. I dove in to try and better understand and describe any issues. Here's what each relevant theme setting seems to be doing: - `Bone Solid`: Color of polygons of un-colored bones (including custom shapes, including edit and object mode) - `Bone Pose Active`: Outline of active un-colored pose bone - `Bone Pose`: Outline of selected un-colored pose bone - `Wire`: Outline of un-selected un-colored pose bones (pose mode only; Affects bone body, tail, and head. Makes sense.) - `Wire Edit`: Outline of un-selected un-colored edit bones (edit mode only; Affects body, **does not affect tail or head**. That seems like a bug.) - `Vertex`: Outline of unselected edit bones' heads and tails. (both colored and un-colored) - `Edge Select`: Outline of selected and active edit bones' body, but not heads and tails. - `Vertex Select`: Outline of selected and active edit bones' heads and tails, but not body. Conclusions: - Weird: Un-selected edit bone heads and tails should use `Wire Edit` instead of `Vertex`. - Weird: Selected and Active edit bones' heads and tails should use `Edge Select` instead of `Vertex Select`. - Bad name: `Bone Pose` should indeed be renamed to `Bone Pose Selected` - Bad name: Theme settings called `Wire` should be renamed `Edge` for consistency, the fact that these words are used interchangably here is super confusing. - Overloaded: `Wire` being used for both mesh edges and bone outlines is arguably affecting too many things and should be split up into a separate `Bone Outline` color. Note that this is only used for un-colored bones anyways, though. - Note: The fact that the heads and tails of colored edit bones don't use the bone color is likely intentional, otherwise there can be cases where there's no visual distinction between pose and edit mode, which would be pretty bad. That said, this not the most rock solid design either, but it's low priority and it does the job for now. File with some bones to play around: [bone_theme_weirdness.blend](/attachments/dc577950-15ae-4368-b6e3-e131b74c9ce1) I've taken the liberty to update the task text to clarify the issues that I believe were originally being reported. I would say these are rather minor issues, perhaps papercuts?
Demeter Dzadik changed title from Error in bone Theming to Theme: Bone color issues 2023-10-06 11:28:04 +02:00
  1. The Head/Tail of bone uses color of vertex, including when selecting state. There should be a separate parameter for the color of the Head/Tail itself, and the color when they are selected

This is not a bug, it's just how Blender was designed. You can disagree with that design, but that doesn't turn it into an error ;-)

  1. Outline of bone body uses Wire edit color, outline when bone is selected don`t have parameter

Same as above.

  1. bone outline when don`t select, uses Wire color, it's inconvenient

What's the inconvenience? I find it rather convenient that I only have to select one wire color, and that this is used for meshes as well as bones.

  1. "Pose bone" - the confusing parameter name, is actually the color of the selected bone. a parameter similar to object selected. it should be called "Pose bone Selected"

That's fair.

> 1. The Head/Tail of bone uses color of vertex, including when selecting state. There should be a separate parameter for the color of the Head/Tail itself, and the color when they are selected This is not a bug, it's just how Blender was designed. You can disagree with that design, but that doesn't turn it into an error ;-) > 2. Outline of bone body uses Wire edit color, outline when bone is selected don`t have parameter Same as above. > 1. bone outline when don`t select, uses Wire color, it's inconvenient What's the inconvenience? I find it rather convenient that I only have to select one wire color, and that this is used for meshes as well as bones. > 2. "Pose bone" - the confusing parameter name, is actually the color of the selected bone. a parameter similar to object selected. it should be called "Pose bone Selected" That's fair.
Sybren A. Stüvel changed title from Theme: Bone color issues to Anim: Theme confusion with 'Pose Bone' label 2023-10-06 11:42:55 +02:00
Sybren A. Stüvel self-assigned this 2023-10-06 11:47:05 +02:00
Sybren A. Stüvel added
Status
Confirmed
Type
Bug
and removed
Status
Needs Info from Developers
Type
Report
labels 2023-10-06 11:48:30 +02:00
Blender Bot added
Status
Resolved
and removed
Status
Confirmed
labels 2023-10-06 11:58:16 +02:00
Sign in to join this conversation.
No Label
Interest
Alembic
Interest
Animation & Rigging
Interest
Asset Browser
Interest
Asset Browser Project Overview
Interest
Audio
Interest
Automated Testing
Interest
Blender Asset Bundle
Interest
BlendFile
Interest
Collada
Interest
Compatibility
Interest
Compositing
Interest
Core
Interest
Cycles
Interest
Dependency Graph
Interest
Development Management
Interest
EEVEE
Interest
EEVEE & Viewport
Interest
Freestyle
Interest
Geometry Nodes
Interest
Grease Pencil
Interest
ID Management
Interest
Images & Movies
Interest
Import Export
Interest
Line Art
Interest
Masking
Interest
Metal
Interest
Modeling
Interest
Modifiers
Interest
Motion Tracking
Interest
Nodes & Physics
Interest
OpenGL
Interest
Overlay
Interest
Overrides
Interest
Performance
Interest
Physics
Interest
Pipeline, Assets & IO
Interest
Platforms, Builds & Tests
Interest
Python API
Interest
Render & Cycles
Interest
Render Pipeline
Interest
Sculpt, Paint & Texture
Interest
Text Editor
Interest
Translations
Interest
Triaging
Interest
Undo
Interest
USD
Interest
User Interface
Interest
UV Editing
Interest
VFX & Video
Interest
Video Sequencer
Interest
Virtual Reality
Interest
Vulkan
Interest
Wayland
Interest
Workbench
Interest: X11
Legacy
Blender 2.8 Project
Legacy
Milestone 1: Basic, Local Asset Browser
Legacy
OpenGL Error
Meta
Good First Issue
Meta
Papercut
Meta
Retrospective
Meta
Security
Module
Animation & Rigging
Module
Core
Module
Development Management
Module
EEVEE & Viewport
Module
Grease Pencil
Module
Modeling
Module
Nodes & Physics
Module
Pipeline, Assets & IO
Module
Platforms, Builds & Tests
Module
Python API
Module
Render & Cycles
Module
Sculpt, Paint & Texture
Module
Triaging
Module
User Interface
Module
VFX & Video
Platform
FreeBSD
Platform
Linux
Platform
macOS
Platform
Windows
Priority
High
Priority
Low
Priority
Normal
Priority
Unbreak Now!
Status
Archived
Status
Confirmed
Status
Duplicate
Status
Needs Info from Developers
Status
Needs Information from User
Status
Needs Triage
Status
Resolved
Type
Bug
Type
Design
Type
Known Issue
Type
Patch
Type
Report
Type
To Do
No Milestone
No project
No Assignees
5 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: blender/blender#110874
No description provided.