Recalculate the horizontal FListBox scroll bar size on lazy conversion

ChangeLog

@@ -1,3 +1,7 @@
+2017-05-20  Markus Gans  <guru.mail@muenster.de>
+	* Recalculate the horizontal FListBox scroll bar size 
+	  on lazy conversion
+
 2017-05-19  Markus Gans  <guru.mail@muenster.de>
 	* FListBox: Import of data from a container via 
 	  "lazy conversion" during item print

src/flistbox.cpp

@@ -274,37 +274,14 @@ void FListBox::hide()
 //----------------------------------------------------------------------
 void FListBox::insert (FListBoxItem listItem)
 {
-  int len, element_count;
+  int len = int(listItem.text.getLength());
-
+  bool has_brackets = bool(listItem.brackets);
-  len = int(listItem.text.getLength());
+  recalculateHorizontalBar(len, has_brackets);
-
-  if ( listItem.brackets )
-    len += 2;
-
-  if ( len > max_line_width )
-  {
-    max_line_width = len;
-
-    if ( len >= getWidth() - nf_offset - 3 )
-    {
-      hbar->setMaximum(max_line_width - getWidth() + nf_offset + 4);
-      hbar->setPageSize(max_line_width, getWidth() - nf_offset - 4);
-      hbar->calculateSliderValues();
-
-      if ( ! hbar->isVisible() )
-        hbar->setVisible();
-    }
-  }
 
   data.push_back (listItem);
 
-  element_count = int(getCount());
+  int element_count = int(getCount());
-  vbar->setMaximum(element_count - getHeight() + 2);
+  recalculateVerticalBar(element_count);
-  vbar->setPageSize(element_count, getHeight() - 2);
-  vbar->calculateSliderValues();
-
-  if ( ! vbar->isVisible() && element_count >= getHeight() - 1 )
-    vbar->setVisible();
 }
 
 //----------------------------------------------------------------------
@@ -1625,14 +1602,22 @@ void FListBox::drawList()
 
   for (uInt y=start; y < end; y++)
   {
-    setPrintPos (2, 2 + int(y));
     bool serach_mark = false;
     bool lineHasBrackets = hasBrackets(iter);
     bool isLineSelected = isSelected(iter);
     bool isCurrentLine = bool(y + uInt(yoffset) + 1 == uInt(current));
 
     if ( conv_type == lazy_convert && iter->getText().isNull() )
+    {
       convertToItem (*iter, source_container, int(y) + yoffset);
+      int len = int(iter->text.getLength());
+      recalculateHorizontalBar(len, lineHasBrackets);
+
+      if ( hbar->isVisible() )
+        hbar->redraw();
+    }
+
+    setPrintPos (2, 2 + int(y));
 
     if ( isLineSelected )
     {
@@ -1848,6 +1833,39 @@ void FListBox::drawList()
   last_current = current;
 }
 
+//----------------------------------------------------------------------
+void FListBox::recalculateHorizontalBar(int len, bool has_brackets)
+{
+  if ( has_brackets )
+    len += 2;
+
+  if ( len > max_line_width )
+  {
+    max_line_width = len;
+
+    if ( len >= getWidth() - nf_offset - 3 )
+    {
+      hbar->setMaximum(max_line_width - getWidth() + nf_offset + 4);
+      hbar->setPageSize(max_line_width, getWidth() - nf_offset - 4);
+      hbar->calculateSliderValues();
+
+      if ( ! hbar->isVisible() )
+        hbar->setVisible();
+    }
+  }
+}
+
+//----------------------------------------------------------------------
+void FListBox::recalculateVerticalBar(int element_count)
+{
+  vbar->setMaximum (element_count - getHeight() + 2);
+  vbar->setPageSize (element_count, getHeight() - 2);
+  vbar->calculateSliderValues();
+
+  if ( ! vbar->isVisible() && element_count >= getHeight() - 1 )
+    vbar->setVisible();
+}
+
 //----------------------------------------------------------------------
 void FListBox::processClick()
 {

src/flistbox.h

@@ -229,6 +229,8 @@ class FListBox : public FWidget
    void        draw();
    void        drawLabel();
    void        drawList();
+   void        recalculateHorizontalBar (int, bool);
+   void        recalculateVerticalBar (int);
    void        processClick();
    void        processSelect();
    void        processChanged();
@@ -467,6 +469,8 @@ void FListBox::insert (Container container, LazyConverter convert)
 
   if ( size > 0 )
     data.resize(size);
+
+  recalculateVerticalBar(int(size));
 }
 
 //----------------------------------------------------------------------

test/listbox.cpp

@@ -1,4 +1,4 @@
-// File: ui.cpp
+// File: listbox.cpp
 
 #include <iostream>
 #include <fstream>