Jump to content
Eternal Lands Official Forums
Sign in to follow this  
Lhibou

Translation via XML of the windowstittle

Recommended Posts

@@ -360,18 +370,17 @@
void display_storage_menu()
{

...

-  vscrollbar_add_extended(storage_win, 1200, NULL, 130, 10, 20, 192, 0, 1.0, 0.77f, 0.57f, 0.39f, 0, 1, 10);
+  vscrollbar_add_extended(storage_win, 1200, NULL, 130, 10, 20, 192, 0, 1.0, 0.77f, 0.57f, 0.39f, 0, 1, 30);
 	vscrollbar_add_extended(storage_win, 1201, NULL, 352, 10, 20, 192, 0, 1.0, 0.77f, 0.57f, 0.39f, 0, 1, 28);
 } else {
 	int i;

...

@@ -151,6 +153,14 @@
 	storage_items[i].quantity=0;
 }
 
+	if ( (cat-12) > 0)
+	{
+  vscrollbar_set_pos(storage_win, 1200, cat-12);
+	}
+	else
+	{
+  vscrollbar_set_pos(storage_win, 1200, 0);
+	}
 vscrollbar_set_pos(storage_win, 1201, 0);
}

These hunks don't belong to this patch, please remove them.

 

-	strncpy(var->shortname, shortname, 5);
+	strncpy(var->shortname, shortname, 7);

Please don't introduce new buffer overflows :P -- You have to change the size of the shortname field as well. Btw, we need a volunteer to replace these integer constants in favour of sizeof().

Anyone interested? :inquisitive:

 

With regards

Lachesis

Edited by Lachesis

Share this post


Link to post
Share on other sites

Storage is part of the patch 596 http://www.eternal-lands.com/forum/index.php?showtopic=17615

 

The length of the scrollbar was modified because we have more categories in the french version.

The second part is for showing the red selected categories.

 

I forgot to add the part related to the size of shortname (stats.h).

 

Patch resubmitted with some other translations (new_character windows, some buttons, stats.h, etc..), and I maintained the 2 above modifications.

Share this post


Link to post
Share on other sites
Btw, we need a volunteer to replace these integer constants in favour of sizeof().

Anyone interested? :hiya:

 

With regards

Lachesis

208403[/snapback]

What happened to a bit of grep/sed or awk action?

Share this post


Link to post
Share on other sites
What happened to a bit of grep/sed or awk action?

208588[/snapback]

Maybe that they're constants turning to sizeof calls? :P

Share this post


Link to post
Share on other sites
Btw, we need a volunteer to replace these integer constants in favour of sizeof().

Anyone interested? :)

208403[/snapback]

 

Ill have a go at it.

Edited by Zep

Share this post


Link to post
Share on other sites
Storage is part of the patch 596 http://www.eternal-lands.com/forum/index.php?showtopic=17615

The second part is for showing the red selected categories.

208469[/snapback]

This is only necessary if they are not visible at the moment, otherwise it's rather annoying. Can you change it so that it works that way? Thank you

 

With Regards

Lachesis

Share this post


Link to post
Share on other sites

Patch 639 now contains only the stuff needed for the translation, no more interference with patch 596.

 

Going to improve patch 596 later

 

Edit: sorry for the double post, thought that was not added. Error during the submit.

Edited by Lhibou

Share this post


Link to post
Share on other sites

Hi Lhibou,

 

I applied patch #639. Sorry for being picky, but if I had committed the new behavior (patch #596) without the team's blessing I would have gotten the bashings. Please make a new thread for patch #596 so everyone can comment. Thank you for it all!

 

With Regards

Lachesis

Share this post


Link to post
Share on other sites

Patch id: 682

 

A lot of messages translated, but the most important is the translation of the local commands and the possibility to redefine #, @ and / for the commands (usefull for players having azerty keyboard)

Share this post


Link to post
Share on other sites

patch id 757

 

some variables was not used in the creation interface.

 

Note: I see patch 751 removing the <ctrl-q>, I think it's not needed as the @ can be redefined via XML (implemented in a previous patch).

Share this post


Link to post
Share on other sites

Patch is incomplete, the code for loading the strings [translate.c] is missing.

 

With regards

Lachesis

 

P.S. I accepted patch #751 (masked by #ifdef WINDOWS), because that's the only way to avoid support requests by new users.

Edited by Lachesis

Share this post


Link to post
Share on other sites

Patch is incomplete, the code for loading the strings [translate.c] is missing.

 

With regards

Lachesis

 

Code for loading the string is allready in translate.c. Thus patch is complete.

 

Declarations were missing and translate.h.

Share this post


Link to post
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now
Sign in to follow this  

  • Recently Browsing   0 members

    No registered users viewing this page.

×