tesses50
/
freetube
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix issue with importing playlists and fix issue with video bar being incorrect in playlists view

This commit is contained in:
Preston 2022-02-06 16:48:24 -05:00
parent 7c6f3d2f47
commit f69c53522e
2 changed files with 12 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/renderer/components/data-settings/data-settings.js
View File

@ -1145,7 +1145,7 @@ export default Vue.extend({
playlistData.videos.forEach((video) => { playlistData.videos.forEach((video) => {
let hasAllKeys = true let hasAllKeys = true
Object.keys(video).forEach((videoKey) => { Object.keys(video).forEach((videoKey) => {
if (!requiredVideoKeys.includes(videoKey)) { if (!Object.keys(video).includes(videoKey)) {
hasAllKeys = false hasAllKeys = false
} }
}) })

21
src/renderer/views/UserPlaylists/UserPlaylists.js
View File

@ -44,16 +44,17 @@ export default Vue.extend({
} }
}, },
watch: { watch: {
// This implementation of loading effect activeData() {
// causes "scroll to top" side effect which is reported as a bug const scrollPos = window.scrollY || window.scrollTop || document.getElementsByTagName('html')[0].scrollTop
// https://github.com/FreeTubeApp/FreeTube/issues/1507 this.isLoading = true
// setTimeout(() => {
// activeData() { this.isLoading = false
// this.isLoading = true // This is kinda ugly, but should fix a few existing issues
// setTimeout(() => { setTimeout(() => {
// this.isLoading = false window.scrollTo(0, scrollPos)
// }, 100) }, 100)
// } }, 100)
}
}, },
mounted: function () { mounted: function () {
const limit = sessionStorage.getItem('favoritesLimit') const limit = sessionStorage.getItem('favoritesLimit')