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

Outline goals and/or non-goals #1

Open
jfsiii opened this issue Jul 10, 2016 · 3 comments
Open

Outline goals and/or non-goals #1

jfsiii opened this issue Jul 10, 2016 · 3 comments

Comments

@jfsiii
Copy link

jfsiii commented Jul 10, 2016

document.createElement('foo') doesn't exist in a vacuum. Outline what is, and is not, in scope. This would clarify what's meant by minimally viable and save a lot of back-and-forth in issues/PRs.

If a list of criteria/props isn't desired, a paragraph or two on why this was created the purpose it serves would be educational.

e.g. should this work with d3, jQuery, cheerio, etc? Which DOM props/attrs are supported and why? Same for unsupported props/attrs (e.g. style, on*, etc).

@developit
Copy link
Owner

@jfsiii good idea. My initial use-case was to serve the very basic needs of Preact, and I published this because there are other libraries out there that interface with the DOM in a similarly minimal way.

Events are an interesting case, since I think that would be useful for people looking to run tests on top of a minimal DOM implementation.

I think in terms of intended scope, I just want to avoid bundling a parser and even avoid building in a serliazer. Everything else would be fair game as long as it stays super small. I'm looking at dropping the constants since they add little value.

@jfsiii
Copy link
Author

jfsiii commented Jul 10, 2016

Thanks. Size is a great constraint. Especially if it's possible for others to extend/decorate with their desired features.

@jfsiii jfsiii closed this as completed Jul 10, 2016
@jfsiii jfsiii reopened this Jul 10, 2016
@developit
Copy link
Owner

Exactly. That'd be the goal with HTML parsing / serialization - allow them to be added, but don't ship them by default.

developit added a commit that referenced this issue Jul 10, 2016
@jfsiii jfsiii mentioned this issue Jul 17, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants