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

Add Skip-Ad media session action #957

Closed
1 task done
jiajiabingcheng opened this issue May 15, 2024 · 7 comments
Closed
1 task done

Add Skip-Ad media session action #957

jiajiabingcheng opened this issue May 15, 2024 · 7 comments
Assignees
Labels
Missing: venue Mode: breakout Work done during a time-limited breakout session Resolution: satisfied with concerns The TAG is satisfied with this work overall but requires changes Topic: media

Comments

@jiajiabingcheng
Copy link

jiajiabingcheng commented May 15, 2024

こんにちは TAG-さん!

I'm requesting a TAG review of Adding Skip-Ad media session action.

This feature was initially proposed and implemented four years ago but remained disabled due to a lack of practical use cases. It already received LGTM from the Blink dev group back in February 2019. Given our team's plan to implement a feature related to this action, we are now proposing to enable it.

Further details:

  • I have reviewed the TAG's Web Platform Design Principles
  • Relevant time constraints or deadlines: [please provide]
  • The group where the work on this specification is currently being done:
  • The group where standardization of this work is intended to be done (if current group is a community group or other incubation venue):
  • Major unresolved issues with or opposition to this specification:
  • This work is being funded by:

You should also know that...

[please tell us anything you think is relevant to this review]


CAREFULLY READ AND DELETE CONTENT BELOW THIS LINE BEFORE SUBMITTING

Please preview the issue and check that the links work before submitting.

In particular, if anything links to a URL which requires authentication (e.g. Google document), please make sure anyone with the link can access the document. We would prefer fully public documents though, since we work in the open.

¹ We require an explainer to give the relevant context for the spec review, even if the spec has some background information. For background, see our explanation of how to write a good explainer. We recommend the explainer to be in Markdown.

² A Security and Privacy questionnaire helps us understand potential security and privacy issues and mitigations for your design, and can save us asking redundant questions. See https://www.w3.org/TR/security-privacy-questionnaire/.

³ For your own organization, you can simply state the organization's position instead of linking to it. Chromium doesn't have a standards-positions repository and prefers to use comments from the teams that maintain the relevant area of their codebase.

@torgo
Copy link
Member

torgo commented Aug 14, 2024

Just a quick note - and noting this is not formal TAG feedback, but a question that came up in our discussion in this morning's breakout call where we discussed this: could this API could be generalized so that it's not specifically referencing advertizing but more the general use case of skipping an intro or skipping "to the main content"? It feels like there are multiple cases where this could be used that are not advertizing related, and also we don't currently have other APis in the platform that are just supporting advertising.

@martinthomson
Copy link
Contributor

Along the lines that @torgo mentions, it occurs to me that video services often have the ability to skip an intro sequence. This fills a similar function, so if you are looking for generality, that might be worth looking into.

@plinss plinss added the Progress: pending external feedback The TAG is waiting on response to comments/questions asked by the TAG during the review label Aug 19, 2024
@martinthomson
Copy link
Contributor

Firstly, we need to ask whether this was implemented in browsers other than Chrome. If it wasn't, this presents an opportunity to improve it.

One desire we have is to make this feature generic, rather than being specific to skipping ads.

We observe that the Media Chapter information (that we recently reviewed) might provide a useful hook for this sort of control. "Chapters" that are skippable can be marked as such in that metadata. The type of thing to skip might be included in that information. Consider { ..., skippable: "ad" }. That would allow for skipping of intro sequences and credits in addition to ads.

For accessibility purposes, this sort of thing might need to be given a symbol so that non-textual users are able to more readily access the function. Please consult an accessibility expert about this (@matatk indicated that APA is likely to want to take another look; it's been a while since they reviewed the original MediaSession proposal).

@miketaylr
Copy link

Firstly, we need to ask whether this was implemented in browsers other than Chrome. If it wasn't, this presents an opportunity to improve it.

Looks to be implemented in Gecko for quite some time:

https://searchfox.org/mozilla-central/source/dom/webidl/MediaSession.webidl#16-26
https://bugzilla.mozilla.org/show_bug.cgi?id=1582569

@chrisn
Copy link

chrisn commented Sep 4, 2024

I very much agree with #957 (comment) that a more generic skip action would be useful.

@torgo torgo modified the milestones: 2024-09-02-week, 2024-09-09-week Sep 5, 2024
@torgo torgo added Progress: propose closing we think it should be closed but are waiting on some feedback or consensus and removed Progress: pending external feedback The TAG is waiting on response to comments/questions asked by the TAG during the review labels Sep 9, 2024
@martinthomson
Copy link
Contributor

I just opened w3c/mediasession#340 to track that proposal.

@matatk
Copy link

matatk commented Sep 18, 2024

Thanks for the review request. As this is shipping in multiple engines, and our concerns have been raised in an issue to be addressed in future, we'll close this review.

@matatk matatk closed this as completed Sep 18, 2024
@matatk matatk removed the Progress: propose closing we think it should be closed but are waiting on some feedback or consensus label Sep 18, 2024
@matatk matatk added the Resolution: satisfied with concerns The TAG is satisfied with this work overall but requires changes label Sep 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Missing: venue Mode: breakout Work done during a time-limited breakout session Resolution: satisfied with concerns The TAG is satisfied with this work overall but requires changes Topic: media
Projects
None yet
Development

No branches or pull requests

7 participants