#19603 closed task (blessed) (fixed)
Support locale-specific modifications in core
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 3.4 | Priority: | normal |
Severity: | normal | Version: | |
Component: | I18N | Keywords: | has-patch commit |
Focuses: | Cc: |
Description
(See also #19598, #19599, #19600, #19601, #19602, which lead up to this ticket.)
WordPress should support, maintain, and ship with locale-specific modifications, acting on the advice of the translation teams.
I've gone through i18n.svn.wordpress.org. Some of these modifications include:
- Targeted custom CSS, such as font family optimizations for Hebrew (Arial), ug_CN, etc.
- Conversions to Latin characters (sr_RS and I think one other locale).
- Some current modifications that can be done either with existing gettext, or adding specific gettext cases to core.
- Some locales have plugins (some of which are extensive) -- in particular, the wp-multibyte-patch (ja), but also cxapelado and transliterado (eo), and wp-jalali (fa_IR).
Attachments (6)
Change History (34)
#3
in reply to:
↑ 2
@
13 years ago
Replying to scribu:
Isn't this what the locale.php file is for, though?
Yes, but the idea is to render it unnecessary. I'll be posting to wppolyglots shortly.
#4
@
13 years ago
And here is the announcement:
http://wppolyglots.wordpress.com/2011/12/18/moving-locale-specific-modifications-to-core/
#5
@
13 years ago
More thoughts:
Some locales also override a number of core CSS files (including for TinyMCE and editor styles), so we will need to see why, and act accordingly.
#7
@
13 years ago
- Keywords 2nd-opinion added
Still working on an inventory for all locale modifications. I hope to spin different aspects off into specific tickets. For now, a test of converting to Arial for he_IL, piggybacked on the existing RTL stylesheet - 19603.diff.
#29
@
13 years ago
I did an initial few sweeps through all locales over the last week. That has resulted in a number of commits across a swath of 3.4 i18n tickets. I've then gone through and cleaned up each i18n directory (in i18n.svn.wordpress.org) that needed it, across around 75 commits.
For every locale that has received a single 3.3 download, all are clear, except the following:
- zh_CN — css, word count, locale settings
- ru_RU — css, spellchecker
- ja — multibyte plugin (http://wordpress.org/extend/plugins/wp-multibyte-patch/)
- fa_IR — plugin and hacks (http://wordpress.org/extend/plugins/wp-jalali/)
- he_IL — css, feed text direction
- ca — date filter
- sr_RS — plugin for transliteration, locale settings
- ug_CN — hacks, mainly for fonts
- eo — transliteration plugins (http://wordpress.org/extend/plugins/transliterado/)
Each locale link goes to their repo. And by "clear," I mean they have at most wp-config-sample.php, a readme, and a license in their language.
#35
@
13 years ago
- Cc kaipgis@… added
Could you add some lithuanian CSS?
We need some more space for our long words.
/* lt_LT: QuickPress + Bulk Edit */ #dashboard_quick_press .input-text-wrap, #dashboard_quick_press .textarea-wrap, #dashboard_quick_press .wp-media-buttons { margin-left: 6.7em; } #dashboard_quick_press h4 { width: 6.7em; } .inline-edit-row fieldset label span.title { width: 8em; } .inline-edit-row fieldset label span.input-text-wrap { margin-left: 8em; }
And consider making these QuickEdit and QuickPress areas wider be default. It is too narrow for lithuanian, russian and probably some other languages.
#36
@
13 years ago
URL-encoded comment_author_url gets broken by MySQL varchar 200 length limit. Please take a look at #17771 if there is a chance.
#39
follow-up:
↓ 40
@
13 years ago
- Cc pavelevap@… added
I found another problem which should be solved in the case of removing locale.php files
http://core.trac.wordpress.org/ticket/10373
Should I reopen some of these tickets?
#41
@
13 years ago
Great, thank you. I would like to mention another open ticket, which is related to locale.php support: http://core.trac.wordpress.org/ticket/13651
We have to use comments_number filter and str_replace() now.
#42
@
13 years ago
- Keywords has-patch added
- Resolution fixed deleted
- Status changed from closed to reopened
Some ru_RU-specific CSS in [19826] was meant for Press This.
Turned out this wasn't enough, because unlike admin-header.php
, press-this.php
doesn't apply locale class to <body>
element.
Currently Press This looks like this in ru_RU: 19603.press-this-ru_RU.png (sidebar is broken).
19603.press-this.patch fixes the issue: 19603.press-this-ru_RU.after.png.
#44
@
13 years ago
- Owner set to ryan
- Resolution set to fixed
- Status changed from reopened to closed
In [21052]:
Isn't this what the locale.php file is for, though?