Skip to content

fix(pivot-grid): prevent bootstrap searchbox from overlapping selector list - #17627

Open
simeonoff wants to merge 1 commit into
22.1.xfrom
simeonoff/fix-17586-22.1.x
Open

simeonoff wants to merge 1 commit into
22.1.xfrom
simeonoff/fix-17586-22.1.x

Conversation

@simeonoff

Copy link
Copy Markdown
Member

Fixes #17586

Description

igx-input-group inside the pivot selector's .igx-pivot-data-selector__filter had no flex-shrink: 0. With the Bootstrap-only padding: 4px override, the flex column layout shrank it back to its unpadded min-height, causing the search box's bordered bundle to overflow into the dimensions list below.

Added flex-shrink: 0 to that rule in themes/shared/_bootstrap.scss so the search box keeps its full padded height.

Motivation / Context

Fixes #17586 - [Bootstrap] Pivot grid selector searchbox does not look good with bootstrap theme.

Type of Change (check all that apply):

  • Bug fix

Component(s) / Area(s) Affected:

Pivot Grid (selector search box, Bootstrap theme)

How Has This Been Tested?

  • Manual testing

Test Configuration:

  • Browser(s): Chrome

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

The reviewed styling change addresses the overlap with no unresolved issues.

Pull request overview

Fixes Bootstrap pivot-grid selector searchbox overflow by preventing its padded input group from shrinking.

Changes:

  • Adds flex-shrink: 0 to the Bootstrap pivot selector filter input group.
File summaries
File Description
projects/igniteui-angular/grids/pivot-grid/src/themes/shared/_bootstrap.scss Prevents searchbox height shrinkage and overlap.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

@mddragnev mddragnev assigned igdmdimitrov and unassigned mddragnev Sep 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants