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

Introduce 'text-writing-mode' layout property that allows to control orientation of point labels #8372

Closed
wants to merge 3 commits into from

Conversation

alexshalamov
Copy link
Contributor

@alexshalamov alexshalamov commented Jun 20, 2019

Brief description

For languages that support vertical writing mode, map designers might want point labels to be rendered vertically. This PR adds definition for text-writing-mode layout property that controls preferred orientation of a symbol. The new property's value is an array, whose values are enumeration values from a ( horizontal | vertical ) set. PR also adds render tests for newly added property.

Native implementation mapbox/mapbox-gl-native#14932

Launch Checklist

  • briefly describe the changes in this PR
  • write tests for all new functionality
  • document any changes to public APIs
  • post benchmark scores
  • manually test the debug page
  • tagged @mapbox/studio and/or @mapbox/maps-design if this PR includes style spec changes

@alexshalamov alexshalamov requested a review from vakila June 20, 2019 09:52
@alexshalamov alexshalamov force-pushed the alexshalamov_wip_vertical_labels branch from c0211a3 to 069c6f0 Compare June 20, 2019 10:02
"doc": "If a text's language supports horizontal writing mode, symbols with point placement would be laid out horizontally."
},
"vertical": {
"doc": "If a text's language supports vertical writing mode, symbols with point placement would be laid out vertically."
Copy link
Contributor

Choose a reason for hiding this comment

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

It would be useful to hint what languages are supported with this property.

@alexshalamov
Copy link
Contributor Author

@asheemmamoowala @vakila should I close this PR? I think it is no longer required, all of the changes are in #8399

@alexshalamov alexshalamov deleted the alexshalamov_wip_vertical_labels branch July 9, 2019 08:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants