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

Feature: Enable Scrolling and Strict Width to table #94

Closed
wants to merge 8 commits into from
Closed

Feature: Enable Scrolling and Strict Width to table #94

wants to merge 8 commits into from

Conversation

malzariey
Copy link
Contributor

I have faced a condition where I had many columns, static and dynamic ones. As shown below:
Screenshot from 2024-01-29 11-38-58

Added the features to enable scrolling and strictWidth to replace (th,td) width with min-wdith to enforce the columnWidths as shown below:
Screenshot from 2024-01-29 11-22-29

Copy link
Owner

@awcodes awcodes left a comment

Choose a reason for hiding this comment

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

Need to make sure this doesn't prevent the columns from stacking on mobile. The mobile stacking is one of the main features to the plugin.

Also please update the readme to reflect how to use these.

src/Components/TableRepeater.php Outdated Show resolved Hide resolved
@malzariey
Copy link
Contributor Author

malzariey commented Jan 30, 2024

sure this doesn't prevent the columns from stacking on mobile. The mobile stacking is one of the main features to the plugin.

@awcodes I have tried to test the breaking points, it doesn't seem to work for me (tried both modals and pages, any suggestions?), but I believe the the user will only use one of the two features, either scrollable or breakPoints.

Appreciate your comment

@malzariey
Copy link
Contributor Author

Canceled

@malzariey malzariey closed this Feb 6, 2024
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