patorjk / figlet.js

A FIG Driver written in JavaScript which aims to fully implement the FIGfont spec.

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Feature Request: Sync function that returns data instead of logging it to console

philkunz opened this issue · comments

Hi,
first of all: awesome node module!

Would it be possible for you to implement a Sync function that simply returns data instead of logging it to console? The Async way is fine, however that requires to work with promises which sometimes is overhead.

Thank you and regards
Phil

What do you mean? There's a textSync function that returns text.

I like the promise idea. The text function probably should return a promise.

I'm so sorry. I somehow was under the impression that the function doesn't return the text, but logs it to console directly... My bad.

No worries, I might update the docs to make it a little more clear.

Hi, love this module, ..
I tried this, but it doesn't seem to be supported in the browser, .. one way could be to preload selected fonts so they are available at run-time?

Create a pull request to fix this issue by providing fonts preload option, #22

why did not use promise? I like promise syntax more than the textSync function. Thanks for this great package @patorjk

@nodejsdeveloperskh Returning a promise is something I plan on adding at some point in (probably the near) future. I liked the idea previously, but I didn't want this library having to rely on a polyfill. Promises are are very well supported now though so there's really no reason not to.