The length of ImGui::Separator() not correct

[ketoo]

The issue showed as below:

The length of the line should the same length as the window.

[Nelarius]

Oh shoot, I'm not sure I have a good solution for this right now 😰

When ImGui::Separator is called, we don't yet know the size of the node because imnodes::EndNode hasn't been called yet. So the ImGui::Separator call uses the size of the current window, which is the canvas, I guess? 😬

I will have to create a list of known issues or something, and add this there until I can figure out what to do about this.

EDIT: in general, this is related to the problem of layout using imnodes: you also have to manually "align" your UI elements on the right side of the node for the same reason. I will have to investigate better ways of handling window size.

EDIT2: it also looks like ImGui::PushItemWidth has no effect on ImGui::Separator 😞

[Nelarius]

Added a mention of this in the README for now, thanks for reporting this!

[ketoo]

Thanks, I use a small button to meet the same goal.

[hiroMTB]

This can be solved by similar approach like #167 (comment)

But for Separator() we overwrite window->Size.x

ImGuiWindow* window = ImGui::GetCurrentWindow();
  
int xbackup = window->Size.x;
window->Size.x = window->DC.CursorPos.x + width;

ImGui::Separator();

// don't forget to tidy up
window->Size.x = xbackup;

I tested and can confirm it works but somehow Separator's line became slightly longer than node width like 10%.
It might be good to do this overwrite window variable inside BeginNode() so that all imgui widgets work fine until imgui officially supports this kind of feature.