Publish minutes of 2022-04-14 meeting #202
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
xeenon
approved these changes
Apr 14, 2022
_minutes/2022-04-14-wecg.md
Outdated
| * [timothy] That's the only thing we allow in Safari. | ||
| * [rob] In the issue you mentioned “Safari 14+ allows navigations” - does that mean same-origin navigation only and not remote URLs? | ||
| * [timothy] Yes. | ||
| * [simeon] Already covered what Chrome does here, but |
There was a problem hiding this comment.
@dotproto Did you have more thoughts that you said for this line?
There was a problem hiding this comment.
Thanks for tagging this, I'm updating this comment as part of my review.
Rob--W
force-pushed
the
meeting-2022-04-14
branch
from
e9eb64c
to
3eff6a2
Compare
dotproto
requested changes
Apr 19, 2022
There was a problem hiding this comment.
I have a few polish suggestions, but two are substantive (thus flagging as "requested changes")
- Added a comment from Dave on which properties are exposed on which events.
- Fleshed out my comment on background page navigation.
Rob--W
force-pushed
the
meeting-2022-04-14
branch
from
3eff6a2
to
f1d792c
Compare
|
I have updated the PR to include the requested changes (and synced that with the Google doc), and pushed an update (along with some minor other fixes): https://github.com/w3c/webextensions/compare/3eff6a2a618c48cef033b94f42b145ed933baef3..f1d792c8a8a6251c6e35eabec2b7f5c4532f510a I've also included a link to #200 in the meeting notes, since that was one of the requested changes. |
dotproto
approved these changes
Apr 22, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Generated from https://docs.google.com/document/d/1QkwhEMtMS67JBUkl_WVPZ4lRSKoWcQNlLJSf_GwSXg8/edit using the tool and process from #105.
During this meeting we discussed or mentioned #195, #8, #191, #196, #197, #198, #199, #119 and #97. #170 was also part of the agenda, but we ran out of time before we got to discussing it.