Skip to content
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

remove fixed column size for samples nav #1528

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

walesch-yan
Copy link
Collaborator

In the Samples page, the column sizes of the top row including the filter and some buttons are fixed, which causes the different elements to overlap and sometimes hide features on smaller screens.

This PR removes the fixed sizes, you can see the differences below for different screen sizes
Before:
Screenshot from 2025-01-23 15-05-48

Screenshot from 2025-01-23 15-06-10

Screenshot from 2025-01-23 15-07-06

Now:

Screenshot from 2025-01-23 15-02-46

Screenshot from 2025-01-23 15-06-33

Screenshot from 2025-01-23 15-06-49

@@ -845,8 +845,8 @@ class SampleListViewContainer extends React.Component {
) : null}
<Card className="samples-grid-table-card">
<Card.Header className="samples-grid-table-card-header">
<Row className="samples-grid-table-row-header">
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

While at it, I removed this className reference, since the css class, does not seem to exist anymore

Copy link
Collaborator

@axelboc axelboc left a comment

Choose a reason for hiding this comment

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

Nice! Any chance you could add some vertical spacing when the elements wrap onto multiple lines? Maybe with gap-2 or style={{ rowGap: '1rem' }} on the Row element probably?

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.

2 participants