Opened 15 years ago
Closed 15 years ago
#5563 closed enhancement (fixed)
[Patch] ShowImage app localization
Reported by: | mt | Owned by: | stippi |
---|---|---|---|
Priority: | normal | Milestone: | R1 |
Component: | Kits/Locale Kit | Version: | R1/Development |
Keywords: | ShowImage | Cc: | |
Blocked By: | Blocking: | ||
Platform: | All |
Description
Localize GUI strings for ShowImage app. Patch and en.catkeys added.
Attachments (4)
Change History (11)
by , 15 years ago
Attachment: | ShowImage.patch added |
---|
by , 15 years ago
Attachment: | en.catkeys added |
---|
comment:1 by , 15 years ago
Owner: | changed from | to
---|---|
Status: | new → in-progress |
The patch looks pretty good, thanks a lot!
Just some possible improvements for future patches:
- You could include ControlLook.h and use be_control_look->DefaultItemSpacing() instead of hardcoding 5 or so for layout item spacing. I.e. "float spacing = be_control_look->DefaultItemSpacing();" once and then using "spacing" everywhere.
- The bs_printf() function has some tabs or weird spacing between type and variable names.
- The coding cleanups are greatly appreciated. In some places you could also improve variable names to be slightly more descriptive, but it would certainly not prevent me from applying this patch.
Thanks for the good work you are doing, mt. IMHO you should be proposed for commit access soon. :-)
comment:2 by , 15 years ago
I applied be_control_look->DefaultItemSpacing() and modified bs_printf(). please review.
follow-up: 4 comment:3 by , 15 years ago
Resolution: | → fixed |
---|---|
Status: | in-progress → closed |
Thanks a bunch! Applied in hrev35929. Don't know what to do with en.catkeys. Isn't that file auto-generated?
comment:4 by , 15 years ago
by , 15 years ago
Attachment: | ShowImageUpdate.patch added |
---|
Another localization update for ShowImage app.
comment:5 by , 15 years ago
Resolution: | fixed |
---|---|
Status: | closed → reopened |
I found some strings not yet localized in ShowImage app. I compiled ShowImageUpdate.patch to fix these. This is my first patch, so I do not know, if I compiled it correct.
comment:6 by , 15 years ago
The patch is the right direction in principle, but the string composition needs to be changed. One sentence with a variable in it should be translatable as one sentence, so other translations can put the variable at any place. I know this is still possible if the sentence is composed of just three parts, but it's not good style to keep it like that. It can be changed for example by using a stack allocated buffer and using snprintf().
comment:7 by , 15 years ago
Resolution: | → fixed |
---|---|
Status: | reopened → closed |
Fixed the issues myself and applied the patch in hrev36647, thanks!
patch file