Merge lp:~majcherlk/widelands/bug-1387784 into lp:widelands

[Lukasz]

Now when the text doesn't fit in the Warehouse window the font is changed to smaller one.

[SirVer (sirver)]

Welcome Lukasz and thanks for the patch :).

This patch is technically correct, but they drag a dependency on wui (widelands ui) into basic ui which is not desirable. basic ui is more fundamental than wui.

review: Needs Fixing

[GunChleoc (gunchleoc)]

We will have another method of fixing this once the fh1 branch has been merged. The new renderer creates a LayoutInfo object that will have the resulting size recorded, so we can hook into that.

I would also go for picking the compressed font before reducing the font size. This will be possible after the fonts branch has been merged.

[Lukasz (majcherlk)]

Thanks for information and suggestions,
I will try fix that after fonts branch merge.

[SirVer (sirver)]

I am setting this back to work in progress so that it gets out of the review queue. Please ping here if you continued working on it.

Unmerged revisions

7277. By Lukasz on 2014-11-24

using smaller font when text is overflowing

=== modified file 'src/ui_basic/textarea.cc'
--- src/ui_basic/textarea.cc	2014-11-22 10:18:20 +0000
+++ src/ui_basic/textarea.cc	2014-11-24 15:39:15 +0000
@@ -20,6 +20,7 @@
 #include "ui_basic/textarea.h"
 
 #include "graphic/font_handler.h"
+#include "wui/text_constants.h"
 
 namespace UI {
 
@@ -118,6 +119,16 @@
 	if (m_text == text)
 		return;
 
+	int font_size=UI_FONT_SIZE_SMALL;
+	m_textstyle.font=Font::get(UI_FONT_SMALL);
+
+	if(this->get_parent()->get_inner_w() != 0){
+		while(font_size>0 && m_textstyle.calc_bare_width(text) > this->get_parent()->get_inner_w()){
+			font_size--;
+			m_textstyle.font=Font::get(UI_FONT_NAME, font_size);
+		}
+	}
+
 	if (m_layoutmode == AutoMove)
 		collapse(); // collapse() implicitly updates