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

[WIP]give the ability to load custom renderers #36

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

toddjordan
Copy link
Owner

The goal is to be able to swap the alpaca renderer with other kinds of renderers, namely renderers that render actual ember components to the form.

We'll use ember config to specify which renderer to use. We'll also create a base ember component renderer that will render components.

For a sample usage, we'll pick a component library such as ember-paper to render the form.

@acorncom
Copy link
Contributor

We'd discussed (on Ember Slack) that there is another add-on (ember-frost-bunsen) that does something similar to this. However, it seems to be corporate-backed with most decisions made in-house (instead of discussed on Github). Seems like it would be hard to work on a common add-on. However, it's worth looking at for some ideas 😄

Two items in particular might be of interest:

@toddjordan
Copy link
Owner Author

Thanks @acorncom, I'll take a look.

On Wed, May 18, 2016 at 5:24 PM David Baker [email protected]
wrote:

We'd discussed (on Ember Slack) that there is another add-on (
ember-frost-bunsen) that does something similar to this. However, it
seems to be corporate-backed with most decisions made in-house (instead of
discussed on Github). Seems like it would be hard to work on a common
add-on. However, it's worth looking at for some ideas 😄

Two items in particular might be of interest:

https://github.com/ciena-frost/ember-frost-bunsen/blob/master/app/templates/components/frost-bunsen-input-wrapper.hbs

https://github.com/ciena-frost/ember-frost-bunsen/blob/master/addon/components/input-wrapper.js


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#36 (comment)

@k3n
Copy link

k3n commented Aug 5, 2016

Hi @toddjordan, co-worker of @jschilli here, I plan to take a look at this pretty soon. Have you made any further updates? Or do you have any additional thoughts? Thanks in advance!

@toddjordan
Copy link
Owner Author

@k3n Hi, awesome! I have been doing some work on this lately and will update the branch this week (probably after some friday/saturday open source time) I'll document my current thinking/approach on this in the issue #23

@k3n
Copy link

k3n commented Sep 1, 2016

HI @toddjordan, juggling a couple things at once, but trying to keep tabs on this still. Thanks for the updates!

load alpaca renderer from its own file

move alpaca renderer under forms dir

add htmlbars compile to ember renderer
fix merge part duex
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

Successfully merging this pull request may close these issues.

3 participants