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

Provide dist file #94

Closed
wants to merge 37 commits into from
Closed

Conversation

cristijora
Copy link
Contributor

@cristijora cristijora commented Mar 27, 2017

This PR targets issue #71 and basically provides the ability to use VueTable with a js script only and without npm.
Update: This PR also targets issue #10
The usage would be pretty simple:

<script src="path to bundle"></script>
Vue.use(VueTable);

The .use from above will register all the components globally

function install(Vue){
  Vue.component("vuetable", Vuetable);
  Vue.component("vuetable-pagination", VueTablePaginationInfo);
  Vue.component("vuetable-pagination-dropdown", VueTablePaginationDropDown);
  Vue.component("vuetable-pagination-info", VueTablePaginationInfo);
}

Also you have the ability to access certain components if you need them:

VueTable: VueTable.default/VueTable.VueTable,
VueTablePagination: VueTable.VueTablePagination,
VueTablePaginationInfo: VueTable.VueTablePaginationInfo,
VueTablePaginationDropdown: VueTable.VueTablePaginationDropdown

ratiw and others added 25 commits January 19, 2017 19:35
Refresh table on apiUrl change
fixed:document.querySelectorAll return the typeof nodeList not array
* row-class-callbacks:
  allow functions as rowClassCallbacks too
@ratiw
Copy link
Owner

ratiw commented Mar 27, 2017

@cristijora Wow, really thank you for that! Now, I don't have to set aside time to research on how to do that. :) 👍

@cristijora
Copy link
Contributor Author

Glad I could help. Also noticed that you require Vue and VueResource internally and even found an issue for that. I think those can be removed and just throw an error or warning if this.$http is not defined. I think the http library has to be defined outside of vuetable

@ratiw
Copy link
Owner

ratiw commented Mar 27, 2017

That actually is a good idea. Let me think about on how it should proceed.

@cristijora cristijora mentioned this pull request Mar 31, 2017
@ratiw ratiw closed this May 30, 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.

6 participants