You're right (of course). I reverted it. J'
On Sat, Feb 18, 2012 at 09:46:16AM -0800, Ben Pfaff wrote:
I believe that this commit (quoted below) is incorrect. The
names that it causes to be translated are the "id" members of
toggle buttons in the UI file. If you translate those, then
get_widget_assert() will assert-fail, because the toggle buttons
will not be found.
Here's an example from text-data-import.ui. Notice how the "id"
is "space", which should not be translated. The "label" is
"_Space" and is marked for translation:
<object class="GtkCheckButton" id="space">
<property name="label" translatable="yes">_Space</property>
<property name="visible">True</property>
<property name="can_focus">True</property>
<property name="receives_default">False</property>
<property name="events">GDK_POINTER_MOTION_MASK |
GDK_POINTER_MOTION_HINT_MASK | GDK_BUTTON_PRESS_MASK |
GDK_BUTTON_RELEASE_MASK</property>
<property name="use_underline">True</property>
<property name="draw_indicator">True</property>
</object>
If I properly understand the change that your commit makes, then
will you please revert it?
Thanks,
Ben.
--8<--------------------------cut here-------------------------->8--
From: John Darrington <[email protected]>
Date: Mon, 13 Feb 2012 18:35:53 +0100
Subject: [PATCH] Mark separator names for translation
---
src/ui/gui/text-data-import-dialog.c | 20 ++++++++++----------
1 files changed, 10 insertions(+), 10 deletions(-)
diff --git a/src/ui/gui/text-data-import-dialog.c
b/src/ui/gui/text-data-import-dialog.c
index 4f9e5d6..547105f 100644
--- a/src/ui/gui/text-data-import-dialog.c
+++ b/src/ui/gui/text-data-import-dialog.c
@@ -1009,15 +1009,15 @@ struct separator
/* All the separators in the dialog box. */
static const struct separator separators[] =
{
- {"space", ' '},
- {"tab", '\t'},
- {"bang", '!'},
- {"colon", ':'},
- {"comma", ','},
- {"hyphen", '-'},
- {"pipe", '|'},
- {"semicolon", ';'},
- {"slash", '/'},
+ {N_("space"), ' '},
+ {N_("tab"), '\t'},
+ {N_("bang"), '!'},
+ {N_("colon"), ':'},
+ {N_("comma"), ','},
+ {N_("hyphen"), '-'},
+ {N_("pipe"), '|'},
+ {N_("semicolon"), ';'},
+ {N_("slash"), '/'},
};
#define SEPARATOR_CNT (sizeof separators / sizeof *separators)
@@ -1077,7 +1077,7 @@ init_separators_page (struct import_assistant *ia)
g_signal_connect (p->custom_cb, "toggled",
G_CALLBACK (on_separators_custom_cb_toggle), ia);
for (i = 0; i < SEPARATOR_CNT; i++)
- g_signal_connect (get_widget_assert (builder, separators[i].name),
+ g_signal_connect (get_widget_assert (builder,
gettext(separators[i].name)),
"toggled", G_CALLBACK (on_separator_toggle), ia);
g_signal_connect (p->escape_cb, "toggled",
G_CALLBACK (on_separator_toggle), ia);
--
1.7.2.5
--
Ben Pfaff
http://benpfaff.org
--
PGP Public key ID: 1024D/2DE827B3
fingerprint = 8797 A26D 0854 2EAB 0285 A290 8A67 719C 2DE8 27B3
See http://keys.gnupg.net or any PGP keyserver for public key.
signature.asc
Description: Digital signature
_______________________________________________ pspp-dev mailing list [email protected] https://lists.gnu.org/mailman/listinfo/pspp-dev
