#4059 closed defect (bug) (wontfix)
L10N Support in Default Theme
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Milestone: | Priority: | low | |
| Severity: | normal | Version: | 2.1.2 |
| Component: | I18N | Keywords: | has-patch 2nd-opinion i18n |
| Focuses: | Cc: |
Description
See #3953 for details
Attachments (2)
Change History (16)
#2
@
19 years ago
- Keywords i18n added
- The beginning of comments-popup has a naked
printf, without__ - There are some places (for example "
« %link" and "%s at %s", %1$s to “%2$s”"), where we could use the new_cfunction (see #3687) - Some more changes, differenet from i18n have passed through your diff: some stylesheet modifications (Arabic fonts), for example.
- It is nice that the strings are in a different domain--the theme authors will be more and more inspired and will gain some know-how :-)
Very good work, Sewar! Let's hope Matt wouldn't mind letting this in.
#3
@
19 years ago
- Cc Sewar added
I have updated the patch, fixed the bug in comments-popup.php file, removed CSS related fixes (the Arabic fonts) and will add them in new ticket.
About the new _c function, it looks its development didn't finished yet.
#4
@
19 years ago
The _c() function has already landed in subversion. Just that nbachiyski propose to put the context part behind the real translatable string, instead of the current status (before the real string).
#6
@
19 years ago
Couple of places that looks curious to me:
- Why some of the ‘ and ’ are replaced by ' ?
- In sidebar.php, why inserting link breaks in between middle of certain sentences but not others?
#7
@
19 years ago
I have uploaded additional stuff on top of Sewar's patch:
- Header image and color page now localizable
- Mark more strings translatable and minor fixes here and there
Note: See
TracTickets for help on using
tickets.
I think it's a good idea. Makes sense that seeing it done in the default theme would encourage people to do it in their own themes.