Closed Bug 1322989 Opened 8 years ago Closed 7 years ago

When applying text-transform:uppercase to Greek, the accent on disjunctive eta (ή) should not be removed

Categories

(Core :: Internationalization, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla53
Tracking Status
firefox53 --- fixed

People

(Reporter: jfkthame, Assigned: jfkthame)

References

(Blocks 1 open bug, )

Details

(Keywords: dev-doc-complete)

Attachments

(2 files)

This was mentioned in bug 307039 (see discussion at bug 307039 comment 65 and following), but never followed up.

One of the CSS Text testcases (text-transform-tailoring-005) covers this; see the URL field. Currently, we fail that test.
Assuming this is a behavior we want to support, here's a patch that fixes it so that we pass the related CSSWG testcase.
Attachment #8819279 - Flags: review?(m_kato)
Assignee: nobody → jfkthame
Status: NEW → ASSIGNED
And a simple reftest that exercises this.
Attachment #8819280 - Flags: review?(m_kato)
Attachment #8819279 - Flags: review?(m_kato) → review+
Attachment #8819280 - Flags: review?(m_kato) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/422af49b7ae8cb9218457535c8b0d3f4bec5f467
Bug 1322989 - Preserve accent on Greek disjunctive eta (ή) when applying text-transform:uppercase. r=m_kato

https://hg.mozilla.org/integration/mozilla-inbound/rev/f9455be3e7f59f00bfc209aebc471073e23f20a7
Bug 1322989 - Reftest for uppercasing of Greek disjunctive eta (ή). r=m_kato
Oops, sorry - I forgot we require those to be annotated these days. Will fix.
Flags: needinfo?(jfkthame)
https://hg.mozilla.org/integration/mozilla-inbound/rev/d12ee8da73bc3448251d07b284a698a7b336842a
Bug 1322989 - Preserve accent on Greek disjunctive eta (ή) when applying text-transform:uppercase. r=m_kato

https://hg.mozilla.org/integration/mozilla-inbound/rev/4d651df764664db0b6be181a7da57d8e5435e508
Bug 1322989 - Reftest for uppercasing of Greek disjunctive eta (ή). r=m_kato
https://hg.mozilla.org/mozilla-central/rev/d12ee8da73bc
https://hg.mozilla.org/mozilla-central/rev/4d651df76466
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
I've made sure that this requirement is successfully described on the text-transform ref page:
https://developer.mozilla.org/en-US/docs/Web/CSS/text-transform

I've also added a note to the Fx53 release notes:
https://developer.mozilla.org/en-US/Firefox/Releases/53#CSS

Let me know if you think anything else is required for the docs. Thanks!
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: