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 table html javascript placeholders and Column url_name for href link #2

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

Conversation

mbanaouas
Copy link
Contributor

added to Column: url_name
added to table.html: before_js and after_js

before_js and after_js are javascript placeholders that allow derived tables to add useful features to manage and format the datatable, like row highlight and so, thus avoiding placing them repeatedly in each template.
added to Column:
url_name is a string that identifies a name of a url that will be accessible from the link href cell column
two formats:
1/ url_name=some_link
or
2/ url_name=somelink:field_name
field_name (default is id) is the queryset field name whose value is used to format the href link
@cchurch
Copy link
Member

cchurch commented Feb 24, 2014

Thanks for the pull request. I haven't had much time to work on this project lately.

I'm leaning toward your url_name parameter being part of a separate Column subclass (e.g. LinkColumn), but otherwise I like the idea.

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