Skip to content

Conversation

@mwagena
Copy link

@mwagena mwagena commented Jan 28, 2026

Description

This Pull request adds the possibility to stack the table cells on mobile screens, while maintaining the horizontal scroll on larger screens.

Visual changes

Scherm­afbeelding 2026-01-28 om 09 41 18 Scherm­afbeelding 2026-01-28 om 09 41 07

Functional changes

  • Code style has been fixed by running the composer cs command.
  • Changes have been tested to not break existing functionality.
  • Documentation is up-to-date.

@mwagena
Copy link
Author

mwagena commented Jan 28, 2026

@danharrin How would you apply the css in a more generic and clean way and also how would this be done on the heading? I had a some issues with the those. Thanks!

@danharrin danharrin added the enhancement New feature or request label Jan 28, 2026
@danharrin danharrin added this to the v4 milestone Jan 28, 2026
@danharrin
Copy link
Member

danharrin commented Jan 28, 2026

Please rebase these commits on 4.x and change the target branch

All CSS needs to be in CSS files, we do not allow Tailwind classes in Blade view files like you have. Please unmark as draft once that is refactored.

@danharrin danharrin marked this pull request as draft January 28, 2026 13:02
@mwagena mwagena changed the base branch from 5.x to 4.x January 28, 2026 18:39
@mwagena
Copy link
Author

mwagena commented Jan 28, 2026

Please rebase these commits on 4.x and change the target branch

All CSS needs to be in CSS files, we do not allow Tailwind classes in Blade view files like you have. Please unmark as draft once that is refactored.

Managed to do the rebase and updated the css logic.

@mwagena mwagena changed the base branch from 4.x to 5.x January 28, 2026 18:43
@mwagena mwagena force-pushed the 5.x-stacked-table-rows-on-mobile branch from 4ffb7d3 to eee57a5 Compare January 28, 2026 20:31
@mwagena mwagena changed the base branch from 5.x to 4.x January 28, 2026 20:31
@mwagena mwagena marked this pull request as ready for review January 28, 2026 20:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request pending review

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

2 participants