-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Fix group, org member tables in mobile view #3435
Fix group, org member tables in mobile view #3435
Conversation
This new CSS will potentially alter the styling of more tables than you intend, since it will apply to the first 3 columns of any table that uses |
@MrkGrgsn, I've added class in templates instead col tag. |
@wildcatzita , i just noticed, that you'll get the same effect if you omit with attribute on col tags, even without adding anything to css. Could you check this and update you PR if i right? |
@smotornyuk, fixed. Can yo check please? |
ckan/templates/group/members.html
Outdated
<th scope="col"></th> | ||
<th class="col-user">{{ _('User') }}</th> | ||
<th class="col-role">{{ _('Role') }}</th> | ||
<th class="col-actions"></th> | ||
</tr> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are you still need these classes?
I think it can be merged now |
No description provided.