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

Bootstrap support #40

Open
stephanstapel opened this issue Jan 29, 2015 · 3 comments
Open

Bootstrap support #40

stephanstapel opened this issue Jan 29, 2015 · 3 comments

Comments

@stephanstapel
Copy link

Hi,

did you consider theming support, aka optionally supporting Bootstrap? If you have some advice for me, I'd volunteer implementing it.

@danhunsaker
Copy link
Owner

The plan is to support HTML templates instead of hard coded elements. It may be a little tricky to pull off properly, and the default templates should produce identical HTML to the current implementation, but otherwise I don't expect much in the way of problems. The tricky part is how to define such templates, and identify which to use. My initial thoughts are to define another Directive to hold the template itself, then either put it inside the main dynForm elements (which can get tricky with nesting), or reference it from an attribute on the dynForm element instead. The latter approach seems more flexible and stable, so we just need a name for the attribute. And one for the template element, too.

@stephanstapel
Copy link
Author

isn't it only the element's class which you'd influence? That could work much easier.

@danhunsaker
Copy link
Owner

If it's just classes you're looking for, see the README for the attributes option.

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

No branches or pull requests

2 participants