X-Git-Url: http://git.tdb.fi/?a=blobdiff_plain;f=source%2Fwidget.cpp;h=2183be6623b5467b0d1431f531a2cea6cb534897;hb=a122a209782e5b5400b5a70bd23ce4feadc6b36b;hp=9dbd9da5ba2df7f60f44d824a8a7989d11d917dd;hpb=fdc7fecc65f5f517d66abe3546a949a46836c4a6;p=libs%2Fgltk.git

diff --git a/source/widget.cpp b/source/widget.cpp
index 9dbd9da..2183be6 100644
--- a/source/widget.cpp
+++ b/source/widget.cpp
@@ -30,38 +30,47 @@ Widget::~Widget()
 
 void Widget::set_position(int x, int y)
 {
-	geom.x = x;
-	geom.y = y;
-	on_geometry_change();
-	rebuild();
+	set_geometry(Geometry(x, y, geom.w, geom.h));
 }
 
 void Widget::set_size(unsigned w, unsigned h)
 {
-	geom.w = w;
-	geom.h = h;
-	on_geometry_change();
-	rebuild();
+	set_geometry(Geometry(geom.x, geom.y, w, h));
 }
 
 void Widget::autosize()
 {
-	geom.w = 0;
-	geom.h = 0;
+	if(!style)
+		return;
+
+	Geometry ageom;
 	const Style::PartSeq &parts = style->get_parts();
 	for(Style::PartSeq::const_iterator i=parts.begin(); i!=parts.end(); ++i)
+	{
 		if(i->get_name().empty())
 		{
-			geom.w = max(geom.w, i->get_geometry().w);
-			geom.h = max(geom.h, i->get_geometry().h);
+			const Geometry &pgeom = i->get_geometry();
+			const Sides &pmargin = i->get_margin();
+			ageom.w = max(ageom.w, pgeom.w+pmargin.left+pmargin.right);
+			ageom.h = max(ageom.h, pgeom.h+pmargin.top+pmargin.bottom);
 		}
+		else
+			autosize_special(*i, ageom);
+	}
+
+	set_geometry(ageom);
 }
 
 void Widget::set_geometry(const Geometry &g)
 {
+	bool size_changed = (g.w!=geom.w || g.h!=geom.h);
 	geom = g;
-	on_geometry_change();
-	rebuild();
+	if(size_changed)
+	{
+		// TODO maybe rename this to on_size_change
+		on_geometry_change();
+		rebuild();
+	}
 }
 
 void Widget::set_parent(Container *p)
@@ -70,10 +79,20 @@ void Widget::set_parent(Container *p)
 		throw hierarchy_error("widget already parented");
 	else if(p==parent)
 		return;
-	parent = p;
 
-	on_reparent();
-	update_style();
+	try
+	{
+		parent = p;
+
+		on_reparent();
+		update_style();
+	}
+	catch(...)
+	{
+		// The container has not yet added the widget as its child
+		parent = 0;
+		throw;
+	}
 }
 
 void Widget::set_style(const string &s)
@@ -147,19 +166,22 @@ void Widget::rebuild()
 	if(!style)
 		return;
 
+	part_cache.clear();
 	const Style::PartSeq &parts = style->get_parts();
-	list<CachedPart>::iterator j = cached_parts.begin();
-	for(Style::PartSeq::const_iterator i=parts.begin(); i!=parts.end(); ++i, ++j)
+	for(Style::PartSeq::const_iterator i=parts.begin(); i!=parts.end(); ++i)
 	{
-		if(j==cached_parts.end())
-			j = cached_parts.insert(j, CachedPart());
 		if(i->get_name().empty())
-			i->build(geom, state, *j);
+			i->build(geom, state, part_cache);
 		else
-			rebuild_special(*i, *j);
+			rebuild_special(*i);
 	}
 }
 
+void Widget::rebuild_special(const Part &part)
+{
+	part_cache.insert_special(part);
+}
+
 void Widget::render(GL::Renderer &renderer) const
 {
 	if(!style)
@@ -167,17 +189,16 @@ void Widget::render(GL::Renderer &renderer) const
 
 	GL::MatrixStack::Push _pushm(renderer.matrix_stack());
 	renderer.matrix_stack() *= GL::Matrix::translation(geom.x, geom.y, 0);
-	const Style::PartSeq &parts = style->get_parts();
-	list<CachedPart>::const_iterator j = cached_parts.begin();
-	for(Style::PartSeq::const_iterator i=parts.begin(); (i!=parts.end() && j!=cached_parts.end()); ++i, ++j)
+	const PartCache::PartList &parts = part_cache.get_parts();
+	for(PartCache::PartList::const_iterator i=parts.begin(); i!=parts.end(); ++i)
 	{
-		if(j->mesh && j->texture)
+		if(i->mesh && i->texture)
 		{
-			renderer.set_texture(j->texture);
-			j->mesh->draw(renderer);
+			renderer.set_texture(i->texture);
+			i->mesh->draw(renderer);
 		}
-		else if(!i->get_name().empty())
-			render_special(*i, renderer);
+		else if(i->part)
+			render_special(*i->part, renderer);
 	}
 }