Skip to content
This repository has been archived by the owner on Apr 10, 2018. It is now read-only.

Rename circle-pitch-scale to circle-scale-alignment #645

Closed
1ec5 opened this issue Jan 5, 2017 · 2 comments
Closed

Rename circle-pitch-scale to circle-scale-alignment #645

1ec5 opened this issue Jan 5, 2017 · 2 comments

Comments

@1ec5
Copy link
Contributor

1ec5 commented Jan 5, 2017

circle-pitch-scale should be renamed circle-scale-alignment.

#459 (comment) proposed {circle,icon,text}-pitch-scale properties as complements to the {circle,icon,text}-{rotation,pitch}-alignment properties. As discussed in mapbox/mapbox-gl-native#6098, the circle-pitch-scale name incorrectly implies a numeric value. It’s also inconsistent with the rest of the properties:

Property a feature’s according to the map or viewport’s when the map has
circle-rotation-alignment aligns rotation rotation rotation
circle-pitch-alignment aligns pitch pitch pitch
circle-pitch-scale aligns scale scale pitch

Renaming circle-pitch-scale to circle-scale-alignment would make the property’s type more obvious and better harmonize the name with the others:

Property does what a feature’s according to the map or viewport’s when
circle-scale-alignment aligns scale scale pitch

/cc @jfirebaugh @lucaswoj

@1ec5
Copy link
Contributor Author

1ec5 commented Jan 5, 2017

An alternative to *-scale-alignment would be *-size-alignment, which would be consistent with icon-size and text-size.

@lucaswoj
Copy link

lucaswoj commented Feb 1, 2017

This issue was moved to mapbox/mapbox-gl-js#4165

@lucaswoj lucaswoj closed this as completed Feb 1, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants