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

Copy to clipboard message is hardcoded #28

Closed
MarcosBL opened this issue Jun 18, 2013 · 4 comments
Closed

Copy to clipboard message is hardcoded #28

MarcosBL opened this issue Jun 18, 2013 · 4 comments

Comments

@MarcosBL
Copy link

"Table copied

Copied [ X ] rows to the clipboard."

That message is hardcoded in the .js file, could be useful to have it configurable as the Print message is in the "sInfo" var:

"sInfo": "<h6>Table copied</h6><p>Copied * rows</p>"

...or something like that.

@DataTables
Copy link
Collaborator

Its hardcoded only in the sense that it is the default value for the fnComplete function for the copy button. You can easily just use a custom fnComplete function to allow configuration of the text.

@MarcosBL
Copy link
Author

Sure, you are right, is just i thought it could be handy to be able to pass a custom text, like in the "Print" version, seems just a couple of lines more.

Anyway, feel free to just close it if it doesn't fit, i'll not feel offended ;D

@DataTables
Copy link
Collaborator

Yes it probably would be handy to have it in an i18n option.Problem is that it is only that button which uses it - perhaps it should be another button config option like the Ajax url is for the ajax button.

I need to have a bit of a think about how to change TableTools for the new DataTables API and TableTools 3.0, so I'll leave this open for now so it can be folded into that.

Thanks for bring it up.

@alesnav
Copy link
Contributor

alesnav commented Dec 16, 2013

Sorry, I don't know how to add a pull request in this issue and another one is opened to solve this issue (or at least, to try it) :(

#38

I think that maybe this is not the best way to solve it, but it works.

Thanks!

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

No branches or pull requests

2 participants