Skip to content

Custom UI styling: hamburger menu, tab spacing, centered labels #1

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.

Sign up for GitHub

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

Open
noahbowers wants to merge 1 commit into
base: hazel
Choose a base branch
from
+44 −30
Open

Custom UI styling: hamburger menu, tab spacing, centered labels #1

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
19 changes: 13 additions & 6 deletions imgui.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7736,7 +7736,8 @@ bool ImGui::Begin(const char* name, bool* p_open, ImGuiWindowFlags flags)
// Lock menu offset so size calculation can use it as menu-bar windows need a minimum size.
window->DC.MenuBarOffset.x = ImMax(ImMax(window->WindowPadding.x, style.ItemSpacing.x), g.NextWindowData.MenuBarOffsetMinVal.x);
window->DC.MenuBarOffset.y = g.NextWindowData.MenuBarOffsetMinVal.y;
window->TitleBarHeight = (flags & ImGuiWindowFlags_NoTitleBar) ? 0.0f : g.FontSize + g.Style.FramePadding.y * 2.0f;
const float title_bar_padding = (window->DockIsActive) ? 8.0f : g.Style.FramePadding.y;
window->TitleBarHeight = (flags & ImGuiWindowFlags_NoTitleBar) ? 0.0f : g.FontSize + title_bar_padding * 2.0f;
Comment on lines +7739 to +7740
Copy link

@coderabbitai coderabbitai bot Mar 9, 2026

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟠 Major

🧩 Analysis chain

🏁 Script executed:

# First, let's check the structure of the repo and locate imgui.cpp
git ls-files | grep -E "imgui\.cpp$"

Repository: StudioCherno/imgui

Length of output: 71

🏁 Script executed:

# Get the line count to understand file size
wc -l imgui.cpp

Repository: StudioCherno/imgui

Length of output: 77

🏁 Script executed:

# View the code around lines 7739-7740
sed -n '7735,7745p' imgui.cpp

Repository: StudioCherno/imgui

Length of output: 1116

🏁 Script executed:

# View the code around lines 18865-18866
sed -n '18860,18870p' imgui.cpp

Repository: StudioCherno/imgui

Length of output: 607

🏁 Script executed:

# Check the context for popup anchor logic around lines 18453-18465
sed -n '18450,18470p' imgui.cpp

Repository: StudioCherno/imgui

Length of output: 961

🏁 Script executed:

# Search for existing dock padding or style-related constants
rg "DockIsActive|title_bar_padding|FramePadding" -A 2 -B 2 imgui.cpp | head -100

Repository: StudioCherno/imgui

Length of output: 9853

🏁 Script executed:

# Search for GetFrameHeight() definition
rg "float GetFrameHeight|ImFloat GetFrameHeight" -A 5 imgui.cpp

Repository: StudioCherno/imgui

Length of output: 44

🏁 Script executed:

# Also search in header files
fd "\.h$" | xargs rg "GetFrameHeight" -B 2 -A 2

Repository: StudioCherno/imgui

Length of output: 2354

🏁 Script executed:

# Check if there are any comments or TODOs around the dock padding lines
sed -n '7730,7750p' imgui.cpp

Repository: StudioCherno/imgui

Length of output: 2063

🏁 Script executed:

# Check comments around line 18865
sed -n '18855,18875p' imgui.cpp

Repository: StudioCherno/imgui

Length of output: 955

Avoid hard-coding dock header padding; it breaks style/DPI scaling and misaligns popups.

Line 7739 and line 18865 both use hard-coded 8.0f for dock padding. This breaks two things:

  1. DPI/style scaling: Hard-coded pixels don't scale with style.FramePadding during DPI changes or custom style configurations. The default FramePadding.y is 3.0f, not 8.0f, so dock headers will appear oversized or undersized relative to other UI elements.

  2. Popup misalignment: The dock window menu popup uses GetFrameHeight() (which returns FontSize + style.FramePadding.y * 2) for positioning, but the dock header is sized with hard-coded 8.0f * 2 padding. This creates a vertical mismatch between the popup anchor and the dock header boundary.

Derive padding from style.FramePadding or expose it as a style value to maintain consistency across scaled and custom-styled UIs.

🤖 Prompt for AI Agents 
Verify each finding against the current code and only fix it if needed.

In `@imgui.cpp` around lines 7739 - 7740, Summary: Hard-coded dock padding (8.0f)
causes DPI/style scaling and popup alignment issues. Fix: replace the magic 8.0f
used when computing title_bar_padding (the expression setting
window->TitleBarHeight where window->DockIsActive is checked) with a value
derived from the active style e.g. use g.Style.FramePadding.y (or introduce a
new ImGui style variable like DockPadding and read it from g.Style) so the
computed title_bar_padding matches GetFrameHeight()’s use of g.FontSize +
g.Style.FramePadding.y * 2 and keep the same change at the other occurrence (the
one noted around line 18865) so dock header sizing and popup anchoring are
consistent across DPI/style changes.

window->MenuBarHeight = (flags & ImGuiWindowFlags_MenuBar) ? window->DC.MenuBarOffset.y + g.FontSize + g.Style.FramePadding.y * 2.0f : 0.0f;
window->FontRefSize = g.FontSize; // Lock this to discourage calling window->CalcFontSize() outside of current window.

Expand Down Expand Up @@ -18452,12 +18453,16 @@ static void ImGui::DockNodeWindowMenuUpdate(ImGuiDockNode* node, ImGuiTabBar* ta
SetNextWindowPos(ImVec2(node->Pos.x, node->Pos.y + GetFrameHeight()), ImGuiCond_Always, ImVec2(0.0f, 0.0f));
else
SetNextWindowPos(ImVec2(node->Pos.x + node->Size.x, node->Pos.y + GetFrameHeight()), ImGuiCond_Always, ImVec2(1.0f, 0.0f));
ImVec4 popupBg = g.Style.Colors[ImGuiCol_PopupBg];
popupBg.w = 1.0f;
PushStyleColor(ImGuiCol_PopupBg, popupBg);
if (BeginPopup("#WindowMenu"))
{
node->IsFocused = true;
g.DockNodeWindowMenuHandler(&g, node, tab_bar);
EndPopup();
}
PopStyleColor();
Comment on lines 18453 to +18465
Copy link

@coderabbitai coderabbitai bot Mar 9, 2026

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor

Anchor the window menu popup to the new dock header height.

The dock header is now g.FontSize + tab_padding_y * 2, but #WindowMenu is still positioned with GetFrameHeight(), which is based on style.FramePadding.y. That leaves the popup a few pixels too high and overlapping the enlarged tab bar. Reuse the same header-height calculation here, or read it from the host window, instead of GetFrameHeight().

Also applies to: 18865-18866

🤖 Prompt for AI Agents 
Verify each finding against the current code and only fix it if needed.

In `@imgui.cpp` around lines 18453 - 18465, The popup for "#WindowMenu" is
anchored using GetFrameHeight(), which is wrong now that dock header height is
g.FontSize + tab_padding_y * 2; update the positioning call that uses
GetFrameHeight() (the SetNextWindowPos call that computes ImVec2(node->Pos.x,
node->Pos.y + GetFrameHeight()) and the alternate branch using node->Pos.y +
GetFrameHeight()) to use the same header-height calculation as the dock tabs
(compute header_height = g.FontSize + tab_padding_y * 2 or read the header
height from the host window if available) so the popup sits immediately below
the enlarged dock header before calling BeginPopup("#WindowMenu") and
g.DockNodeWindowMenuHandler.

}

// User helper to append/amend into a dock node tab bar. Most commonly used to add e.g. a "+" button.
Expand Down Expand Up @@ -18857,28 +18862,30 @@ static void ImGui::DockNodeCalcTabBarLayout(const ImGuiDockNode* node, ImRect* o
ImGuiContext& g = *GImGui;
ImGuiStyle& style = g.Style;

ImRect r = ImRect(node->Pos.x, node->Pos.y, node->Pos.x + node->Size.x, node->Pos.y + g.FontSize + g.Style.FramePadding.y * 2.0f);
const float tab_padding_y = 8.0f;
ImRect r = ImRect(node->Pos.x, node->Pos.y, node->Pos.x + node->Size.x, node->Pos.y + g.FontSize + tab_padding_y * 2.0f);
if (out_title_rect) { *out_title_rect = r; }

r.Min.x += style.WindowBorderSize;
r.Max.x -= style.WindowBorderSize;

float button_sz = g.FontSize;
float button_y = r.Min.y + (r.GetHeight() - button_sz) * 0.5f;
r.Min.x += style.FramePadding.x;
r.Max.x -= style.FramePadding.x;
ImVec2 window_menu_button_pos = ImVec2(r.Min.x, r.Min.y + style.FramePadding.y);
ImVec2 window_menu_button_pos = ImVec2(r.Min.x, button_y);
if (node->HasCloseButton)
{
if (out_close_button_pos) *out_close_button_pos = ImVec2(r.Max.x - button_sz, r.Min.y + style.FramePadding.y);
if (out_close_button_pos) *out_close_button_pos = ImVec2(r.Max.x - button_sz, button_y);
r.Max.x -= button_sz + style.ItemInnerSpacing.x;
}
if (node->HasWindowMenuButton && style.WindowMenuButtonPosition == ImGuiDir_Left)
{
r.Min.x += button_sz + style.ItemInnerSpacing.x;
r.Min.x += button_sz + style.FramePadding.x;
}
else if (node->HasWindowMenuButton && style.WindowMenuButtonPosition == ImGuiDir_Right)
{
window_menu_button_pos = ImVec2(r.Max.x - button_sz, r.Min.y + style.FramePadding.y);
window_menu_button_pos = ImVec2(r.Max.x - button_sz, button_y);
r.Max.x -= button_sz + style.ItemInnerSpacing.x;
}
if (out_tab_bar_rect) { *out_tab_bar_rect = r; }
Expand Down
11 changes: 9 additions & 2 deletions imgui_draw.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4430,8 +4430,15 @@ void ImGui::RenderArrowPointingAt(ImDrawList* draw_list, ImVec2 pos, ImVec2 half
// and because the saved space means that the left-most tab label can stay at exactly the same position as the label of a loose window.
void ImGui::RenderArrowDockMenu(ImDrawList* draw_list, ImVec2 p_min, float sz, ImU32 col)
{
draw_list->AddRectFilled(p_min + ImVec2(sz * 0.20f, sz * 0.15f), p_min + ImVec2(sz * 0.80f, sz * 0.30f), col);
RenderArrowPointingAt(draw_list, p_min + ImVec2(sz * 0.50f, sz * 0.85f), ImVec2(sz * 0.30f, sz * 0.40f), ImGuiDir_Down, col);
// Draw a hamburger menu icon (three horizontal lines)
float cx = p_min.x + sz * 0.5f;
float cy = p_min.y + sz * 0.5f;
float half = sz * 0.30f;
float spacing = sz * 0.22f;
float thickness = ImMax(sz * 0.12f, 1.0f);
draw_list->AddLine(ImVec2(cx - half, cy - spacing), ImVec2(cx + half, cy - spacing), col, thickness);
draw_list->AddLine(ImVec2(cx - half, cy), ImVec2(cx + half, cy), col, thickness);
draw_list->AddLine(ImVec2(cx - half, cy + spacing), ImVec2(cx + half, cy + spacing), col, thickness);
}

static inline float ImAcos01(float x)
Expand Down
44 changes: 22 additions & 22 deletions imgui_widgets.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -903,13 +903,10 @@ bool ImGui::CloseButton(ImGuiID id, const ImVec2& pos)
return pressed;

// Render
ImU32 bg_col = GetColorU32(held ? ImGuiCol_ButtonActive : ImGuiCol_ButtonHovered);
if (hovered)
window->DrawList->AddRectFilled(bb.Min, bb.Max, bg_col);
RenderNavCursor(bb, id, ImGuiNavRenderCursorFlags_Compact);
const ImU32 cross_col = GetColorU32(ImGuiCol_Text);
const ImU32 cross_col = (hovered || held) ? GetColorU32(ImGuiCol_Text) : GetColorU32(ImGuiCol_TextDisabled);
const ImVec2 cross_center = bb.GetCenter() - ImVec2(0.5f, 0.5f);
const float cross_extent = g.FontSize * 0.5f * 0.7071f - 1.0f;
const float cross_extent = g.FontSize * 0.5f * 0.5f - 1.0f;
const float cross_thickness = 1.0f; // FIXME-DPI
window->DrawList->AddLine(cross_center + ImVec2(+cross_extent, +cross_extent), cross_center + ImVec2(-cross_extent, -cross_extent), cross_col, cross_thickness);
window->DrawList->AddLine(cross_center + ImVec2(+cross_extent, -cross_extent), cross_center + ImVec2(-cross_extent, +cross_extent), cross_col, cross_thickness);
Expand All @@ -932,10 +929,7 @@ bool ImGui::CollapseButton(ImGuiID id, const ImVec2& pos, ImGuiDockNode* dock_no

// Render
//bool is_dock_menu = (window->DockNodeAsHost && !window->Collapsed);
ImU32 bg_col = GetColorU32((held && hovered) ? ImGuiCol_ButtonActive : hovered ? ImGuiCol_ButtonHovered : ImGuiCol_Button);
ImU32 text_col = GetColorU32(ImGuiCol_Text);
if (hovered || held)
window->DrawList->AddRectFilled(bb.Min, bb.Max, bg_col);
RenderNavCursor(bb, id, ImGuiNavRenderCursorFlags_Compact);

if (dock_node)
Expand Down Expand Up @@ -9593,8 +9587,9 @@ static void ImGui::TabBarLayout(ImGuiTabBar* tab_bar)
ImQsort(tab_bar->Tabs.Data, tab_bar->Tabs.Size, sizeof(ImGuiTabItem), TabItemComparerBySection);

// Calculate spacing between sections
sections[0].Spacing = sections[0].TabCount > 0 && (sections[1].TabCount + sections[2].TabCount) > 0 ? g.Style.ItemInnerSpacing.x : 0.0f;
sections[1].Spacing = sections[1].TabCount > 0 && sections[2].TabCount > 0 ? g.Style.ItemInnerSpacing.x : 0.0f;
const float tab_spacing = 0.0f;
sections[0].Spacing = sections[0].TabCount > 0 && (sections[1].TabCount + sections[2].TabCount) > 0 ? tab_spacing : 0.0f;
sections[1].Spacing = sections[1].TabCount > 0 && sections[2].TabCount > 0 ? tab_spacing : 0.0f;
Comment on lines +9590 to +9592
Copy link

@coderabbitai coderabbitai bot Mar 9, 2026

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟠 Major

Keep zero-spacing math consistent across layout and hit-testing.

tab_spacing = 0 only changes section separators and the final tab->Offset increment here. Line 9648 still adds g.Style.ItemInnerSpacing.x into sections[*].Width, and Lines 10002-10003 still expand reorder thresholds by the old spacing. With the default 4px inner spacing, three central tabs over-report their width by 8px, so shrink/scroll/reorder start before the rendered tabs actually need it.

Also applies to: 9722-9722, 9983-9983

🤖 Prompt for AI Agents 
Verify each finding against the current code and only fix it if needed.

In `@imgui_widgets.cpp` around lines 9590 - 9592, The zero tab_spacing path
currently leaves leftover inner spacing in layout and hit-testing: when
tab_spacing == 0 you must also remove g.Style.ItemInnerSpacing.x from
sections[].Width calculation and from any reorder threshold/expand computations
so widths and hit targets match rendered separators; update the code that
computes sections[0].Width/sections[1].Width (where g.Style.ItemInnerSpacing.x
is added) and the reorder threshold math that uses the old spacing (the code
that expands reorder thresholds and uses tab->Offset) to subtract or use
tab_spacing instead of always adding ItemInnerSpacing.x, and apply the same fix
to the other occurrences mentioned (the other sections[*].Spacing/Width and
reorder threshold calculations).


// Setup next selected tab
ImGuiID scroll_to_tab_id = 0;
Expand Down Expand Up @@ -9724,7 +9719,7 @@ static void ImGui::TabBarLayout(ImGuiTabBar* tab_bar)
ImGuiTabItem* tab = &tab_bar->Tabs[section_tab_index + tab_n];
tab->Offset = tab_offset;
tab->NameOffset = -1;
tab_offset += tab->Width + (tab_n < section->TabCount - 1 ? g.Style.ItemInnerSpacing.x : 0.0f);
tab_offset += tab->Width + (tab_n < section->TabCount - 1 ? tab_spacing : 0.0f);
}
tab_bar->WidthAllTabs += ImMax(section->Width + section->Spacing, 0.0f);
tab_offset += section->Spacing;
Expand Down Expand Up @@ -9985,6 +9980,7 @@ void ImGui::TabBarQueueReorderFromMousePos(ImGuiTabBar* tab_bar, ImGuiTabItem* s
if ((tab_bar->Flags & ImGuiTabBarFlags_Reorderable) == 0)
return;

const float tab_spacing = 0.0f;
const bool is_central_section = (src_tab->Flags & ImGuiTabItemFlags_SectionMask_) == 0;
const float bar_offset = tab_bar->BarRect.Min.x - (is_central_section ? tab_bar->ScrollingTarget : 0);

Expand Down Expand Up @@ -10475,7 +10471,7 @@ bool ImGui::TabItemEx(ImGuiTabBar* tab_bar, const char* label, bool* p_open,
if (is_visible)
{
ImDrawList* display_draw_list = window->DrawList;
const ImU32 tab_col = GetColorU32((held || hovered) ? ImGuiCol_TabHovered : tab_contents_visible ? (tab_bar_focused ? ImGuiCol_TabSelected : ImGuiCol_TabDimmedSelected) : (tab_bar_focused ? ImGuiCol_Tab : ImGuiCol_TabDimmed));
const ImU32 tab_col = GetColorU32(tab_contents_visible ? (tab_bar_focused ? ImGuiCol_TabSelected : ImGuiCol_TabDimmedSelected) : (tab_bar_focused ? ImGuiCol_Tab : ImGuiCol_TabDimmed));
TabItemBackground(display_draw_list, bb, flags, tab_col);
if (tab_contents_visible && (tab_bar->Flags & ImGuiTabBarFlags_DrawSelectedOverline) && style.TabBarOverlineSize > 0.0f)
{
Expand Down Expand Up @@ -10572,11 +10568,12 @@ ImVec2 ImGui::TabItemCalcSize(const char* label, bool has_close_button_or_unsave
{
ImGuiContext& g = *GImGui;
ImVec2 label_size = CalcTextSize(label, NULL, true);
ImVec2 size = ImVec2(label_size.x + g.Style.FramePadding.x, label_size.y + g.Style.FramePadding.y * 2.0f);
const float tab_padding_y = 8.0f;
ImVec2 size = ImVec2(label_size.x + g.Style.FramePadding.x * 2.0f, label_size.y + tab_padding_y * 2.0f);
if (has_close_button_or_unsaved_marker)
size.x += g.Style.FramePadding.x + (g.Style.ItemInnerSpacing.x + g.FontSize); // We use Y intentionally to fit the close button circle.
size.x += g.Style.FramePadding.x * 2.0f + (g.Style.ItemInnerSpacing.x + g.FontSize); // We use Y intentionally to fit the close button circle.
else
size.x += g.Style.FramePadding.x + 1.0f;
size.x += g.Style.FramePadding.x * 2.0f + 1.0f;
Comment on lines +10571 to +10576
Copy link

@coderabbitai coderabbitai bot Mar 9, 2026

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟠 Major

This 8px padding regresses non-docking tab bars.

TabItemCalcSize() now makes every tab FontSize + 16 tall, but BeginTabBar() still builds manual tab bars with g.Style.FramePadding.y * 2 at Line 9422. On the default style the tab item ends up noticeably taller than the bar/separator, and the extra height also applies outside dock nodes even though the PR goal was docked windows. Either gate this on dock-node bars or propagate the same padding into the generic tab-bar rect/layout path.

🤖 Prompt for AI Agents 
Verify each finding against the current code and only fix it if needed.

In `@imgui_widgets.cpp` around lines 10571 - 10576, TabItemCalcSize()
unconditionally adds tab_padding_y = 8.0f making each tab FontSize+16 tall which
regresses non-docking tab bars; update the logic so the extra vertical padding
is only applied for dock-node tab bars (or alternatively make BeginTabBar() use
the same padding). Modify TabItemCalcSize() to check the tab bar type/state (the
same condition BeginTabBar() uses for docked bars) before adding tab_padding_y,
or propagate g.Style.FramePadding.y * 2 into the non-docking layout path so
sizes match BeginTabBar(); adjust the size computation around the tab_padding_y
variable and the calls/sites that rely on TabItemCalcSize() to ensure non-docked
bars keep the original FramePadding-based height.

return ImVec2(ImMin(size.x, TabBarCalcMaxTabWidth()), size.y);
}

Expand All @@ -10602,11 +10599,7 @@ void ImGui::TabItemBackground(ImDrawList* draw_list, const ImRect& bb, ImGuiTabI
draw_list->PathFillConvex(col);
if (g.Style.TabBorderSize > 0.0f)
{
draw_list->PathLineTo(ImVec2(bb.Min.x + 0.5f, y2));
draw_list->PathArcToFast(ImVec2(bb.Min.x + rounding + 0.5f, y1 + rounding + 0.5f), rounding, 6, 9);
draw_list->PathArcToFast(ImVec2(bb.Max.x - rounding - 0.5f, y1 + rounding + 0.5f), rounding, 9, 12);
draw_list->PathLineTo(ImVec2(bb.Max.x - 0.5f, y2));
draw_list->PathStroke(GetColorU32(ImGuiCol_Border), 0, g.Style.TabBorderSize);
draw_list->AddLine(ImVec2(bb.Min.x + 0.5f, y1), ImVec2(bb.Min.x + 0.5f, y2), IM_COL32(0, 0, 0, 255), g.Style.TabBorderSize);
}
Comment on lines 10600 to 10603
Copy link

@coderabbitai coderabbitai bot Mar 9, 2026

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor

Don't hardcode the left border color.

IM_COL32(0, 0, 0, 255) bypasses theme colors and global style alpha, so this strip stays opaque black in light themes or during fades. Pull it from the style instead.

🎨 Suggested fix 
-        draw_list->AddLine(ImVec2(bb.Min.x + 0.5f, y1), ImVec2(bb.Min.x + 0.5f, y2), IM_COL32(0, 0, 0, 255), g.Style.TabBorderSize);
+        draw_list->AddLine(ImVec2(bb.Min.x + 0.5f, y1), ImVec2(bb.Min.x + 0.5f, y2), GetColorU32(ImGuiCol_Border), g.Style.TabBorderSize);
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
if (g.Style.TabBorderSize > 0.0f)
{
draw_list->PathLineTo(ImVec2(bb.Min.x + 0.5f, y2));
draw_list->PathArcToFast(ImVec2(bb.Min.x + rounding + 0.5f, y1 + rounding + 0.5f), rounding, 6, 9);
draw_list->PathArcToFast(ImVec2(bb.Max.x - rounding - 0.5f, y1 + rounding + 0.5f), rounding, 9, 12);
draw_list->PathLineTo(ImVec2(bb.Max.x - 0.5f, y2));
draw_list->PathStroke(GetColorU32(ImGuiCol_Border), 0, g.Style.TabBorderSize);
draw_list->AddLine(ImVec2(bb.Min.x + 0.5f, y1), ImVec2(bb.Min.x + 0.5f, y2), IM_COL32(0, 0, 0, 255), g.Style.TabBorderSize);
}
if (g.Style.TabBorderSize > 0.0f)
{
draw_list->AddLine(ImVec2(bb.Min.x + 0.5f, y1), ImVec2(bb.Min.x + 0.5f, y2), GetColorU32(ImGuiCol_Border), g.Style.TabBorderSize);
}
🤖 Prompt for AI Agents 
Verify each finding against the current code and only fix it if needed.

In `@imgui_widgets.cpp` around lines 10600 - 10603, The left border color is
hardcoded as IM_COL32(0,0,0,255) which ignores theme colors and global alpha;
replace that literal with the style-derived color (e.g. use
ImGui::GetColorU32(ImGuiCol_Border) or read g.Style.Colors[ImGuiCol_Border] and
convert with ImGui::GetColorU32 to honor theme and global alpha) when calling
draw_list->AddLine in the branch that checks g.Style.TabBorderSize.

}

Expand Down Expand Up @@ -10634,7 +10627,14 @@ void ImGui::TabItemLabelAndCloseButton(ImDrawList* draw_list, const ImRect& bb,
#endif

// Render text label (with clipping + alpha gradient) + unsaved marker
ImRect text_ellipsis_clip_bb(bb.Min.x + frame_padding.x, bb.Min.y + frame_padding.y, bb.Max.x - frame_padding.x, bb.Max.y);
// Center text vertically within visible tab area (background starts at bb.Min.y + 1.0f)
float visible_top = bb.Min.y + 1.0f;
float visible_height = bb.Max.y - visible_top;
float text_y = visible_top + (visible_height - label_size.y) * 0.5f;
// Center text horizontally within the tab
float avail_width = bb.GetWidth() - frame_padding.x * 2.0f;
float text_offset_x = (avail_width > label_size.x) ? (avail_width - label_size.x) * 0.5f : 0.0f;
ImRect text_ellipsis_clip_bb(bb.Min.x + frame_padding.x + text_offset_x, text_y, bb.Max.x - frame_padding.x, bb.Max.y);

// Return clipped state ignoring the close button
if (out_text_clipped)
Expand All @@ -10644,7 +10644,7 @@ void ImGui::TabItemLabelAndCloseButton(ImDrawList* draw_list, const ImRect& bb,
}

const float button_sz = g.FontSize;
const ImVec2 button_pos(ImMax(bb.Min.x, bb.Max.x - frame_padding.x - button_sz), bb.Min.y + frame_padding.y);
const ImVec2 button_pos(ImMax(bb.Min.x, bb.Max.x - frame_padding.x - button_sz), bb.Min.y + (bb.GetHeight() - button_sz) * 0.5f);

// Close Button & Unsaved Marker
// We are relying on a subtle and confusing distinction between 'hovered' and 'g.HoveredId' which happens because we are using ImGuiButtonFlags_AllowOverlapMode + SetItemAllowOverlap()
Expand Down