Skip to content

Fixes #39275 - prevent global override of radio position#10967

Merged
jeremylenz merged 1 commit into
theforeman:developfrom
MariaAga:radio-position-css
Apr 29, 2026
Merged

Fixes #39275 - prevent global override of radio position#10967
jeremylenz merged 1 commit into
theforeman:developfrom
MariaAga:radio-position-css

Conversation

@MariaAga
Copy link
Copy Markdown
Member

#10591
Added a global rule to all PF5 radio inputs. This makes all radio inputs position change, and sometimes be in a completely different place than they should.
custom CSS rules should be specific to the component.

Use specific css selectors to avoid conflicts with other plugins or the core.

https://theforeman.org/handbook.html

@ekohl
Copy link
Copy Markdown
Member

ekohl commented Apr 29, 2026

Not looking at the actual commit, but on the process:

Refs #38416 - prevent global override of radio postion

You can't use Refs on a closed issue because it'll never show up in the release notes. The bot doesn't check for it, but you should open a new bug.

Also, a typo in position there.

@MariaAga MariaAga force-pushed the radio-position-css branch from e82423c to 86fbf8b Compare April 29, 2026 12:31
@MariaAga
Copy link
Copy Markdown
Member Author

@ekohl Thanks, fixed!

Copy link
Copy Markdown
Contributor

@jeremylenz jeremylenz left a comment

Choose a reason for hiding this comment

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

PR title still needs an update. Should probably be "Fixes" since it's a new bug now, but not blocking.

thanks for finding and fixing 🙂

@MariaAga MariaAga force-pushed the radio-position-css branch from 86fbf8b to 712a91b Compare April 29, 2026 12:57
@MariaAga MariaAga changed the title Refs #38416 - prevent global override of radio postion Fixes #39275 - prevent global override of radio position Apr 29, 2026
@MariaAga
Copy link
Copy Markdown
Member Author

Updated the commit and title, thanks 🙏

@lfu
Copy link
Copy Markdown
Contributor

lfu commented Apr 29, 2026

@MariaAga Thanks!!

@jeremylenz jeremylenz merged commit 56d1031 into theforeman:develop Apr 29, 2026
26 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.

4 participants