Browserify support about cssstyle HOT 8 CLOSED

Forked and pushed my workaround for this issue: TreehouseJS/CSSStyleDeclaration@9fa65f9b79

from cssstyle.

ugh, yeah ugly, but if that's what's need, that's what's needed. I actually don't push versions from the master branch anymore, but from the 0.2.x or 0.3.x branches. I'll think a little bit about organizing this better. Maybe have a require for CSS2.1 and one for CSS3, and then break them down even more in there, for related properties.

from cssstyle.

Ok, sounds good. Let me know what you decide and I'll integrate my changes if necessary.

from cssstyle.

Would be pretty cool to somehow make this work so that we don't have to bundle the separate fork with jsdom for browserify users.

One way to do it is to create a prepublish script that generates those lines ahead of time. You can then make the pretest script be npm run prepublish and such. We did that for the Promises/A+ tests in promises-aplus/promises-tests@e099884

from cssstyle.

my mind is slow today, I'll get this fixed and still have lazy load later today

from cssstyle.

Hmm, if it can be done in an easily-maintainable way, maybe use the browser field in package.json so that browser consumers get one file and Node another?

From: chad3814mailto:[email protected]
Sent: ý2014-ý10-ý13 13:44
To: chad3814/CSSStyleDeclarationmailto:[email protected]
Cc: Domenic Denicolamailto:[email protected]
Subject: Re: [CSSStyleDeclaration] Browserify support (#11)

actually, the problem is now it lazy loads the files because of #14https://github.com/chad3814/CSSStyleDeclaration/pull/14 I'm not sure what I can do make both happy...

—
Reply to this email directly or view it on GitHubhttps://github.com/chad3814/CSSStyleDeclaration/issues/11#issuecomment-58928480.

from cssstyle.

% npm publish

[email protected] prepublish .
node ./scripts/generate_properties.js

npm http PUT https://registry.npmjs.org/cssstyle
npm http 201 https://registry.npmjs.org/cssstyle

• [email protected]

from cssstyle.

% npm publish

[email protected] prepublish .
node ./scripts/generate_properties.js

npm http PUT https://registry.npmjs.org/cssstyle
npm http 201 https://registry.npmjs.org/cssstyle

• [email protected]

from cssstyle.