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
Next Next commit
Fixed lint errors in video-watch component
  • Loading branch information
@Poslovitch
Poslovitch committed Apr 17, 2021
commit f1ea475faee4e7cf97deeb4424c5a7378734cd7d
4 changes: 2 additions & 2 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 @@ -191,8 +191,8 @@ export class VideoWatchComponent implements OnInit, OnDestroy {
// Handle the ?playlistPosition
let position = queryParams[ 'playlistPosition' ]
Comment thread
Poslovitch marked this conversation as resolved.
Outdated
if (position === 'last') position = -1 // Handle the "last" index
if (typeof position === 'string') position = parseInt(position); // Sanitization needed for the "reverse" index
if (isNaN(position)) position = 1;
if (typeof position === 'string') position = parseInt(position, 10) // Sanitization needed for the "reverse" index
if (isNaN(position)) position = 1
Comment thread
Poslovitch marked this conversation as resolved.
Outdated

if (this.playlist && position < 0) this.playlistPosition = this.playlist.videosLength - position + 1
Comment thread
Poslovitch marked this conversation as resolved.
Outdated
else this.playlistPosition = position
Expand Down