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

list view: getCssClasses for the whole row #794

Closed
fedotxxl opened this issue Nov 16, 2015 · 10 comments
Closed

list view: getCssClasses for the whole row #794

fedotxxl opened this issue Nov 16, 2015 · 10 comments

Comments

@fedotxxl
Copy link
Contributor

Hello
Our list view also contains deleted items. In this case they should be marked somehow (change background color for the row) + actions / selection checkbox should be hidden.
I thought to apply special CSS to these rows but it looks like ng-admin doesn't support dynamic classes per row.

Can you add it?

@fzaninotto
Copy link
Member

Fixed by #772

@fzaninotto
Copy link
Member

Oops, no, sorry for the fast close. Reopening.

@fzaninotto
Copy link
Member

@fedotxxl great, I suggest you make a PR out of it

@fedotxxl
Copy link
Contributor Author

Created marmelab/admin-config#52 / #795 but I'm not sure that suggested method names / implementation is the best way to solve the problem

@fzaninotto
Copy link
Member

Well, PRs are a better way to discuss that than an external repo!

@fedotxxl
Copy link
Contributor Author

thank you! it works!

@lucval
Copy link

lucval commented Nov 20, 2015

It would be nice to have this feature available for referenced_list as well.

@fzaninotto
Copy link
Member

@lucval yes it would be nice. I think it's not very hard, would you like to give it a try?

@lucval
Copy link

lucval commented Dec 10, 2015

@fzaninotto got a bit of spare time and done it (#844)

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

No branches or pull requests

3 participants