-
Notifications
You must be signed in to change notification settings - Fork 51
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
Make sure that the first column after the checkbox in the table view displays row actions #51
Comments
@danielbachhuber @jeremyfelt just discovered this. |
ghost
assigned jeremyfelt
Oct 19, 2012
I'll take a stab at this soon. |
jeremyfelt
added a commit
that referenced
this issue
Mar 6, 2013
…ut may not be added When a provider class does not provide a method to output one of its registered columns, our list table should account for the possible lack of row actions. This commit adds a tracking property for the display of row actions. If a column's output is not handled by the provider class and row actions have not yet been displayed, we handle output of the row actions. Once row actions have been output once, the flag is maintained throughout the row. When the row is complete, the flag is reset for the next. One quirk - if a provider class handles the output for the first column, but does not handle row actions, and does not handle the output for the next column, row actions will be output in that second column. This feels a little weird, but I think gets us close enough. Fixes #51
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
If provider class doesn't have columns_{$custom_column} method, there will be no row actions presented. It would be nice to figure out default logic for the first column to display dem actions.
The text was updated successfully, but these errors were encountered: