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.

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

Make vectorize-text functions top-level #3380

Closed
wants to merge 3 commits into from

Conversation

archmoj
Copy link
Contributor

@archmoj archmoj commented Dec 31, 2018

Fixes the error reported in #3379.
@plotly/plotly_js

@archmoj archmoj added the regression this used to work label Dec 31, 2018
@etpinard
Copy link
Contributor

etpinard commented Jan 2, 2019

Thanks @archmoj !

Were you able to reproduce #3379 on your machine?

If not, let's wait and see if #3379 (comment) gets a reply.

@etpinard
Copy link
Contributor

etpinard commented Jan 4, 2019

Were you able to reproduce #3379 on your machine?

Ping @archmoj !

@archmoj
Copy link
Contributor Author

archmoj commented Jan 4, 2019

No I was not able to do so. Any hint?

@@ -4958,7 +4954,7 @@
"glslify": "^6.1.0",
"is-string-blank": "^1.0.1",
"typedarray-pool": "^1.0.2",
"vectorize-text": "^3.2.0"
"vectorize-text": "git://github.com/archmoj/vectorize-text.git#805784cb822eb29096555be6d047b45370650e9e"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@fuhry
Copy link

fuhry commented Jan 11, 2019

For those looking for an easy way to reproduce this, we @ZorusTech have found it reproduces under Qt Webkit including wkhtmltopdf.

@archmoj
Copy link
Contributor Author

archmoj commented Jan 24, 2019

@etpinard OK.
vectorize-text version 3.2.1 is out. Let me redo the patches using that...

@archmoj
Copy link
Contributor Author

archmoj commented Jan 24, 2019

Closed. Now in #3474

@archmoj archmoj closed this Jan 24, 2019
@archmoj archmoj deleted the 3379-vectorize-text-functions branch January 24, 2019 15:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
regression this used to work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants