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

Added possibility of extending and creating new elements #30

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

Conversation

somewebmedia
Copy link

This way users can create their own elements, or extend the current ones. See the example in the readme.

@somewebmedia somewebmedia reopened this Apr 5, 2016
@rafibomb
Copy link
Member

👍 This is definitely where we're going with Inky. We'll be reviewing it for merge or suggestions - thanks!

@kball
Copy link
Contributor

kball commented Apr 14, 2016

@somewebmedia I love the direction this is going, but I'd like to push for this to move further from the details of node and the current inky approach of cheerio and string formatting, and more into something that can be compartmentalized into a package that can be used by different implementations (e.g. we're building a ruby version of inky, similarly I think we need to move away from cheerio and towards something that gives us a tree-based view of the html)

We're planning to open a discussion around this in a bit, but I'm imagining a pluggable component architecture that looks something like

  1. A template (probably handlebars)
  2. An scss file
  3. A json file of structure:
{
  component: componentName,
  template: templateFilePath,
  scss: scssFilePath
}

What do you think? Do you have any examples of components you'd like to build that we can check and make sure this would satisfy their needs?

@somewebmedia
Copy link
Author

@kball

I think changing to a json object spec is a great idea for a major iteration (e.g. major version bump) for the purpose of porting inky to other languages. However, porting this library to other languages seems like a large amount of work that is not specifically related to adding the ability for current users of inky to make their own components. This approach is 100% compatible with the current version so users can drop this in without having to concern themselves with a breaking change.

This will be very helpful to me and likely other current users. The json manifest approach is a big change and it sounds like there are decisions that still need to be made (ex: should you use handlebars or another DSL).

Would you be ok merging this now and taking the refactor as future work?

@rafibomb
Copy link
Member

rafibomb commented Apr 22, 2016

@somewebmedia Just published the Foundation for Emails 2.1 release blog post and the release went out this morning! We also started a discussion on the future of Inky and what amazing integrations can be made. I'm sure you have some comments, questions, or ideas: http://foundation.zurb.com/forum/posts/39717-architecting-the-future-of-foundation-for-emails

@rafibomb
Copy link
Member

@somewebmedia @kball Just checking in on this! Love the idea and progress so far! What is left to be done to move this toward completion?

@alexisgahon
Copy link

Any update on this PR ?

@ahebrank
Copy link

ahebrank commented Sep 8, 2017

This would have been a much more elegant, client-friendly solution that what I ended up doing, which was a Handlebars partial to inject some markup. I agree with @somewebmedia that an incremental, non-breaking improvement now is far better than a big refactor... someday.

@SharpeRAD
Copy link

Any reason why this PR hasn't been accepted? I understand you may want to change the direction of inky in the future but in the meantime this looks like a great way of adding extensibility to the project NOW 👍

@rickysullivan
Copy link

DED

@JimmyMultani
Copy link

Hey @rafibomb, any idea on when this will be merged? Are we waiting on @somewebmedia to update the pull request with something?

Thanks!

@DanielRuf
Copy link
Contributor

There are currently conflicts which have to be resolved.

@somewebmedia
Copy link
Author

I'm not using this project for a long time now. Someone should fork my code and resolve the conflicts if you want this changes to be implemented.

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

Successfully merging this pull request may close these issues.

9 participants