After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 144529 - gtk dialog is not localized
gtk dialog is not localized
Status: RESOLVED FIXED
Product: libgnomeui
Classification: Deprecated
Component: file-chooser
CVS HEAD
Other All
: High major
: future
Assigned To: Alexander Larsson
Nautilus Maintainers
Depends on:
Blocks:
 
 
Reported: 2004-06-17 12:57 UTC by Takao Fujiwara
Modified: 2004-12-22 21:47 UTC
See Also:
GNOME target: 2.8.0
GNOME version: 2.7/2.8


Attachments
patch for gtk+-2.4.1/gtk/gtkfilechooserdefault.c (578 bytes, patch)
2004-06-17 12:57 UTC, Takao Fujiwara
none Details | Review
patch for libgnomeui-2.6.0/file-chooser/gtkfilesystemgnomevfs.c (1.19 KB, patch)
2004-06-22 12:48 UTC, Takao Fujiwara
none Details | Review
simpler fix (605 bytes, patch)
2004-09-02 17:35 UTC, Christian Persch
committed Details | Review

Description Takao Fujiwara 2004-06-17 12:57:03 UTC
Description of Problem:
gtkdialog is not localized.

Steps to reproduce the problem:
1. Invoke gedit.
2. Choose [File] -> [Open]

Actual Results:
"Filesystem" in right pane in the dialog is not
localized.

Expected Results:
All is correctly internationalized.

How often does this happen? 
To open the dialog.

Additional Information:
I attached the patche.
Comment 1 Takao Fujiwara 2004-06-17 12:57:54 UTC
Created attachment 28794 [details] [review]
patch for gtk+-2.4.1/gtk/gtkfilechooserdefault.c

I added a patch.
Comment 2 Takao Fujiwara 2004-06-17 13:00:42 UTC
Sorry, I cannot file via general forms because Netscape 7.0 on Solaris outputs an 
error when submit bugs. So I submitted bugs via easy forms.

Comment 3 Matthias Clasen 2004-06-17 17:50:13 UTC
The patch is wrong, you can't stuff variables in _(), just literal strings. But
I can't even reproduce the problem here, "Filesystem" appears nicely localized
as "Dateisystem" in German. So maybe it is just that the translation for your
locale is incomplete ? Please reopen if you find out that there really is a bug.
Comment 4 Takao Fujiwara 2004-06-22 12:44:53 UTC
I found "Filesystem" is localized on SuSE Linux.
So this is a Solaris specific issue.

gtk_file_system_volume_get_display_name in gtk+-2.4.1/gtk/gtkfilesystem.c calls 
gtk_file_system_unix_volume_get_display_name in gtkfilesystemunix.c on Linux.

Then the "Filesystem" is localzed.

But gtk_file_system_volume_get_display_name calls 
gtk_file_system_gnome_vfs_volume_get_display_name in
libgnomeui-2.6.0/file-chooser/gtkfilesystemgnomevfs.c on Solaris.

Then the message is not localized because the domainname is not configured 
correctly.

So I attach the Solaris patch for libgnomeui.
Comment 5 Takao Fujiwara 2004-06-22 12:48:29 UTC
Created attachment 28929 [details] [review]
patch for libgnomeui-2.6.0/file-chooser/gtkfilesystemgnomevfs.c

I attached the patch.
Comment 6 Kjartan Maraas 2004-09-01 11:59:02 UTC
I see this same problem in gedit and gpdf (and probably all other apps) using
Fedora Core 3 test releases. This means it's not a Solaris specific problem at all.
Comment 7 Christian Persch 2004-09-02 17:35:40 UTC
Created attachment 31214 [details] [review]
simpler fix

should this just include gi18n-lib.h instead of gi18n.h ?
Comment 8 Federico Mena Quintero 2004-09-03 01:16:26 UTC
Oops, Christian is right.  It should use gi18n-lib.h.  Please go ahead and commit.
Comment 9 Kjartan Maraas 2004-09-03 18:53:57 UTC
Commited. Thanks for the prompt fix Christian.