Skip to content

fix(uui-combobox): Remove bottom gap #1164

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

Merged
merged 1 commit into from
Aug 6, 2025

Conversation

leekelleher
Copy link
Member

Description

When inspecting the uui-combobox element, there is some form of whitespace/padding/gap at the bottom of the component.

Screenshot 2025-08-05 132603

Inspecting the element and shadow DOM, I haven't been able to locate the cause of the gap.

I did find that changing the display from inline-block to inline-flex appears to resolve the gap issue.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)

Motivation and context

In Umbraco CMS, there are features that need to flexbox align components next to the combobox, but the additional gapping at the bottom causes them to appear misaligned (by a few pixels).

Screenshot 2025-08-05 134344

How to test?

Using the browser's devtools, inspect the combobox element, notice if the bottom gap has gone.

@leekelleher leekelleher added the bug Something isn't working label Aug 5, 2025
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR fixes a visual issue with the uui-combobox component where an unwanted bottom gap was causing alignment problems when using flexbox layouts. The fix changes the display property from inline-block to inline-flex to eliminate the gap.

  • Changed display property from inline-block to inline-flex in the combobox host styles

Copy link

sonarqubecloud bot commented Aug 5, 2025

Copy link

github-actions bot commented Aug 5, 2025

Azure Static Web Apps: Your stage site is ready! Visit it here: https://delightful-beach-055ecb503-1164.westeurope.azurestaticapps.net

@iOvergaard iOvergaard merged commit b3320e3 into main Aug 6, 2025
12 of 13 checks passed
@iOvergaard iOvergaard deleted the v1/bugfix/combobox-remove-padding branch August 6, 2025 06:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants