Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@lukasmasuch
Copy link
Collaborator

@lukasmasuch lukasmasuch commented Feb 27, 2025

Describe your changes

Makes the dataframe search bar more responsive to the available width and to the root font size.

image

GitHub Issue Link (if applicable)

Testing Plan

  • Added e2e test & update snapshots.

Contribution License Agreement

By submitting this pull request you agree that all contributions to this project are made under the Apache 2.0 license.

@lukasmasuch lukasmasuch added security-assessment-completed Security assessment has been completed for PR change:bugfix PR contains bug fix implementation impact:users PR changes affect end users labels Feb 27, 2025
@sfc-gh-lmasuch sfc-gh-lmasuch changed the title Make the dataframe search field more responsive Make the dataframe search bar more responsive Feb 27, 2025
@lukasmasuch lukasmasuch marked this pull request as ready for review February 27, 2025 11:48
@lukasmasuch lukasmasuch added change:feature PR contains new feature or enhancement implementation and removed change:bugfix PR contains bug fix implementation labels Feb 27, 2025
@lukasmasuch lukasmasuch changed the title Make the dataframe search bar more responsive Revamp the dataframe search bar to be more responsive Feb 27, 2025
"& .gdg-seveqep": {
// Make the search field more responsive to the grid width and use
// rem units for everything.
maxWidth: "19rem",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Could you please leave a comment about how we landed at 19rem? Even if it is just We picked 19rem arbitrarily because it looked good on a laptop screen or whatever. This will help others in the future in case this has to be revisited

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Sounds good, added some comments 👍

// rem units for everything.
maxWidth: "19rem",
width: "80%",
minWidth: "6rem",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Added comments 👍

Copy link
Collaborator

@raethlein raethlein left a comment

Choose a reason for hiding this comment

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

LGTM! 🔍

@lukasmasuch lukasmasuch enabled auto-merge (squash) February 27, 2025 14:56
@lukasmasuch lukasmasuch merged commit f146893 into develop Feb 27, 2025
32 checks passed
@lukasmasuch lukasmasuch deleted the fix/make-dataframe-search-more-responsive branch February 27, 2025 15:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

change:feature PR contains new feature or enhancement implementation impact:users PR changes affect end users security-assessment-completed Security assessment has been completed for PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Make the search box in st.dataframe responsive

3 participants