-
Notifications
You must be signed in to change notification settings - Fork 43
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
Add WeightedContentTrackSize #23
Comments
(and not like last time...work is super light for me right now, so I'm full speed ahead 😉 ) |
I'm not quite done yet, but here's how it's looking with your user's example data: There's no need for a new track size, because this is how Video of resizing (debug build, so it's a little jumpy): |
That's fantastic! Looking forward to it. |
Hey, so qq about what your user is encountering. What |
At the moment it's |
Gotcha. Are you considering moving to a default of intrinsic? |
Yes, absolutely, for the columns that do not define a fixed width in pixels or percentage. |
Percentage is an interesting one too, because Not sure how accurate you're trying to be with compatibility. |
Not 100% sure on how html tables respond to this. In the end a 100% accuracy isn't our direct goal, but that would be nice of course. There are a lot of edge cases that you potentially have to cover to get close to a browser's behaviour, given colgroups, css,, html width, inconsistent given sizes, etc. Let's take it one step at a time. 🙂 |
👍 |
OK, got it. New sizing algorithm is working awesome. I'm going to clean it up and write some tests tomorrow, then we're off to the races! |
It would be useful to have a
WeightedContentTrackSize
(or whatever we end up calling it), which would size a track based on the weight it's contents have over the total intrinsic size of all columns.This is how an HTML table would size itself (given no other constraints). This would solve Sub6Resources/flutter_html#491.
The text was updated successfully, but these errors were encountered: