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

Add .items option taking object array #25

Merged
merged 1 commit into from Jul 3, 2013

Conversation

Projects
None yet
4 participants
@bglick
Collaborator

bglick commented Jun 18, 2013

Works the same as ajax but without the server call.

This let's users build the setup on the server side and inject it directly into the bootstro.start call like:

bootstro.start('',{items:
   [{selector:'abc',content:'Hello World'},
    {selector:'def',content:'Goodbye World'}
   ]
});

clu3 added a commit that referenced this pull request Jul 3, 2013

Merge pull request #25 from bglick/object_items
Add .items option taking object array .Pulled from @bglick

@clu3 clu3 merged commit 08056d6 into clu3:master Jul 3, 2013

@clu3

This comment has been minimized.

Show comment
Hide comment
@clu3

clu3 Jul 3, 2013

Owner

Thanks alot @bglick

Owner

clu3 commented Jul 3, 2013

Thanks alot @bglick

@bglick bglick deleted the bglick:object_items branch Jul 3, 2013

@vfonic

This comment has been minimized.

Show comment
Hide comment
@vfonic

vfonic Jul 12, 2013

This should be added to documentation. I just found this here on GitHub, but couldn't find anything similar on project page.

I prefer this way than having HTML filled with bootstro classes and data-* attributes. This makes it unified and clean and I can easily remove/change it.

vfonic commented on 7691792 Jul 12, 2013

This should be added to documentation. I just found this here on GitHub, but couldn't find anything similar on project page.

I prefer this way than having HTML filled with bootstro classes and data-* attributes. This makes it unified and clean and I can easily remove/change it.

@clu3

This comment has been minimized.

Show comment
Hide comment
@clu3

clu3 Jul 15, 2013

Owner

@vfonic thanks i updated it

Owner

clu3 commented Jul 15, 2013

@vfonic thanks i updated it

@jgentes

This comment has been minimized.

Show comment
Hide comment
@jgentes

jgentes Dec 4, 2014

Damn, as it turns out, this update is in bootstro.js but not in bootstro.min.js (as of the latest build Dec 4, 2014).. arrgh.

jgentes commented Dec 4, 2014

Damn, as it turns out, this update is in bootstro.js but not in bootstro.min.js (as of the latest build Dec 4, 2014).. arrgh.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment