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: Dynamic Widget #498

Closed
wants to merge 12 commits into from
Closed

WIP: Dynamic Widget #498

wants to merge 12 commits into from

Conversation

domtalbot
Copy link

So, here is my first attempt at a contribution to NetlifyCMS. In simple terms, it's aim is to allow you to create truly dynamic layouts based on prebuilt blocks.

My goal with Dynamic Widget was to allow NetlifyCMS to be used in a way to create component based pages, posts or anything you wish.

Let me take a Blog Post Collection as an example. I want to start with a content block, then break the page with a blockquote, followed by an image gallery. Yes I could do this with a markdown editor, but it isn't simple for a content writer.

Using Dynamic Widgets gives the developer the power to create custom widgets and enable them in the Widget Selector. A content writer then has the power to create truly unique Blog Posts quickly and simply.

ToDo

  • Docs
  • Tests

@domtalbot domtalbot changed the title Dynamic Widget WIP: Dynamic Widget Jul 26, 2017
Copy link
Contributor

@erquhart erquhart left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added a comment on configuration. Also, I'm not seeing previews working - you can check the deploy preview so we're looking at the same thing: https://deploy-preview-498--cms-demo.netlify.com/#/collections/kitchenSink/entries/a-big-entry-with-all-the-things

@@ -4,6 +4,12 @@ backend:

media_folder: "assets/uploads"

dynamic_widgets:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Probably better to keep widget configuration on the widget itself, even if it's redundant. I'd suggest accepting the list of widgets under a widgets key on the dynamic widget config.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like that idea. Then you can choose what widgets are available on a collection by collection basis. Clever.

@domtalbot
Copy link
Author

Deploy preview is looking as I expected:

image

@erquhart
Copy link
Contributor

@domtalbot that's the control panel, which does work great, but the values aren't showing up in the preview pane.

@domtalbot
Copy link
Author

Ahh, of course. It is something I am working on

@erquhart
Copy link
Contributor

erquhart commented Sep 2, 2017

@domtalbot any update on this?

@domtalbot
Copy link
Author

domtalbot commented Sep 13, 2017 via email

@erquhart
Copy link
Contributor

Understood, I'll check back later on :)

@erquhart
Copy link
Contributor

erquhart commented Dec 8, 2017

@domtalbot I'm going to close this for now, but let's discuss the idea further in #565 if you're still interested, I have some thoughts on it.

@erquhart erquhart closed this Dec 8, 2017
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.

2 participants