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

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

@1ec5

Description

@1ec5

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

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions