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

Wax's prefixed class names. #22

Closed
tristen opened this issue Jul 9, 2012 · 2 comments
Closed

Wax's prefixed class names. #22

tristen opened this issue Jul 9, 2012 · 2 comments

Comments

@tristen
Copy link
Member

tristen commented Jul 9, 2012

To be fixed upstream. Currently elements created via mapbox.js uses a prefixed class name like wax-zoomer etc ..
Which is ambiguous to users styling elements and coming from mapbox (not knowing about wax)

Prefixing a name to these controls is the right way to go but we may want a new name for this other than wax. A potential candidate could be .ui-zoomer or .map-zoomer.

@yhahn @tmcw curious on your thoughts here.

@yhahn
Copy link
Member

yhahn commented Jul 9, 2012

Agreed, I'm fine with any prefix that doesn't piss anyone off.

@tristen
Copy link
Member Author

tristen commented Jul 10, 2012

Added.

@tristen tristen closed this as completed Jul 10, 2012
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants