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

conditional display for columns #37

Closed
guruofgentoo opened this issue Apr 24, 2018 · 0 comments · Fixed by #76
Closed

conditional display for columns #37

guruofgentoo opened this issue Apr 24, 2018 · 0 comments · Fixed by #76
Assignees

Comments

@guruofgentoo
Copy link
Member

Columns should be made to conditionally displayed through permissions, app settings, etc. We could do a ConditionalColumn, but I'd rather be able to simply pass a lambda to a Column of any type to determine whether it would display.

Should also look at render_in and make sure it can handle lambdas.

@bchopson bchopson self-assigned this Oct 29, 2019
bchopson added a commit that referenced this issue Oct 29, 2019
- Accepts lambda or boolean
- Hides column if False

Note: subclasses of Column which override __init__ will need to update
their implementations to account for the new argument

Refs #37
bchopson added a commit that referenced this issue Oct 30, 2019
- Accepts lambda or boolean
- Hides column if False
- render_in now accepts lambda

Note: subclasses of Column which override __init__ will need to update
their implementations to account for the new argument

Refs #37
bchopson added a commit that referenced this issue Oct 30, 2019
- Accepts lambda or boolean
- Hides column if False
- render_in now accepts lambda

Note: subclasses of Column which override __init__ will need to update
their implementations to account for the new argument

Refs #37
bchopson added a commit that referenced this issue Nov 1, 2019
- Renderer base class provides caching and simplified initialization

Refs #37
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 a pull request may close this issue.

2 participants