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.

Sign up for GitHub

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

Jump to bottom

Collision boxes don't appear until you move map #4647

Closed
aparlato opened this issue Apr 26, 2017 · 1 comment · Fixed by #4689
Closed

Collision boxes don't appear until you move map #4647

aparlato opened this issue Apr 26, 2017 · 1 comment · Fixed by #4689
Labels
bug 🐞 good first issue

Comments

@aparlato
Copy link

mapbox-gl-js version: All versions above 0.33.0

Minimal demo of bug

No errors or warnings.

Steps to Trigger Behavior

  1. Apply map.showCollisionBoxes = true (happens on load in demo)
  2. No collision boxes appear
  3. Move map
  4. Collision boxes appear

Expected Behavior

Collision boxes show up on map.showCollisionBoxes = true

Actual Behavior

Collision boxes show up only after moving map when map.showCollisionBoxes = true
@mollymerp
Copy link
Contributor

Bisected and #4347 seems to be the culprit ☹️

@mollymerp mollymerp mentioned this issue May 9, 2017
Merged
5 tasks
ChrisLoer added a commit that referenced this issue Aug 15, 2017
@ChrisLoer
Update #4647 regression test to use new collision box color.
1b3e866 
Update Chinese tests to use new collision algorithm (not behavior preserving)
ChrisLoer added a commit that referenced this issue Aug 15, 2017
@ChrisLoer
Update #4647 regression test to use new collision box color.
557295c 
Update Chinese tests to use new collision algorithm (not behavior preserving)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug 🐞 good first issue
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

showCollisionBoxes fix mapbox/mapbox-gl-js
3 participants
@mourner @mollymerp @aparlato