-
Notifications
You must be signed in to change notification settings - Fork 62
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
IBX-963: Buttons on Trash page are hidden on medium resolutions #1888
Conversation
<tr class="ibexa-table__head-row"> | ||
{% if form is defined %} | ||
<div class="ibexa-scrollable-wrapper"> | ||
<table class="ibexa-table table ibexa-table__draft-conflict {{ (is_draft and have_to_paginate ? 'mb-3' : '') }}"> |
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.
you don't have an ibexa-table
parent so i think it should either be ibexa-draft-conflict
or ibexa-table--draft-conflict
{% endif %} | ||
</tr> | ||
</thead> | ||
<tbody |
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.
maybe one line?:)
{% endfor %} | ||
</tbody> | ||
</table> | ||
</div> |
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.
new line
class="ibexa-table__body | ||
{{ table_body_class|default('') -}}" | ||
{{ html.attributes(table_body_attr|default({})) }} | ||
<div class="ibexa-scrollable-wrapper"> |
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.
Do we want to make it an option?
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.
I don't think so, why not just put it in every table, if it's not needed, there would just be scroll @dew326
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.
I think you didn't understand me. I was thinking about adding an option for 3rd party to disable with default value true
.
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
https://github.com/ezsystems/ezplatform-page-builder/pull/827
Checklist:
$ composer fix-cs
)