Skip to content

fix: use single columns on smaller screens#1802

Open
dede999 wants to merge 2 commits intokernelci:mainfrom
profusion:fix/use-single-columns-on-smaller-screens
Open

fix: use single columns on smaller screens#1802
dede999 wants to merge 2 commits intokernelci:mainfrom
profusion:fix/use-single-columns-on-smaller-screens

Conversation

@dede999
Copy link
Contributor

@dede999 dede999 commented Mar 13, 2026

Fixes layout responsiveness for smaller viewpoints by setting the grid size for default, medium and/or large screens

Changes

  • On BuildDetails component, I have wrapped the DetailsInfoCard with grid classes changing on large views, as I thought it was okay to have only one column on small and medium screens
  • On Subsection component, I have made it change the grid size on medium screens as sub sets generally represent more granular data

How to test

  • Visit build/$buildId page, inspect it and check different viewpoints to look for points of improval
  • This page can be reached by checking a hardware details page and then you check on builds details

Related issue

@dede999
Copy link
Contributor Author

dede999 commented Mar 13, 2026

Check for the main changes, I balieve we can still improve that grey area not covering the whole screen

Screencast.from.2026-03-13.19-51-54.webm

@MarceloRobert MarceloRobert added the Frontend Most or all of the changes for this issue will be in the frontend code. label Mar 16, 2026
@dede999 dede999 force-pushed the fix/use-single-columns-on-smaller-screens branch from 1ab18eb to 2637606 Compare March 16, 2026 19:25
{
children: (
<div className="grid grid-cols-2 gap-4">
<div className="grid grid-cols-1 gap-4 lg:grid-cols-2">
Copy link
Contributor

Choose a reason for hiding this comment

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

When going single column the cards are overflowing the screen size.
Take a double look on the cards size.

before after
Image Image

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I do not see this problem unless I switch pages on viewport inspection mode; in this case, I disable and re-enable it, and the problem is gone

Copy link
Collaborator

@MarceloRobert MarceloRobert left a comment

Choose a reason for hiding this comment

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

your changes made it so that the treeDetails page now has a horizontal scroll. It shows the tables entirely, but makes the cards have a very useless empty space
Image

also, cards in details pages can still clip to the right; not necessary, but it's a nice-to-have
Image

@dede999
Copy link
Contributor Author

dede999 commented Mar 19, 2026

your changes made it so that the treeDetails page now has a horizontal scroll. It shows the tables entirely, but makes the cards have a very useless empty space Image

also, cards in details pages can still clip to the right; not necessary, but it's a nice-to-have Image

I was able to fix that

Screencast.from.2026-03-19.17-59-46.webm

@alanpeixinho
Copy link
Contributor

I believe the current behaviour (on prod) for the Test Details page is mostly fine.
use_single_columns_main
We might keep our changes only for the Build Details page.

Previoulsy, the gray-painted div did not extend to the end of the
screen, which was not the intended layout

Closes kernelci#1432
@dede999 dede999 force-pushed the fix/use-single-columns-on-smaller-screens branch from f27a427 to a139498 Compare March 20, 2026 21:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Frontend Most or all of the changes for this issue will be in the frontend code.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Change details pages infor to be single column on small screens

3 participants