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

Layout should more carefully determine image intrinsic width/height #79

Closed
burg opened this issue Sep 18, 2012 · 1 comment
Closed

Layout should more carefully determine image intrinsic width/height #79

burg opened this issue Sep 18, 2012 · 1 comment

Comments

@burg
Copy link

@burg burg commented Sep 18, 2012

In Box::get_min_width(), the layout engine currently just uses the decoded image width/height. But, if the image couldn't be loaded, isn't finished loading, has CSS 'width' or 'height', borders, margins, or Node attributes width=, height= set, its dimensions may be different.

@bzbarsky
Copy link
Contributor

@bzbarsky bzbarsky commented Sep 18, 2012

If this is really about intrinsic dimensions, then CSS width and height do not affect those. Nor do HTML width/height attributes. The only thing that affects them is the image data. When there is no image data, there is typically no image box (with some exceptions for "during the loading process" and "in quirks mode", for which you can use somewhat arbitrary intrinsic dimensions).

@burg burg closed this Nov 7, 2012
ChrisParis pushed a commit to ChrisParis/servo that referenced this issue Sep 7, 2014
Tests for the progress element, from Dan Smith of TestTWF Seattle 2013.
glennw added a commit to glennw/servo that referenced this issue Jan 16, 2017
Implement better scan and split strategies for the atlas allocator, and use as large of a texture as the hardware allows.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.