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 101760 - Remove unnecessary markup from glade messages in gnome-utils
Remove unnecessary markup from glade messages in gnome-utils
Status: RESOLVED FIXED
Product: gnome-utils
Classification: Deprecated
Component: gfloppy
trunk
Other All
: Normal normal
: ---
Assigned To: gnome-utils Maintainers
gnome-utils Maintainers
Depends on: 97061
Blocks:
 
 
Reported: 2002-12-21 22:12 UTC by Christian Rose
Modified: 2005-01-24 15:55 UTC
See Also:
GNOME target: ---
GNOME version: 2.9/2.10


Attachments
Proposed patch (4.34 KB, patch)
2005-01-15 04:15 UTC, Michael Terry
none Details | Review

Description Christian Rose 2002-12-21 22:12:16 UTC
#: gfloppy/src/gfloppy2.glade.h:4
msgid "<span weight=\"bold\">Filesystem settings</span>"

#: gfloppy/src/gfloppy2.glade.h:5
msgid "<span weight=\"bold\">Formatting mode</span>"

#: gfloppy/src/gfloppy2.glade.h:6
msgid "<span weight=\"bold\">Physical settings</span>"


This use of markup inside messages marked for translation is totally
unnecessary, and a problem for the translation process. For details, see
the problem description in bug 96836. Since these are glade messages, I'll
mark it dependant on the glade bug 97061 in the mean time.
Comment 1 Glynn Foster 2002-12-28 16:38:38 UTC
I'm not going to have time to fix this before the 31st dec [string 
freeze] - anyone who has time, please commit patches to HEAD.
Comment 2 Glynn Foster 2003-01-03 22:45:10 UTC
missed the deadline on this one.
Comment 3 Glynn Foster 2003-11-10 01:17:34 UTC
What should be done about this b.t.w.? Is the correct thing just to
remove the markup? or .. ?
Comment 4 Christian Rose 2003-11-10 11:46:37 UTC
The correct thing to do is to fix Glade (see bug 97061).

However, from a translation standpoint, simply removing the markup is
fine too, but then you may not be following the HIG any longer if it
says that these messages should be bold.
Comment 5 Michael Terry 2005-01-15 04:15:56 UTC
Created attachment 36048 [details] [review]
Proposed patch

This patch removes the markup from the glade file and adds it when we are
setting up widgets in main.c

It has one change which may have unintended consequences.  I didn't see any
myself, but I am not familiar with the code.  I had to move the init_devices()
call to lower in the function, after all the widget setup code.  Otherwise, if
you don't have a device, you'll see a greyed out main window without bolded
headers.  I think this move is good even without the rest of the patch though,
as the code sets the widgets up in other ways besides bolding, and if we are
presenting the window to the user at all, we shouldn't show them a
half-finished window.
Comment 6 Vincent Noel 2005-01-18 15:54:43 UTC
James, what do you think of the patch ?
Comment 7 Vincent Noel 2005-01-24 15:55:20 UTC
Michael, I committed your patch in order to get it into 2.9.90. Thanks !