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

Fix percent sized tracks #314

Merged
merged 5 commits into from
Jan 5, 2023

Conversation

nicoburns
Copy link
Collaborator

Objective

Fix percentage sized tracks:

  • Fixes percent helper which was incorrectly returning the Points variant instead of the Percent variant
  • Adds support for resolving percentages against the grid's content size in the case that the grid does not have a defined size
  • Adds tests for the above.

Context

#306 fixed issues with grid ITEMS with percentage sizes. This PR fixes issues with grid TRACKS with percentage sizing.

@nicoburns nicoburns force-pushed the grid/percent-sized-tracks branch from 77dd09c to 87a38ff Compare January 4, 2023 14:14
@nicoburns nicoburns mentioned this pull request Jan 4, 2023
87 tasks
@alice-i-cecile alice-i-cecile added the bug Something isn't working label Jan 5, 2023
@alice-i-cecile alice-i-cecile enabled auto-merge (squash) January 5, 2023 17:46
@nicoburns nicoburns force-pushed the grid/percent-sized-tracks branch from 87a38ff to 74fd28d Compare January 5, 2023 18:01
@alice-i-cecile alice-i-cecile merged commit 06a168c into DioxusLabs:main Jan 5, 2023
@nicoburns nicoburns deleted the grid/percent-sized-tracks branch January 5, 2023 18:08
@nicoburns nicoburns added this to the 0.3 "CSS Grid" milestone Jan 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants