From 888147adfefd98ae8ef8d155bab8397c256bd21a Mon Sep 17 00:00:00 2001
From: Petr Ohlidal <petr@ohlidal.cz>
Date: Fri, 11 Mar 2022 21:09:10 +0100
Subject: [PATCH] Repo UI: Improved item visibility in 'compact' view.

---
 .../gui/panels/GUI_RepositorySelector.cpp     | 33 +++++++++++--------
 1 file changed, 20 insertions(+), 13 deletions(-)

diff --git a/source/main/gui/panels/GUI_RepositorySelector.cpp b/source/main/gui/panels/GUI_RepositorySelector.cpp
index b7ca557c0a..df98272149 100644
--- a/source/main/gui/panels/GUI_RepositorySelector.cpp
+++ b/source/main/gui/panels/GUI_RepositorySelector.cpp
@@ -796,6 +796,7 @@ void RepositorySelector::Draw()
             }
 
             // Draw table body
+            int num_drawn_items = 0;
             for (int i = 0; i < m_data.items.size(); i++)
             {
                 // Skip items from non mod categories
@@ -912,6 +913,24 @@ void RepositorySelector::Draw()
                     else if (m_view_mode == "Compact")
                     {
                         float orig_cursor_x = ImGui::GetCursorPos().x;
+
+                        // Calc box size: Draw 3 boxes per line, 2 for small resolutions
+                        float box_width = (ImGui::GetIO().DisplaySize.x / 1.4) / 3;
+                        if (ImGui::GetIO().DisplaySize.x <= 1280)
+                        {
+                            box_width = (ImGui::GetIO().DisplaySize.x / 1.4) / 2;
+                        }
+
+                        // Skip to new line if at least 50% of the box can't fit on current line.
+                        if (orig_cursor_x > ImGui::GetWindowContentRegionMax().x - (box_width * 0.5)) 
+                        {
+                            // Unless this is the 1st line... not much to do with such narrow window.
+                            if (num_drawn_items != 0)
+                            {
+                                ImGui::NewLine();
+                            }
+                        }
+
                         ImGui::BeginGroup();
 
                         ImGui::PushStyleColor(ImGuiCol_Header, ImVec4(0.70f, 0.70f, 0.70f, 0.90f));
@@ -922,13 +941,6 @@ void RepositorySelector::Draw()
                         float orig_cursor_y = ImGui::GetCursorPosY();
                         std::string item_id = "##" + std::to_string(i);
 
-                        // Draw 3 boxes per line, 2 for small resolutions
-                        float box_width = (ImGui::GetIO().DisplaySize.x / 1.4) / 3;
-                        if (ImGui::GetIO().DisplaySize.x <= 1280)
-                        {
-                            box_width = (ImGui::GetIO().DisplaySize.x / 1.4) / 2;
-                        }
-
                         if (ImGui::Selectable(item_id.c_str(), m_selected_item.resource_id == m_data.items[i].resource_id, 0, ImVec2(box_width - ImGui::GetStyle().ItemSpacing.x, 100)))
                         {
                             m_selected_item = m_data.items[i];
@@ -998,12 +1010,6 @@ void RepositorySelector::Draw()
 
                         ImGui::EndGroup();
                         ImGui::SameLine();
-
-                        // Add new line
-                        if ( orig_cursor_x > ImGui::GetWindowSize().x - box_width)
-                        {
-                            ImGui::NewLine();
-                        }
                     }
                     else if (m_view_mode == "Basic")
                     {
@@ -1068,6 +1074,7 @@ void RepositorySelector::Draw()
                         ImGui::Separator();
                     }
                     ImGui::PopID();
+                    num_drawn_items++;
                 }
             }
             ImGui::EndChild();