Skip to content

Conversation

@hoophalab
Copy link
Contributor

@hoophalab hoophalab commented Oct 9, 2025

Description

Long dataset names were being clipped in the dataset selector popup, making them difficult to read. This PR fixes this issue.

long

Checklist

  • The PR satisfies the contribution guidelines.
  • This is a breaking change and that has been indicated in the PR title, OR this isn't a
    breaking change.
  • Necessary docs have been updated, OR no docs need to be updated.

Validation performed

The popup matches dataset name width

Summary by CodeRabbit

  • Style
    • Updated the dataset selector dropdown so its popup no longer automatically matches the field width. The popup now sizes to its content, improving visibility of long dataset names and reducing truncation.
    • Prevents excessive empty space when options are narrow, enhancing layout responsiveness and readability.
    • No changes to selection behaviour or filtering. Affects the dataset control on the Search page.

@hoophalab hoophalab requested a review from a team as a code owner October 9, 2025 16:27
@hoophalab hoophalab requested a review from davemarco October 9, 2025 16:27
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 9, 2025

Walkthrough

A single UI change adds the prop popupMatchSelectWidth={false} to a Select in DatasetSelect.tsx, altering dropdown sizing behaviour. No other logic or control flow was modified.

Changes

Cohort / File(s) Summary
Web UI Select prop update
components/webui/client/src/pages/SearchPage/SearchControls/Dataset/DatasetSelect.tsx
Added prop popupMatchSelectWidth={false} to the Select component to decouple popup width from the select’s width.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Title Check ✅ Passed The title clearly encapsulates the main change by stating that the dataset selector popup will now display full dataset names, directly reflecting the addition of the width-adjusting prop.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Contributor

@davemarco davemarco left a comment

Choose a reason for hiding this comment

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

lgtm

@hoophalab hoophalab requested a review from junhaoliao October 9, 2025 18:49
Copy link
Member

@junhaoliao junhaoliao left a comment

Choose a reason for hiding this comment

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

deferring to @davemarco 's review

@hoophalab hoophalab merged commit 2e5ab12 into y-scope:main Oct 10, 2025
23 of 24 checks passed
@hoophalab hoophalab deleted the long-dataset branch October 23, 2025 19:06
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