Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[API] Fixed four of the ImGuiTreeNodeFlags that were all set to Framed #26

Merged
merged 1 commit into from
Oct 17, 2020

Conversation

silentorb
Copy link
Contributor

@silentorb silentorb commented Oct 17, 2020

Four of the values of ImGuiTreeNodeFlags were mistakenly set to 1 << 1. I updated the values to match ImGuiTreeNodeFlags_ in Dear ImGui.

https://github.com/ocornut/imgui/blob/v1.78/imgui.h#L876-L879

    ImGuiTreeNodeFlags_FramePadding         = 1 << 10,  // Use FramePadding (even for an unframed text node) to vertically align text baseline to regular widget height. Equivalent to calling AlignTextToFramePadding().
    ImGuiTreeNodeFlags_SpanAvailWidth       = 1 << 11,  // Extend hit box to the right-most edge, even if not framed. This is not the default in order to allow adding other items on the same line. In the future we may refactor the hit system to be front-to-back, allowing natural overlaps and then this can become the default.
    ImGuiTreeNodeFlags_SpanFullWidth        = 1 << 12,  // Extend hit box to the left-most and right-most edges (bypass the indented area).
    ImGuiTreeNodeFlags_NavLeftJumpsBackHere = 1 << 13,  // (WIP) Nav: left direction may move to this TreeNode() from any of its child (items submitted between TreeNode and TreePop)

@SpaiR SpaiR changed the title Fixed four of the ImGuiTreeNodeFlags that were all set to Framed [API] Fixed four of the ImGuiTreeNodeFlags that were all set to Framed Oct 17, 2020
@SpaiR SpaiR merged commit 06322ac into SpaiR:master Oct 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants