Skip to content
This repository has been archived by the owner on Aug 8, 2023. It is now read-only.

[core] Combination of text-variable-anchor and icon-text-fit is broken #15346

Closed
alexshalamov opened this issue Aug 9, 2019 · 0 comments · Fixed by #15367
Closed

[core] Combination of text-variable-anchor and icon-text-fit is broken #15346

alexshalamov opened this issue Aug 9, 2019 · 0 comments · Fixed by #15367
Assignees
@alexshalamov
Labels
Core The cross-platform C++ core, aka mbgl text rendering
Milestone
release-queso

Comments

@alexshalamov
Copy link
Contributor

alexshalamov commented Aug 9, 2019 •
edited
Loading

There are several issues that should be fixed for such combination of layout properties:

  • Collision box for icon has wrong size (+ fix for collision debug program is needed)
  • Collision hit testing for icon should be fixed / improved
  • Collision feature should be shifted together with text box
@chloekraw chloekraw added Core The cross-platform C++ core, aka mbgl text rendering labels Aug 13, 2019
@chloekraw chloekraw added this to the release-queso milestone Aug 13, 2019
@chloekraw chloekraw changed the title Combination of text-variable-anchor and icon-text-fit is broken [core] Combination of text-variable-anchor and icon-text-fit is broken Aug 13, 2019
@chloekraw chloekraw mentioned this issue Aug 13, 2019
Open
@alexshalamov alexshalamov mentioned this issue Aug 13, 2019
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@alexshalamov alexshalamov

Labels
Core The cross-platform C++ core, aka mbgl text rendering
Projects
None yet
Milestone
release-queso
2 participants
@alexshalamov @chloekraw