Remove monkey patch sections

npm1 commented 3 years ago

There are 3 sections that monkey patch other specifications (the ones with name starting with "Interaction with "). The affected specs should probably be updated instead, and those sections should be removed.

chrisn commented 1 year ago

Thanks for raising this, and apologies for taking so long to respond. I'm noting the TAG guidance, so looking at each of those sections:

Interaction with Fullscreen - It could well make sense to add to the Fullscreen spec.
Interaction with Remote Playback - This seems informative, so could be added to the Note after local playback device, e.g:

NOTE: A local playback device might have extra outputs, like an external display or speakers/headphones, or a Picture-in-Picture window.
Interaction with Media Session - I'm not sure what this section is saying. The only thing I see in Media Session is the change proposed in https://github.com/w3c/mediasession/pull/295. Do we need this section at all?
Interaction with Page Visibility - The first paragraph should probably remain in this spec as it's about the PiP window itself. The second paragraph, on system visibility state of a traversable navigable, could be added to the Page Visibility section in HTML.

@beaufortfrancois, @jernoble, what do you think?

chrisn commented 10 months ago

Discussed "Interaction with Remote Playback" in the Second Screen WG meeting at TPAC. The group agreed to move this text to the Remote Playback API spec, but will need discussion if it needs to be normative.

mfoltzgoogle commented 10 months ago

To clarify: Second Screen expects someone working on the PiP spec to open a PR against the Remote Playback API.

chrisn commented 3 months ago

Regarding Interaction with Page Visibility, #223 partially addresses this, but would leave this text remaining:

The Picture-in-Picture window visibility MUST NOT be taken into account by the user agent to determine if the system visibility state of a traversable navigable has changed.

@beaufortfrancois and @marcoscaceres Any thoughts on what to do with this? Should it remain somewhere in this spec, or could be added to the Page Visibility section in HTML?

beaufortfrancois commented 3 months ago

I'd hope we could add this part to the HTML Page Visibility spec now that PiP has been implemented by two different implementations.

w3c / picture-in-picture

Remove monkey patch sections #191