Skip to content

Compute optimal scene hierarchy width#2965

Draft
Meakk wants to merge 1 commit intof3d-app:masterfrom
Meakk:scene-hierarchy-width
Draft

Compute optimal scene hierarchy width#2965
Meakk wants to merge 1 commit intof3d-app:masterfrom
Meakk:scene-hierarchy-width

Conversation

@Meakk
Copy link
Copy Markdown
Member

@Meakk Meakk commented Mar 23, 2026

Describe your changes

Issue ticket number and link if any

Checklist for finalizing the PR

  • I have performed a self-review of my code
  • I have added tests for new features and bugfixes
  • I have added documentation for new features
  • If it is a modifying the libf3d API, I have updated bindings
  • If it is a modifying the .github/workflows/versions.json, I have updated docker_timestamp

Continuous integration

Please write a comment to run CI, eg: \ci fast.
See here for more info.

@Meakk Meakk self-assigned this Mar 23, 2026
@Meakk Meakk marked this pull request as draft March 23, 2026 16:29

constexpr float margin = F3DStyle::GetDefaultMargin();
constexpr float defaultWidth = 200.f;
float defaultWidth = this->CalculateHierarchyWidth(importer);
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like there's an autofit feature in imgui if we specify 0?

Suggested change
float defaultWidth = this->CalculateHierarchyWidth(importer);
float defaultWidth = 0.f;

@Meakk Meakk force-pushed the scene-hierarchy-width branch from b864c15 to 2ef979a Compare March 29, 2026 20:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant