Skip to content

EDM-3394: Fix footer button - overwrite styling from OCP console#549

Open
celdrake wants to merge 1 commit intoflightctl:mainfrom
celdrake:EDM-3394-overwrite-ocp-console-styling
Open

EDM-3394: Fix footer button - overwrite styling from OCP console#549
celdrake wants to merge 1 commit intoflightctl:mainfrom
celdrake:EDM-3394-overwrite-ocp-console-styling

Conversation

@celdrake
Copy link
Collaborator

@celdrake celdrake commented Feb 26, 2026

The OpenShift console has the following CSS

.co-quick-start-drawer {
  .pf-v6-c-action-list {
    justify-content: space-between;
  }
}

Since the QuickStartDrawer sits on the top of the DOM, it affects the FlightCtlWizardFooter, because it's an ActionList with more than 1 ActionListGroup.

Until there's a fix in Console, we must overwrite the style to keep our desired display.

action-group-styling

Summary by CodeRabbit

Bug Fixes

  • Fixed layout alignment issues in the wizard footer that previously caused inconsistent spacing between action buttons. The buttons now display with proper alignment and spacing, providing a cleaner, more polished appearance throughout the form completion wizard. This improvement enhances visual consistency and delivers a more professional presentation of the user interface.

@coderabbitai
Copy link

coderabbitai bot commented Feb 26, 2026

Walkthrough

Adds an inline style to the ActionList component within FlightCtlWizardFooter to override the justify-content CSS property from space-between to normal, addressing a layout presentation issue. No functional or control flow changes.

Changes

Cohort / File(s) Summary
Styling Override
libs/ui-components/src/components/common/FlightCtlWizardFooter.tsx
Added inline justifyContent style override to ActionList container to fix layout spacing behavior.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check ✅ Passed The title clearly and specifically references the main change: fixing footer button styling by overwriting OCP console CSS. It directly relates to the changeset's primary objective.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@celdrake
Copy link
Collaborator Author

Created patternfly/patternfly#8182 although I'm not sure if this is a Patternfly issue, or an issue with the OpenShift console instead.

@celdrake celdrake changed the title EDM-3394: Overwrite styling from OCP console EDM-3394: Fix footer button - overwrite styling from OCP console Mar 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant