-
Notifications
You must be signed in to change notification settings - Fork 238
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
feat: improve webapp, improve mobile player, improve design ✨ #629
Open
IEduStu
wants to merge
88
commits into
Stremio:development
Choose a base branch
from
IEduStu:mobileImprovements
base: development
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
88 commits
Select commit
Hold shift + click to select a range
8f350dc
feat: add dev container config
IEduStu 1089a62
fix: dev container config
IEduStu e45db84
fix: make exposed dev port public by default
IEduStu 054bb8e
fix: add note about exposing port in dev container
IEduStu 382eb44
fix: make maskable icons not get overriden during the build
IEduStu 2eb8d3d
feat: improve mobile design
IEduStu c4c7f20
feat: improve mobile design
IEduStu d8b9f10
feat: mobile controls overlay
IEduStu a9090ca
fix: bugs
IEduStu ff90922
fix: bugs
IEduStu 14e5d6c
fix: bugs
IEduStu 8d529ad
feat: open video in nPlayer on iOS
IEduStu 45bfd45
fix: colors
IEduStu c99a649
fix: iOS webapp config
IEduStu 6e22348
fix: remove double scrollbars
IEduStu c74879b
fix: preserve existing desktop play behavior
IEduStu 2769b03
build: allow manually triggering the CI
IEduStu 3d14818
test: fix tests
IEduStu 6a636c9
style: lint
IEduStu d6806fb
build: make deploying to GitHub Pages work
IEduStu d405cf3
build: fix config
IEduStu 18d66fa
build: make deploying to GitHub Pages work
IEduStu 0e2b259
fix: toggle show controls on tap when buffering on mobile
IEduStu 2abceb3
fix: bugs
IEduStu e6061f1
fix: support toggling video fullscreen on iOS
IEduStu aea1518
fix: bugs
IEduStu de54300
feat: workaround to hide iOS navigation bar in the player on the webapp
IEduStu d00e6b1
pr: update workflow
IEduStu 46d439c
pr: fix shade in addons screen
IEduStu d50e02c
pr: fix Firefix flickers in the Discover screen
IEduStu ac078a3
pr: fix layouts edge cases, text cropping, login screen, search scree…
IEduStu 09efeff
feat: add support for the wordaround to hide iOS system navigation ba…
IEduStu d5bab29
fix: focus on the correct settings section button
IEduStu 912f0ed
fix: gliches in Safari in Settings screen
IEduStu ff4c2f5
fix: layout bugs, settings screen bug
IEduStu 34c4411
refactor: switch from `env()` to css vars
IEduStu 782df15
fix: no hover on touch screens
IEduStu ca278bb
fix: focus scroll on scrollable elements
IEduStu ae1f859
fix: improve navigation buttons contrast
IEduStu 8dbd63e
fix: make slider work correctly on touch screens
IEduStu 72bfd86
fix: make screen switching animation feel smoother
IEduStu f308360
fix: animation was too long in the user menu
IEduStu 9834ac4
fix: fine-tune touch gesture recofnition numbers
IEduStu 02d011c
fix: smaller UI areas on mobile in the player
IEduStu cf5ff44
fix: subtitles should not render under unsafe areas
IEduStu 7ab52f9
fix: double tap gesture called twice
IEduStu 50668c9
fix: toast shadow was clipped
IEduStu 7a0a992
fix: video area should expand blow unsafe areas
IEduStu 691fc7a
feat: airplay support on Safari
IEduStu 474c004
fix: airplay active state effects
IEduStu 014e90d
fix: improve Airplay active state detection
IEduStu b16ae81
fix: show the Airplay button on all Safari browsers
IEduStu f6da39e
fix: toasts container caused a scroll on the library screen
IEduStu 8c69dd2
fix: make player menu font more clear
IEduStu 101c461
fix: don't render embedded subtitles under safe areas
IEduStu ac072ee
fix: pad top and bottom sides of the video to not cover the bottom un…
IEduStu 51ce1b1
fix: only pad the bottom of the video
IEduStu 584acb8
pr: support multi-branch and single-branch GitHub Pages deployments
IEduStu e4cba7a
fix: slider touch end event
IEduStu 06130ea
fix: remove duplicate top navbar top safe inset margin
IEduStu edd2d8d
fix: make sure the scrollbar is not cropped on all platforms, improve…
IEduStu 1081a1e
fix: generic detection of overlay scrollbars
IEduStu 9e31fda
fix: when clicking on the `Download this video` button in the player,…
IEduStu 48198a4
fix: simplify download video button logic
IEduStu 097af8d
feat: copy download link button, use i18n
IEduStu e380c7e
fix: align texts with new translations PR
IEduStu ba55ea9
fix: workarounds for CSS issues on Safari
IEduStu ff2d9a9
fix: position issue
IEduStu b2fdded
fix: check for `overflow: clip` support
IEduStu a744f02
Merge remote-tracking branch 'origin/development' into mobileImprovem…
IEduStu f005cc8
fix: shrink header in player only when there's no enough space
IEduStu c817d6f
fix: dialog buttons direction on mobile
IEduStu b9304b4
fix: attempt to fix status bar color issue
IEduStu 0fccb85
Merge remote-tracking branch 'origin/development' into mobileImprovem…
IEduStu 1e0a56c
fix: focused settings scrtion buttons on some zoom levels
IEduStu 51fe50f
chore: use new translation texts from sub-PR
IEduStu 6392137
fix: streaming server message margin
IEduStu 2911efd
fix: CI workflow
IEduStu 30f0346
fix: focused settings section buttons
IEduStu 25a62ad
fix: focused settings section buttons
IEduStu 85544ca
Merge remote-tracking branch 'origin/development' into mobileImprovem…
IEduStu b791e7e
Merge remote-tracking branch 'origin/development' into mobileImprovem…
IEduStu ed078e1
fix: CR comments
IEduStu 3513606
style: lint
IEduStu f357299
Merge remote-tracking branch 'origin/development' into mobileImprovem…
IEduStu f15e365
fix: rever to last not broken `@stremio/stremio-core-web` version
IEduStu 596190a
Merge remote-tracking branch 'origin/development' into mobileImprovem…
IEduStu 1aa1c11
Merge remote-tracking branch 'origin/development' into mobileImprovem…
IEduStu File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
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 |
---|---|---|
@@ -0,0 +1,28 @@ | ||
// For format details, see https://aka.ms/devcontainer.json. For config options, see the | ||
// README at: https://github.com/devcontainers/templates/tree/main/src/typescript-node | ||
{ | ||
"name": "Stremio Web dev container", | ||
"image": "mcr.microsoft.com/devcontainers/typescript-node:1-20-bullseye", | ||
"features": { | ||
"ghcr.io/devcontainers/features/git:1": {}, | ||
"ghcr.io/devcontainers/features/git-lfs:1": {} | ||
}, | ||
"forwardPorts": [8080], | ||
"portsAttributes": { | ||
"8080": { | ||
"label": "Web Server (don't forget to manually change this port to public in the Ports tab in VS Code to access it externally)", | ||
"onAutoForward": "notify", | ||
"protocol": "https" | ||
} | ||
}, | ||
"postCreateCommand": "npm install", | ||
"customizations": { | ||
"vscode": { | ||
"settings": { | ||
"window.menuBarVisibility": "visible", | ||
"files.autoSave" : "afterDelay", | ||
"window.commandCenter": true | ||
} | ||
} | ||
} | ||
} |
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
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
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
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
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
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
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
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
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
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
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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@IEduStu Could you create one more branch out of mobileImprovements branch in your fork, for me to see how the new CD pipeline will behave?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
You can see the CI of a commit on the forked branch here.
I've make this change to the CI to make it deploy the build to GitHub Pages, since the existing action for this didn't work.
This new code uses GitHub Action's official actions to deploy to GitHub Pages.
To make it work for any fork, you just have to enable GitHub Actions on the fork and enable GitHub Pages in the repo settings and it would work without configuring any environment variable or something like that.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The expected result is a new directory in the gh-pages branch, with the name of the new branch so that we can access it at
https://stremio.github.io/stremio-web/<new-branch-name>/
The goal is to have all branches deployed to gh pages simultaniously
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I originally made this change because I thought the deployment didn't work, but it seems that it did just in a different manner than I anticipated.
This is really cool :)
I see how useful it is to have it for a repo with multiple developers working on it, but I think it may lead other forks the same path that I went - thinking the deployment didn't work.
So I made another change for the CI to use the previous action when an environment variable named
GITHUB_PAGES_MULTI_BRANCH
is set totrue
, and otherwise use the simpler deploy action.This way you can set
GITHUB_PAGES_MULTI_BRANCH
totrue
in this repo settings and preserve the current behavior, and forks that people create won't inherit this environment variable and thus will have the simpler deploy action.The idea of this is to remove friction in developing this repo on forks, while preserving the current CI behavior.
Let me know if you think it's a good idea or whether should I just revert this whole CI change.