- From: Copilot <notifications@github.com>
- Date: Tue, 28 Oct 2025 01:58:39 -0700
- To: w3c/screen-orientation <screen-orientation@noreply.github.com>
- Cc: Subscribed <subscribed@noreply.github.com>
- Message-ID: <w3c/screen-orientation/pull/273/review/3387470930@github.com>
@Copilot commented on this pull request.
## Pull Request Overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.
---
💡 <a href="/w3c/screen-orientation/new/gh-pages/.github?filename=*.instructions.md" class="Link--inTextBlock" target="_blank" rel="noopener noreferrer">Add Copilot custom instructions</a> for smarter, more guided reviews. <a href="https://docs.github.com/en/copilot/customizing-copilot/adding-repository-custom-instructions-for-github-copilot" class="Link--inTextBlock" target="_blank" rel="noopener noreferrer">Learn how to get started</a>.
> + The user agent does not support screen orientation locking, or
+ locking is not possible in the current context. This includes cases
+ where:
+ <ul>
+ <li>The user agent does not implement the Screen Orientation API
+ </li>
+ <li>The document is not [=Document/fully active=]
+ </li>
+ <li>The document has the [=sandboxed orientation lock browsing
+ context flag=] set
+ </li>
+ <li>The document's [=Document/visibility state=] is "hidden"
+ </li>
+ <li>The current context doesn't meet the [=pre-lock conditions=]
+ </li>
+ </ul>This keyword value evaluates as false in the [=boolean
Missing space after the closing `</ul>` tag. There should be a space between `</ul>` and 'This' to ensure proper text formatting.
```suggestion
</ul> This keyword value evaluates as false in the [=boolean
```
--
Reply to this email directly or view it on GitHub:
https://github.com/w3c/screen-orientation/pull/273#pullrequestreview-3387470930
You are receiving this because you are subscribed to this thread.
Message ID: <w3c/screen-orientation/pull/273/review/3387470930@github.com>
Received on Tuesday, 28 October 2025 08:58:43 UTC