Martin Vermeer wrote:
> Better this way?

I'll keep you busy ;-)

The attached patch adds i10n support for the inset labels and gives them our 
new names. BTW why should the color of comment depend on note and why is the 
color of greyedout hardcoded?

Furthermore, I think that you should update the xforms dialog and add 
_("Strings") with the new names too. Actually, I think that
controller().params().type = fl_get_choice_text(dialog_->choice_type);
will just break when
_("Note|Comment|Greyedout");
gets translated.

Juergen.
Index: src/insets/insetnote.C
===================================================================
RCS file: /usr/local/lyx/cvsroot/lyx-devel/src/insets/insetnote.C,v
retrieving revision 1.35
diff -u -r1.35 insetnote.C
--- src/insets/insetnote.C	29 Jul 2003 12:20:04 -0000	1.35
+++ src/insets/insetnote.C	30 Jul 2003 13:44:31 -0000
@@ -41,7 +41,7 @@
 {
 	params_.type = label;
 	init();
-	setLabel(label);
+	setButtonLabel();
 }


@@ -91,11 +91,16 @@
 	font.decSize();
 	font.decSize();

-	setLabel(params_.type);
-	if (params_.type == "Note" || params_.type == "Comment") {
+	if (params_.type == "Note") {
+		setLabel(_("LyX Note"));
+		font.setColor(LColor::note);
+		setBackgroundColor(LColor::notebg);
+	} else if (params_.type == "Comment") {
+		setLabel(_("Comment"));
 		font.setColor(LColor::note);
 		setBackgroundColor(LColor::notebg);
 	} else {
+		setLabel(_("Greyed out"));
 		font.setColor(LColor::red);
 		setBackgroundColor(LColor::background);
 	}

Reply via email to