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

Remove fold angle value from the specification. #58

Merged
merged 1 commit into from
Mar 9, 2021

Conversation

darktears
Copy link
Contributor

@darktears darktears commented Mar 1, 2021

Some editors and implementers of the Screen Fold API attended the Second
Screen WG/CG Virtual meeting to discuss this API. During that meeting
that group made the following resolution for DAS WG consideration
RESOLUTION: Drop ScreenFold.angle from the Screen Fold API first version (for DAS WG consideration)

A following commit will rename the specification so references can ber
renamed to better match what we're exposing.

Closes #56, #45.


Preview | Diff

@darktears
Copy link
Contributor Author

@anssiko @kenchris PTAL. It's in an odd state after this commit but the rename will clean up the rest (references etc).

index.html Outdated
@@ -90,9 +90,7 @@
<section id='abstract'>
<p>
This document specifies an API that allows web applications to request the
Copy link
Contributor

Choose a reason for hiding this comment

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

and be notified of changes?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed

@@ -159,7 +154,6 @@ <h2>
<pre class="idl">
[SecureContext, Exposed=(Window)]
interface ScreenFold : EventTarget {
Copy link
Contributor

Choose a reason for hiding this comment

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

We should rename ScreenFold etc as well, maybe different PR

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, part of the rename.

@kenchris
Copy link
Contributor

kenchris commented Mar 1, 2021

@diekus

@kenchris
Copy link
Contributor

kenchris commented Mar 2, 2021

LGTM

Copy link
Member

@anssiko anssiko left a comment

Choose a reason for hiding this comment

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

Thanks! Some nits as suggestions. In addition:

The spec currently lacks a "posture" definition. Given this term can mean many things, we should explicitly define this term in the context of this specification.

The spec has definitions for "current posture" and "posture mode" which is good.

index.html Outdated
Comment on lines 113 to 114
when the device folds, it physically can be different form factor. The main interest in knowing the posture is because there
are opportunities in responsive design that enable new user experiences.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
when the device folds, it physically can be different form factor. The main interest in knowing the posture is because there
are opportunities in responsive design that enable new user experiences.
when the device folds, it can be a physically different form factor. The main interest in knowing the posture is to enable new user experiences with responsive design.

index.html Outdated
provide a different layout for it depending on the state of the angle of the fold. Additionally, developers can
adapt content depending on various “postures” and potentially also animate some of these transitions. You can
refer to a description of use cases.
provide a different layout for it depending on the state in which the device is being used.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
provide a different layout for it depending on the state in which the device is being used.
provide a different layout for it depending on the posture state in which the device is being used.

index.html Outdated Show resolved Hide resolved
@darktears
Copy link
Contributor Author

Thanks! Some nits as suggestions. In addition:

The spec currently lacks a "posture" definition. Given this term can mean many things, we should explicitly define this term in the context of this specification.

Let make these improvements as follow up.

The spec has definitions for "current posture" and "posture mode" which is good.

Some editors and implementers of the Screen Fold API attended the Second
Screen WG/CG Virtual meeting to discuss this API. During that meeting
that group made the following resolution for DAS WG consideration
RESOLUTION: Drop ScreenFold.angle from the Screen Fold API first version (for DAS WG consideration)

A following commit will rename the specification so references can ber
renamed to better match what we're exposing.

Fixes w3c#56, w3c#45.
@anssiko
Copy link
Member

anssiko commented Mar 3, 2021

The spec currently lacks a "posture" definition. Given this term can mean many things, we should explicitly define this term in the context of this specification.

Let make these improvements as follow up.

SGTM. Opened https://github.com/w3c/screen-fold/issues/59

@anssiko
Copy link
Member

anssiko commented Mar 4, 2021

All - please review this PR and submit your feedback by the end of next Monday, 8 March 2021.

@anssiko
Copy link
Member

anssiko commented Mar 9, 2021

This PR has received adequate review and is ready to land.

@anssiko anssiko merged commit 0cfeeb1 into w3c:gh-pages Mar 9, 2021
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.

Drop ScreenFold.angle from the Screen Fold API?
3 participants