Closed
Bug 1057330
Opened 10 years ago
Closed 10 years ago
add Vertical_Orientation to Unicode character property tables
Categories
(Core :: Internationalization, defect)
Core
Internationalization
Tracking
()
RESOLVED
FIXED
mozilla34
People
(Reporter: jfkthame, Assigned: jfkthame)
References
Details
Attachments
(3 files, 1 obsolete file)
6.46 KB,
patch
|
smontagu
:
review+
|
Details | Diff | Splinter Review |
813.48 KB,
patch
|
smontagu
:
review+
|
Details | Diff | Splinter Review |
1.25 KB,
patch
|
smontagu
:
review+
|
Details | Diff | Splinter Review |
This will be needed to support text-orientation:mixed in vertical writing modes.
Assignee | ||
Comment 1•10 years ago
|
||
Attachment #8478938 -
Flags: review?(smontagu)
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → jfkthame
Status: NEW → ASSIGNED
Assignee | ||
Comment 2•10 years ago
|
||
Attachment #8478939 -
Flags: review?(smontagu)
Assignee | ||
Comment 3•10 years ago
|
||
Attachment #8478940 -
Flags: review?(smontagu)
Assignee | ||
Comment 4•10 years ago
|
||
Sorry, uploaded a stale patch the first time.
Attachment #8478942 -
Flags: review?(smontagu)
Assignee | ||
Updated•10 years ago
|
Attachment #8478940 -
Attachment is obsolete: true
Attachment #8478940 -
Flags: review?(smontagu)
Comment 5•10 years ago
|
||
Comment on attachment 8478938 [details] [diff] [review]
pt 1 - update genUnicodePropertyData tool to add Vertical_Orientation property.
Review of attachment 8478938 [details] [diff] [review]:
-----------------------------------------------------------------
::: intl/unicharutil/tools/genUnicodePropertyData.pl
@@ +770,5 @@
> + unsigned char mEAW:3;
> + unsigned char mCategory:5;
> + unsigned char mBidiCategory:5;
> + unsigned char mXidmod:4;
> + signed char mNumericValue:5;
Nit: might as well line up mNumericValue with the other names (yes, I admit I'm obsessional/compulsive about this kind of thing)
Attachment #8478938 -
Flags: review?(smontagu) → review+
Comment 6•10 years ago
|
||
Comment on attachment 8478939 [details] [diff] [review]
pt 2 - regenerate Unicode character property tables with updated tool.
Review of attachment 8478939 [details] [diff] [review]:
-----------------------------------------------------------------
rs=me
Attachment #8478939 -
Flags: review?(smontagu) → review+
Updated•10 years ago
|
Attachment #8478942 -
Flags: review?(smontagu) → review+
Assignee | ||
Comment 7•10 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/0eb32e869ee3
https://hg.mozilla.org/integration/mozilla-inbound/rev/e0933bde618b
https://hg.mozilla.org/integration/mozilla-inbound/rev/7f9a9cfeabe7
Target Milestone: --- → mozilla34
Comment 8•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/0eb32e869ee3
https://hg.mozilla.org/mozilla-central/rev/e0933bde618b
https://hg.mozilla.org/mozilla-central/rev/7f9a9cfeabe7
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•