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.

Sign up for GitHub

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

Jump to bottom

Deprecate responsive grid #81

Open
matthewpritchett opened this issue Jun 7, 2022 · 0 comments
Open

Deprecate responsive grid #81

matthewpritchett opened this issue Jun 7, 2022 · 0 comments
Labels
performance A code change that improves performance

Comments

@matthewpritchett
Copy link
Collaborator

Originally created the responsive grid to help on making the app more responsive between desktop and mobile. But the app really only uses it for the padding/margins of children on the pages. We could reduce extra views by removing the responsive grid and just adding a property on the pages that replicates the current Gutter. The TransactionEditPage is the only page that uses the columns currently and we could just replace it with a standard grid.
@matthewpritchett matthewpritchett added the performance A code change that improves performance label Jun 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
performance A code change that improves performance
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@matthewpritchett