-
-
Notifications
You must be signed in to change notification settings - Fork 552
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Update building for browser stores.md
- Loading branch information
1 parent
8ff4487
commit 48a579a
Showing
1 changed file
with
25 additions
and
16 deletions.
There are no files selected for viewing
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,34 +1,43 @@ | ||
- Microsoft store has a bug/complification with languages https://github.com/code4charity/YouTube-Extension/discussions/966#discussioncomment-963315 | ||
## Our build script should consider these rules per browser&store (currently building manually): | ||
|
||
### Extension name (title in manifest.js) | ||
|
||
- [ ] Edge & Whale deny ' (Replace ' with *) | ||
- [ ] Whale denies '&' (Replace with '+') | ||
- <del> (Opera removes Emoji automatically) | ||
|
||
### June 2024, Manifest3: | ||
|
||
- Only chrome is required yet. | ||
- Firefox remains in Manifest2, because it requires a different signature for Manifest3 | ||
|
||
--- | ||
--- | ||
|
||
#### ImprovedTube: | ||
- [ ] We can include m.youtube(?) AND *.youtube.com`? AND global permission, to work for all \<video\>'s? | ||
<br> besides right now only our Google Webstore version remains with www.youtube, until we announce change | ||
|
||
### Our build script should consider these rules per browser&store: | ||
Misc: | ||
- Microsoft store has a bug/complification with languages https://github.com/code4charity/YouTube-Extension/discussions/966#discussioncomment-963315 | ||
|
||
## Previously: | ||
|
||
##### FIREFOX: | ||
|
||
- [ ] Comment out PiP button in code and menu. Or at least dont set it by defaults. | ||
- [x] Comment out PiP button in code and menu. Or at least dont set it by defaults. | ||
- done ine background.js | ||
- Version 3.2: Firefox has commented-out // video.parentNode.parentNode.pauseVideo(); (fixes autoplay-bug) | ||
- Security: remove or comment-out: "remote content" (googleapis.com/youtube) called as dynamic innerHtml | ||
- [x] code changed | ||
|
||
##### Extension name (title in manifest.js) | ||
|
||
- [ ] Edge & Whale deny ' (Replace ' with *) | ||
- [ ] Whale denies '&' (Replace with '+') | ||
- <del> Opera removes Emoji automatically | ||
|
||
#### Firefox & Opera | ||
|
||
- [ ] Require moving google analytics. Can be added differently in Firefox. Could always remain in Webstore. | ||
|
||
#### ImprovedTube: | ||
- [ ] We should include m.youtube. | ||
AND *.youtube.com + global permission, to work for all \<video\>'s. | ||
<br> besides right now only our Google Webstore version should remains with www.youtube, until we announce change | ||
- [x] currently no google analytics in Webstore too | ||
|
||
#### <del> Edge: doesnt allow a chrome update url | ||
- <del> update URL edge: https://edge.microsoft.com/extensionwebstorebase/v1/crx , replace chrome: https://clients2.google.com/service/update2/crx </del> | ||
- [x] update URL removed. (any downside?) | ||
|
||
|
||
|