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

BaseTableRowItem: Use column-content as CSS class #227

Merged
merged 1 commit into from
Jul 19, 2024

Conversation

nilmerg
Copy link
Member

@nilmerg nilmerg commented Jul 19, 2024

content collides with a container's content and confuses Icinga Web's JS (<2.12.2) which applies impact to it, but doesn't remove it sometimes.

@nilmerg nilmerg added the bug Something isn't working label Jul 19, 2024
@nilmerg nilmerg self-assigned this Jul 19, 2024
@cla-bot cla-bot bot added the cla/signed label Jul 19, 2024
@nilmerg nilmerg added this to the v0.10.1 milestone Jul 19, 2024
`content` collides with a container's content and
confuses Icinga Web's JS (<2.12.2) which applies
`impact` to it, but doesn't remove it sometimes.
@nilmerg nilmerg force-pushed the content-as-class-is-too-ambiguous branch from 9fd3651 to 0fedfec Compare July 19, 2024 09:35
nilmerg added a commit to Icinga/icingadb-web that referenced this pull request Jul 19, 2024
@nilmerg nilmerg changed the title BaseTableRowItem: Use col-content as CSS class BaseTableRowItem: Use column-content as CSS class Jul 19, 2024
@nilmerg nilmerg merged commit 6dd2276 into main Jul 19, 2024
22 checks passed
@nilmerg nilmerg deleted the content-as-class-is-too-ambiguous branch July 19, 2024 12:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working cla/signed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant