X-Git-Url: https://git.cweiske.de/enigma2.git/blobdiff_plain/8426492f9ab139f78dd307e4df9cd114801b1630..38be97ebc8fd897de166a2f641df5b7c175dcb6f:/lib/gui/elistbox.cpp

diff --git a/lib/gui/elistbox.cpp b/lib/gui/elistbox.cpp
index 8da691b4..a5d18e6b 100644
--- a/lib/gui/elistbox.cpp
+++ b/lib/gui/elistbox.cpp
@@ -4,22 +4,22 @@
 #include <lib/actions/action.h>
 
 eListbox::eListbox(eWidget *parent)
-	:eWidget(parent), m_prev_scrollbar_page(-1), m_content_changed(false)
-	, m_scrollbar_mode(showNever), m_scrollbar(NULL)
+	:eWidget(parent), m_scrollbar_mode(showNever), m_prev_scrollbar_page(-1)
+	,m_content_changed(false), m_top(0), m_selected(0), m_itemheight(25)
+	,m_items_per_page(0), m_selection_enabled(1), m_scrollbar(NULL)
 {
 	setContent(new eListboxStringContent());
 
 	ePtr<eActionMap> ptr;
 	eActionMap::getInstance(ptr);
-	
-	m_itemheight = 25;
-	m_selection_enabled = 1;
-	
 	ptr->bindAction("ListboxActions", 0, 0, this);
 }
 
 eListbox::~eListbox()
 {
+	if (m_scrollbar)
+		delete m_scrollbar;
+	
 	ePtr<eActionMap> ptr;
 	eActionMap::getInstance(ptr);
 	ptr->unbindAction(this, 0);
@@ -28,12 +28,15 @@ eListbox::~eListbox()
 void eListbox::setScrollbarMode(int mode)
 {
 	m_scrollbar_mode = mode;
-	if ( m_scrollbar_mode == showNever && m_scrollbar )
+	if ( m_scrollbar )
 	{
-		delete m_scrollbar;
-		m_scrollbar=0;
+		if ( m_scrollbar_mode == showNever )
+		{
+			delete m_scrollbar;
+			m_scrollbar=0;
+		}
 	}
-	else if (!m_scrollbar)
+	else
 	{
 		m_scrollbar = new eSlider(this);
 		m_scrollbar->hide();
@@ -136,6 +139,8 @@ void eListbox::moveSelection(int dir)
 		/* m_top should be always valid here as it's selected */
 		m_top += m_items_per_page;
 
+	updateScrollBar();
+
 	if (m_top != oldtop)
 		invalidate();
 	else if (m_selected != oldsel)
@@ -147,20 +152,29 @@ void eListbox::moveSelection(int dir)
 		
 		invalidate(inv);
 	}
-
-	if (m_scrollbar_mode != showNever)
-		updateScrollBar();
 }
 
 void eListbox::moveSelectionTo(int index)
 {
-	m_content->cursorHome();
-	m_content->cursorMove(index);
-	moveSelection(justCheck);
+	if ( m_content )
+	{
+		m_content->cursorHome();
+		m_content->cursorMove(index);
+		moveSelection(justCheck);
+	}
+}
+
+int eListbox::getCurrentIndex()
+{
+	if ( m_content && m_content->cursorValid() )
+		return m_content->cursorGet();
+	return 0;
 }
 
 void eListbox::updateScrollBar()
 {
+	if (!m_content || m_scrollbar_mode == showNever )
+		return;
 	int entries = m_content->size();
 	if ( m_content_changed )
 	{
@@ -177,31 +191,29 @@ void eListbox::updateScrollBar()
 			m_scrollbar->move(ePoint(width-sbarwidth, 0));
 			m_scrollbar->resize(eSize(sbarwidth, height));
 			m_content->setSize(eSize(width-sbarwidth-5, m_itemheight));
-			if ( !m_scrollbar->isVisible() )
-				m_scrollbar->show();
+			m_scrollbar->show();
 		}
-		else if ( m_scrollbar_mode != showAlways )
+		else
 		{
-			if ( m_scrollbar->isVisible() )
-			{
-				m_content->setSize(eSize(width, m_itemheight));
-				m_scrollbar->hide(); // why this hide dont work???
-			}
+			m_content->setSize(eSize(width, m_itemheight));
+			m_scrollbar->hide();
 		}
 	}
-	int curVisiblePage = m_top / m_items_per_page;
-	if ( m_scrollbar->isVisible() &&
-		m_prev_scrollbar_page != curVisiblePage)
+	if ( m_items_per_page && entries )
 	{
-		m_prev_scrollbar_page = curVisiblePage;
-		int pages = entries / m_items_per_page;
-		if ( (pages*m_items_per_page) < entries )
-			++pages;
-		int start=(m_top*100)/(pages*m_items_per_page);
-		int vis=(m_items_per_page*100)/(pages*m_items_per_page);
-		if (vis < 3)
-			vis=3;
-		m_scrollbar->setStartEnd(start,start+vis);
+		int curVisiblePage = m_top / m_items_per_page;
+		if (m_prev_scrollbar_page != curVisiblePage)
+		{
+			m_prev_scrollbar_page = curVisiblePage;
+			int pages = entries / m_items_per_page;
+			if ( (pages*m_items_per_page) < entries )
+				++pages;
+			int start=(m_top*100)/(pages*m_items_per_page);
+			int vis=(m_items_per_page*100)/(pages*m_items_per_page);
+			if (vis < 3)
+				vis=3;
+			m_scrollbar->setStartEnd(start,start+vis);
+		}
 	}
 }
 
@@ -247,6 +259,7 @@ int eListbox::event(int event, void *data, void *data2)
 	case evtChangedSize:
 		recalcSize();
 		return eWidget::event(event, data, data2);
+		
 	case evtAction:
 		if (isVisible())
 		{
@@ -262,8 +275,13 @@ int eListbox::event(int event, void *data, void *data2)
 void eListbox::recalcSize()
 {
 	m_content_changed=true;
+	m_prev_scrollbar_page=-1;
 	m_content->setSize(eSize(size().width(), m_itemheight));
 	m_items_per_page = size().height() / m_itemheight;
+
+	if (m_items_per_page < 0) /* TODO: whyever - our size could be invalid, or itemheigh could be wrongly specified. */
+ 		m_items_per_page = 0;
+
 	moveSelection(justCheck);
 }
 
@@ -312,7 +330,7 @@ void eListbox::entryRemoved(int index)
 		m_selected = m_content->cursorGet();
 
 	moveSelection(justCheck);
-	
+
 	if ((m_top <= index) && (index < (m_top + m_items_per_page)))
 	{
 			/* todo, calc exact invalidation... */
@@ -329,14 +347,17 @@ void eListbox::entryChanged(int index)
 	}
 }
 
-void eListbox::entryReset()
+void eListbox::entryReset(bool cursorHome)
 {
 	m_content_changed=true;
 	m_prev_scrollbar_page=-1;
-	if (m_content)
-		m_content->cursorHome();
-	m_top = 0;
-	m_selected = 0;
+	if ( cursorHome )
+	{
+		if (m_content)
+			m_content->cursorHome();
+		m_top = 0;
+		m_selected = 0;
+	}
 	moveSelection(justCheck);
 	invalidate();
 }