-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Remove width attribute fallback from sizes calculation #5900
Conversation
Yeah, given the non-support this seems fine to remove. @dholbert FYI. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@zcorpan should probably review this.
You can test this without changing DPI: http://software.hixie.ch/utilities/js/live-dom-viewer/saved/8439 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
With the last nit addressed and a WPT as per @zcorpan's comment this can land. Thanks for tackling this!
...from the "Updating the source set" algorithm. Mop-up for Issue whatwg#5893, completing work started in PR whatwg#5900.
...from the "Updating the source set" algorithm. Mop-up for Issue whatwg#5893, completing work started in PR whatwg#5900.
Closes #5893
@annevk - Can you confirm #5893 (comment) represents Mozilla's support?
Regarding WPTs - seems like this requires web-platform-tests/wpt#7140 in order to be WPT-testable (otherwise, the test would depend on the DPR of the screen on which it's running).
No need for implementation bugs, as this removes part of the spec that was not implemented anywhere.
(See WHATWG Working Mode: Changes for more details.)
/images.html ( diff )