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 309657 - Capitalization Fixes Patch
Capitalization Fixes Patch
Status: RESOLVED FIXED
Product: GIMP
Classification: Other
Component: User Interface
2.3.x
Other Linux
: Normal minor
: 2.4
Assigned To: GIMP Bugs
GIMP Bugs
Depends on:
Blocks:
 
 
Reported: 2005-07-07 00:06 UTC by Sven Neumann
Modified: 2008-01-15 14:04 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
patch to fix capitalization issues (114.65 KB, patch)
2005-07-07 00:08 UTC, Sven Neumann
needs-work Details | Review
ellipsis (2.23 KB, patch)
2005-08-16 20:41 UTC, Sven Neumann
committed Details | Review
labels (41.88 KB, patch)
2005-08-16 20:43 UTC, Sven Neumann
committed Details | Review
no or wrong booktitle style (44.51 KB, patch)
2005-08-16 20:43 UTC, Sven Neumann
committed Details | Review
no sentence style (23.90 KB, patch)
2005-08-16 20:44 UTC, Sven Neumann
committed Details | Review

Description Sven Neumann 2005-07-07 00:06:58 UTC
Filing this bug report on behave of Stephan Binner who mailed me this patch.
Attaching it here so that it can be reviewed in detail and is not forgotten.
From a first glimpse at the patch it looks like it should just be committed.
Comment 1 Sven Neumann 2005-07-07 00:08:43 UTC
Created attachment 48759 [details] [review]
patch to fix capitalization issues
Comment 2 Alan Horkan 2005-07-07 00:56:13 UTC
Although it may seem that most of the time Title Case is not exactly the same as
capitalising the first letter of every word.  

Short prepositions like "from", "with", "to", "by" and others are not normally
capitalised and there are various standard style rules to this effect.  Some
applications do not always follow these rules but it seems to be largely
unintentional.  

The Gnome developers have been debating capitalising a few exceptional cases for
example make the following change s/from/to
s/Close without Saving/Close Without Saving/
but the general rule holds and that much of the patch is incorrect.  

Some of the changes require a lot more context to decide.  Title Case is for
titles, not the longer more descriptive strings which I suspect some of these
may be.  I will have to take a closer look tommorow and check the context
carefully.  I'll recheck the Gnome style guide as well to be sure.  

(The suggest capitalisation change to the Tips Dialog buttons is correct but I'd
recommend using the GTK Stock Next and Previous to remove the need to translate
these buttons, and instead get the free Stock translations.)

I'm even more uncertain about the cases where capitalisation has been.  

Reading this patch throws up a lot of minor issues and places where strings
could be made more consistent (by removing unnecessary similar variations on the
same string).  

This is not the correct English spelling of fractal
+        gtk_file_chooser_dialog_new (_("Save as IFS Fraktal File"),


Were all these changes made manually or using some clever scripting?  

I'll look at this again more carefully tomorrow.  
Comment 3 Sven Neumann 2005-07-07 01:04:11 UTC
Setting version to 2.3.2. The patch applies cleanly against this version.
Comment 4 Sven Neumann 2005-07-08 16:48:44 UTC
I've fixed the wrong spelling of Fraktal in the meantime. But that is not really
what we are discussing here.

I agree with Alan's comments. The patch goes too far in many places. The HIG
asks for header capitalization and header style doesn't mean that each word
should be written upper-case. There are certainly some strings that should be
fixed, mainly button labels that don't follow the capitalization rules. Someone
could try to split this into smaller, less controversial, patches.
Comment 5 Sven Neumann 2005-08-16 20:41:54 UTC
Created attachment 50811 [details] [review]
ellipsis
Comment 6 Sven Neumann 2005-08-16 20:43:05 UTC
Created attachment 50812 [details] [review]
labels
Comment 7 Sven Neumann 2005-08-16 20:43:42 UTC
Created attachment 50813 [details] [review]
no or wrong booktitle style
Comment 8 Sven Neumann 2005-08-16 20:44:17 UTC
Created attachment 50814 [details] [review]
no sentence style
Comment 9 Sven Neumann 2005-08-16 20:46:17 UTC
Stephan has split the patch into smaller parts. I would very much appreciate if
Alan could review these smaller patches. Setting on the 2.4 milestone so that
this isn't forgotten.
Comment 10 Sven Neumann 2005-08-22 23:33:00 UTC
2005-08-23  Sven Neumann  <sven@gimp.org>

	* app/actions/gradients-actions.c
	* app/actions/patterns-actions.c
	* plug-ins/metadata/interface.c: applied patch from Stephan Binner
	that fixes wrong use of ellipsis (bug #309657).
Comment 11 Sven Neumann 2005-08-23 11:18:36 UTC
2005-08-23  Sven Neumann  <sven@gimp.org>

        * [lots of files]: applied capitalization patches contributed by
        Stephan Binner. Fixes bug #309657.