Skip to content

Fixes #39523 - Fix React tables toolbar alignment#11101

Merged
ofedoren merged 1 commit into
theforeman:developfrom
MariaAga:fix-left-align-table
Jul 15, 2026
Merged

Fixes #39523 - Fix React tables toolbar alignment#11101
ofedoren merged 1 commit into
theforeman:developfrom
MariaAga:fix-left-align-table

Conversation

@MariaAga

Copy link
Copy Markdown
Member

Adjusting the class names and ids in table index page, to match the page layout component.

after
image
before
image

@adamruzicka adamruzicka left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can't speak much to the changes themselves, but it resolves the issue in rex and webhooks

@ianballou

Copy link
Copy Markdown
Contributor

I've seen the host index page seems to be affected along with Katello pages that use TableIndex.

@sjha4 sjha4 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixes several Katello pages..Alignment looks good with the changes. 🎉

Content credentials table toolbar:
Screenshot from 2026-07-14 12-12-55
Hosts page toolbar:
Hosts

@Lukshio Lukshio left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Adding the page-layout-toolbar id fixed the issue. It applies css introduced in #11036
and specified selectors in #11044

@ofedoren ofedoren left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, @MariaAga ! Multiple people approved, so I feel we can get this in :)

@ofedoren
ofedoren merged commit 6205d62 into theforeman:develop Jul 15, 2026
40 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants