Skip to content

Conversation

@JayGeorge
Copy link

@JayGeorge JayGeorge commented Nov 14, 2025

The asset field works well on its own but it needs some adjustments when used in the Grid fieldtype.

Before

2025-11-14_ at 16 27 36@2x

Stacked UI Before

The stacked UI is actually forced when it comes to mobile views, even when the table UI is selected

2025-11-17 at 10 39 04@2x

After

Table UI After

Various states:

  • Wraps a gray border around the asset cell and rounds the corners
  • Adds a shadow and compressed padding when its in a row

Initially I tried wrangjangling Vue to change the classes, as you can see in one of my earlier commits, but in the end it was much easier, less complex, and performant to do this in a few lines of CSS.

2025-11-14 at 16 26 11@2x

Stacked UI After

The stacked UI is actually forced when it comes to mobile views, even when the table UI is selected

2025-11-17 at 10 39 59@2x

@JayGeorge JayGeorge marked this pull request as draft November 14, 2025 16:34
@jasonvarga
Copy link
Member

I wonder if you could make your life a little easier if you just apply the sr-only class to the whole thead instead of the cells.

@JayGeorge
Copy link
Author

@jasonvarga that's indeed a better idea!

@JayGeorge JayGeorge marked this pull request as ready for review November 17, 2025 10:40
@jasonvarga jasonvarga merged commit c53c9f5 into master Nov 17, 2025
18 checks passed
@jasonvarga jasonvarga deleted the asset-fixes-for-the-grid-fieldtype branch November 17, 2025 19:54
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