Skip to content

Fix Searchable select text overlap#1525

Merged
rSnapkoOpenOps merged 2 commits intomainfrom
feat/select-text-overlap
Oct 28, 2025
Merged

Fix Searchable select text overlap#1525
rSnapkoOpenOps merged 2 commits intomainfrom
feat/select-text-overlap

Conversation

@rSnapkoOpenOps
Copy link
Collaborator

@rSnapkoOpenOps rSnapkoOpenOps commented Oct 23, 2025

Fixes OPS-2905.

image

@linear
Copy link

linear bot commented Oct 23, 2025

loading={loading}
aria-expanded={open}
className="w-full justify-between w-full"
className="w-full justify-between"
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Just removed duplicate

@rSnapkoOpenOps rSnapkoOpenOps marked this pull request as ready for review October 23, 2025 15:20
Copilot AI review requested due to automatic review settings October 23, 2025 15:20
Copy link
Contributor

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 text overlap issue in the SearchableSelect component where the selected text was overlapping with the refresh button. The fix involves:

  • Removing duplicate w-full class from the trigger button
  • Adding proper spacing and text truncation controls to the label span

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@sonarqubecloud
Copy link

@rSnapkoOpenOps rSnapkoOpenOps merged commit 8f230d9 into main Oct 28, 2025
21 checks passed
@rSnapkoOpenOps rSnapkoOpenOps deleted the feat/select-text-overlap branch October 28, 2025 14:55
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.

3 participants