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

Toolbar Menu, left icon should be vertically centered #515

Closed
huayunh opened this issue Sep 26, 2022 · 0 comments · Fixed by #594
Closed

Toolbar Menu, left icon should be vertically centered #515

huayunh opened this issue Sep 26, 2022 · 0 comments · Fixed by #594
Labels
brightlayer-ui Used to identify Brightlayer UI platform issues for easy filtering bug Something not working as expected
Milestone

Comments

@huayunh
Copy link
Contributor

huayunh commented Sep 26, 2022

Describe the desired behavior

ToolbarMenu icon should be center-aligned.

Describe the current behavior

It's not centered.

How to recreate?

  1. Navigate to the ToolbarMenu example that showcases an icon, here.
  2. Observe that the icon is not vertically center-aligned.

Is this request related to a current issue?

No

Anything else to add?

Center this icon
image

@huayunh huayunh added enhancement Request for improvement or new feature needs-review Issues that we would like to revisit/review labels Sep 26, 2022
@daileytj daileytj removed the needs-review Issues that we would like to revisit/review label Nov 17, 2022
@joebochill joebochill added the brightlayer-ui Used to identify Brightlayer UI platform issues for easy filtering label Jan 5, 2023
@joebochill joebochill added this to the 8.0.1 milestone Feb 20, 2023
@joebochill joebochill added bug Something not working as expected and removed enhancement Request for improvement or new feature labels Feb 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
brightlayer-ui Used to identify Brightlayer UI platform issues for easy filtering bug Something not working as expected
Development

Successfully merging a pull request may close this issue.

3 participants