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

Create decoding-image-default-sync.md #332

Merged
merged 2 commits into from
Dec 6, 2019
Merged

Conversation

Malvoz
Copy link
Contributor

@Malvoz Malvoz commented Aug 19, 2019

Concrete proposal for #182.

I used @ehsan-karamad's loading-image-default-eager as a template for the new proposal as they're similar in many ways.

Note that there's been no prior discussion or consensus for taking this particular approach. Alternative approaches could be discussed in #182, otherwise happy to receive feedback on this. :)

Concrete proposal for #182
------------
The `decoding` attribute can take one of the three values:
* **`sync`** indicating that the image should decode synchronously.
* **`auto`** the browser makes the call for the decoding behavior; it probably defaults to `sync`.
Copy link
Contributor Author

@Malvoz Malvoz Aug 19, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

auto [...] it probably defaults to sync.

I'm not sure if this is true, at least for Firefox @emilio's w3c/csswg-drafts#2095 (comment) suggest otherwise.

@clelland clelland merged commit ff52759 into w3c:master Dec 6, 2019
@clelland
Copy link
Collaborator

clelland commented Dec 6, 2019

Thanks for this -- I'm happy to merge it, and we can discuss in GitHub issues whether anyone is willing to implement it, or whether it should now be considered for Document Policy, or any other details.

(Sorry for the long lag there)

@Malvoz Malvoz deleted the patch-9 branch July 9, 2022 20:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants