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

HarfBuzz #3

Closed
kkaefer opened this issue Mar 21, 2013 · 4 comments
Closed

HarfBuzz #3

kkaefer opened this issue Mar 21, 2013 · 4 comments

Comments

@kkaefer
Copy link
Member

kkaefer commented Mar 21, 2013

@springmeyer, you were talking about replacing ICU/freetype with HarfBuzz. Could you please write up your thoughts about this or point me to where the reasons are explained?
@kkaefer
Copy link
Member Author

kkaefer commented Mar 21, 2013

/cc @artemp

@artemp
Copy link

artemp commented Mar 21, 2013

@kkaefer - I think the plan was to use Harfbuzz with ICU/freetype
@springmeyer ?

@springmeyer
Copy link
Contributor

right, we are contemplating using harfbuzz for shaping instead of only ICU. So, we would start depending on harfbuzz in front of ICU (harfbuzz optional depends on ICU and freetype itself). I don't know all the details, but Herm's branch has the code: mapnik/mapnik@master...harfbuzz

@springmeyer
Copy link
Contributor

because that branch is not merged, I would ignore this complexity for the time being

@kkaefer kkaefer closed this as completed Sep 9, 2013
@Que3216 Que3216 mentioned this issue May 27, 2016
Closed
lucaswoj added a commit that referenced this issue Dec 13, 2016
@lucaswoj
Merge pull request #3 from mapbox/fill-color-property-function
5d064aa 
Use pragmas for fill-color and fill-outline-color
lucaswoj pushed a commit that referenced this issue Jan 11, 2017
@tmcw
Validate buckets. Refs #3
093ad15
lucaswoj pushed a commit that referenced this issue Jan 11, 2017
@tmcw
Add repository field. Fixes #3
7fec17a
ai-santos added a commit to ai-santos/mapbox-gl-js that referenced this issue Apr 13, 2017
@ai-santos
# This is a combination of 3 commits.
4f3b0ea 
# This is the 1st commit message:
Stylelint works for application

# This is the commit message mapbox#2:

WIP modify lint-css script in package.json

# This is the commit message mapbox#3:

Ammend warning for lint-css npm script
mollymerp pushed a commit that referenced this issue Apr 25, 2017
@ai-santos @mollymerp
Add stylelinting to ensure mapboxgl- prefix on all classes (#4584)
5ceec40 
* Add stylelint CLI and Add mapbox class name rule in stylelintrc

* # This is a combination of 3 commits.
# This is the 1st commit message:
Stylelint works for application

# This is the commit message #2:

WIP modify lint-css script in package.json

# This is the commit message #3:

Ammend warning for lint-css npm script

* Stylelint implemented for application

WIP modify lint-css script in package.json

Ammend warning for lint-css npm script

Add csscolorparser

* Add lintx to script file

* Took out lintx

* Took out lintx

* Resolve comments to 4584 PR

* Resolve PR comments

* Ignore stylelint in .flowconfig and remove true from scripts
jessisena pushed a commit to jessisena/mapbox-gl-js that referenced this issue Sep 26, 2017
@bolollo
Merge pull request mapbox#3 from bolollo/master
b1dab00 
add property-function to reference
@yapalexei yapalexei mentioned this issue Jan 10, 2020
Closed
astojilj added a commit that referenced this issue Mar 23, 2020
@astojilj
Merge pull request #3 from mapbox/rebase
e552a32 
Rebase to latest mapbox-gl-js. Rubber-stamped.
@openSourcerer9000 openSourcerer9000 mentioned this issue Jan 20, 2022
Closed
@wipfli wipfli mentioned this issue Feb 4, 2023
Closed
@aravindb26 aravindb26 mentioned this issue Jan 1, 2024
Open
@JuanIrache JuanIrache mentioned this issue Jan 19, 2024
Open
@yazidDev88 yazidDev88 mentioned this issue May 2, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@springmeyer @kkaefer @artemp