Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement localized CSS quotes #49835

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

servo-wpt-sync
Copy link
Collaborator

@servo-wpt-sync servo-wpt-sync commented Dec 24, 2024

This PR partially implements CSS quotes (#34446):

  • For non-auto quotes value, retrieves the quote data from the property value
  • For auto value:
    • Hooks up the -x-lang internal property to track the computed lang attribute value
    • Uses -x-lang value to determine current locale
    • Finds out the correct quotes data according to the ICU CLDR data

It also removes the stale and incorrect UA sheet quotes.css that is not used anywhere.

Quote depth calculation is out of the scope of this PR and will be left for future.

Other caveats:

  • -x-lang doesn't track the browser/OS-default locale
  • The quotes data is copy-pasted from the Gecko implementation instead of generated automatically

Reviewed in servo/servo#34770

Copy link
Collaborator

@wpt-pr-bot wpt-pr-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The review process for this patch is being conducted in the Servo project.

@servo-wpt-sync servo-wpt-sync force-pushed the servo_export_34770 branch 3 times, most recently from 088c88d to 278fe6b Compare January 14, 2025 08:20
@servo-wpt-sync servo-wpt-sync changed the title WIP implement localized quotes Implement localized CSS quotes Jan 14, 2025
@servo-wpt-sync servo-wpt-sync force-pushed the servo_export_34770 branch 2 times, most recently from a2272d5 to 6268be8 Compare January 15, 2025 10:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants