Skip to content

Added "last" and a reversed ?playlistPosition in playlist URL #3974

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.

Sign up for GitHub

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

Merged
Chocobozzz merged 6 commits into from
Apr 26, 2021
Merged

Added "last" and a reversed ?playlistPosition in playlist URL #3974

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
177b7e9
Added "last" and a reversed playlistPosition
Poslovitch Apr 17, 2021
f1ea475
Fixed lint errors in video-watch component
Poslovitch Apr 17, 2021
b15d7d0
Applied requested changes
Poslovitch Apr 24, 2021
b3d9571
Removed debug logs
Poslovitch Apr 24, 2021
db3623a
Fixed lint
Poslovitch Apr 24, 2021
c257486
Playlist position styling
Chocobozzz Apr 26, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Playlist position styling
  • Loading branch information
@Chocobozzz
Chocobozzz committed Apr 26, 2021
commit c257486434e35af70730d63dee843ff2df5b256d
9 changes: 6 additions & 3 deletions client/src/app/+videos/+video-watch/video-watch.component.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -190,11 +190,14 @@ export class VideoWatchComponent implements OnInit, OnDestroy {
this.queryParamsSub = this.route.queryParams.subscribe(queryParams => {
// Handle the ?playlistPosition
const positionParam = queryParams[ 'playlistPosition' ]
if (positionParam === 'last') this.playlistPosition = -1 // Handle the "last" index
else this.playlistPosition = parseInt(positionParam, 10)

this.playlistPosition = positionParam === 'last'
? -1 // Handle the "last" index
: parseInt(positionParam, 10)

if (isNaN(this.playlistPosition)) {
console.error(`playlistPosition query param '${positionParam}' was parsed as NaN, defaulting to 1.`)
this.playlistPosition = 1
console.error("'?playlistPosition=" + positionParam + "' was parsed as NaN, defaulting to 1.")
}

this.videoWatchPlaylist.updatePlaylistIndex(this.playlistPosition)
Expand Down