The Wayback Machine - https://web.archive.org/web/20230117193049/https://github.com/whatwg/html/pull/8703
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

Editorial: remove conformance text around limits #8703

Merged
merged 2 commits into from Jan 16, 2023
Merged

Conversation

annevk
Copy link
Member

@annevk annevk commented Jan 9, 2023

Copy link
Member

@domenic domenic left a comment

Choose a reason for hiding this comment

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

LGTM but would be nicer with cross-links. Tagging "do not merge yet" until Infra is merged.

User agents are expected to have limits in how big images can be rendered,
which is allowed by the <a href="#hardwareLimitations">hardware limitations</a> clause.</p>
<p class="note">If <var>density</var> is zero, the <span>intrinsic dimensions</span> will be
infinite. User agents are expected to have limits in how big images can be rendered.</p>
Copy link
Member

Choose a reason for hiding this comment

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

This could cross-link to Infra if we add a section header like I suggested in that PR.

@domenic domenic added the do not merge yet Pull request must not be merged per rationale in comment label Jan 10, 2023
@annevk annevk removed the do not merge yet Pull request must not be merged per rationale in comment label Jan 16, 2023
@annevk annevk merged commit 7488ce0 into main Jan 16, 2023
2 checks passed
@annevk annevk deleted the annevk/infra-limits branch Jan 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants