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

Bug - ToolbarContent - vertical alignment does not work #11427

Open
KevinFCormier opened this issue Jan 14, 2025 · 0 comments
Open

Bug - ToolbarContent - vertical alignment does not work #11427

KevinFCormier opened this issue Jan 14, 2025 · 0 comments

Comments

@KevinFCormier
Copy link
Contributor

Describe the problem
When using the alignItems="center" prop on a ToolbarContent component, a text toolbar item that is not as tall as other items is vertically at the top rather than being centred.

How do you reproduce the problem?
PF4: vertical centering was the default
https://codesandbox.io/p/sandbox/p844wj

PF6: vertical centering is no longer the default, but using alignItems="center" on the ToolbarContent component achieves the desired result
https://codesandbox.io/p/sandbox/t5hgyh

PF5: alignment does not work with the same changes as for PF6
https://codesandbox.io/p/sandbox/jr7hq6

Expected behavior
There should be a way to vertically center when using PF5.

Is this issue blocking you?
No

What is your environment?

  • OS: macOS
  • Browser: Chrome
  • Version: 131.0.6778.265
  • PatternFly: 5.4.8

What is your product and what release date are you targeting?
ACM 2.13 - Feature Complete Feb 12, 2025

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Needs triage
Development

No branches or pull requests

1 participant