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

Browser compatibility #14

Open
nielsbril opened this issue May 22, 2018 · 5 comments
Open

Browser compatibility #14

nielsbril opened this issue May 22, 2018 · 5 comments
Labels
feature request Request for a new feature

Comments

@nielsbril
Copy link

Is your feature request related to a problem? Please describe.
We would like to use your package as an abstraction for console.log in both frontend (aka the browser) and backend (Node.js).

Describe the solution you'd like
Are you planning on making the package work in the browser console aswell? Or how would you achieve it?

Thanks in advance!

@klaudiosinani klaudiosinani added the feature request Request for a new feature label May 24, 2018
@rjoydip-zz
Copy link
Contributor

@nielsbril Signale is built on Chalk and chalk doesn't support that. Here is the comment of @sindresorhus said on the browser compatibility. comment 1 and comment 2

@nielsbril
Copy link
Author

I noticed. Also the usage of process.stdout would not work in browser. But maybe @klauscfhq has plans on how to make the module compatible with browser console 🙏

@jim1795
Copy link

jim1795 commented Jun 9, 2018

This is a similar package for both Browser and Node.

@niftylettuce
Copy link

hey everyone here, please subscribe and view #47

screen shot 2018-06-21 at 12 18 44 am

😄

@astr0junk
Copy link

astr0junk commented Dec 19, 2019

@nielsbril
This is my ugly solution:

https://codesandbox.io/s/signale-browser-consolelog-adapter-tk9v0

need aliases in webpack resolve (or other build tool) for:

'path': path.resolve(__dirname, "../node_modules/path-browserify"),
'readline': path.resolve(__dirname, "../node_modules/readline-browserify"),
'fs': path.resolve(__dirname, "../node_modules/browserify-fs"),

May be i will make this ability like an option for Signale origin package

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request Request for a new feature
Projects
None yet
Development

No branches or pull requests

6 participants