Skip to content

imageLoader: fix upscrolling showing unloaded images #2115

imageLoader: fix upscrolling showing unloaded images

imageLoader: fix upscrolling showing unloaded images #2115

Triggered via pull request July 22, 2025 11:32
@evan314159evan314159
opened #7036
Status Failure
Total duration 4m 12s
Artifacts 1

pull_request.yml

on: pull_request_target
Create comments ✍️  /  CloudFlare Pages deployment 📃🚀
6s
Create comments ✍️ / CloudFlare Pages deployment 📃🚀
Build 🏗️  /  Run production build 🏗️
3m 28s
Build 🏗️ / Run production build 🏗️
Automation 🎛️  /  Merge conflict labeling 🏷️
4s
Automation 🎛️ / Merge conflict labeling 🏷️
Quality checks 👌🧪  /  Vulnerable dependencies 🔎
7s
Quality checks 👌🧪 / Vulnerable dependencies 🔎
Run eslint suggestions
3m 5s
Run eslint suggestions
Matrix: GitHub CodeQL 🔬 / analyze
Matrix: Quality checks 👌🧪 / quality
Deploy 🚀  /  ...  /  CloudFlare Pages deployment 📃🚀
5s
Deploy 🚀 / Compose and push comment 📝 / CloudFlare Pages deployment 📃🚀
Fit to window
Zoom out
Zoom in

Annotations

2 errors, 10 warnings, and 4 notices
Quality checks 👌🧪 / Run lint 🕵️‍♂️
Process completed with exit code 1.
Quality checks 👌🧪 / Run lint 🕵️‍♂️: src/components/images/imageLoader.js#L134
'emptyImageElement' is defined but never used
Quality checks 👌🧪 / Run lint 🕵️‍♂️: src/apps/dashboard/routes/users/add.tsx#L126
Refactor this code to not nest functions more than 4 levels deep
Quality checks 👌🧪 / Run lint 🕵️‍♂️: src/apps/dashboard/routes/users/access.tsx#L239
React Hook useEffect has a missing dependency: 'userId'. Either include it or remove the dependency array
Quality checks 👌🧪 / Run lint 🕵️‍♂️: src/apps/dashboard/routes/users/access.tsx#L141
React Hook useCallback has a missing dependency: 'libraryMenu'. Either include it or remove the dependency array
Quality checks 👌🧪 / Run lint 🕵️‍♂️: src/apps/dashboard/routes/branding/index.tsx#L127
Take the required action to fix the issue indicated by this comment
Quality checks 👌🧪 / Run lint 🕵️‍♂️: src/apps/dashboard/features/sessions/utils/getNowPlayingName.ts#L17
Take the required action to fix the issue indicated by this comment
Quality checks 👌🧪 / Run lint 🕵️‍♂️: src/apps/dashboard/features/logs/api/useServerLog.ts#L14
Take the required action to fix the issue indicated by this comment
Quality checks 👌🧪 / Run lint 🕵️‍♂️: src/apps/dashboard/features/backups/api/useRestoreBackup.ts#L7
Take the required action to fix the issue indicated by this comment
Quality checks 👌🧪 / Run lint 🕵️‍♂️: src/apps/dashboard/features/backups/api/useCreateBackup.ts#L10
Take the required action to fix the issue indicated by this comment
Quality checks 👌🧪 / Run lint 🕵️‍♂️: src/apps/dashboard/features/backups/api/useBackups.ts#L10
Take the required action to fix the issue indicated by this comment
Quality checks 👌🧪 / Run lint 🕵️‍♂️: src/apps/dashboard/components/AppTabs.tsx#L56
Unexpected 'hack' comment: 'HACK: Force resizing to workaround...'
Run eslint suggestions
ESLint passes
Run eslint suggestions
Using ESLint with default configuration
Run eslint suggestions
ESLint version: 9.30.1
Run eslint suggestions
Using ESLint from: /home/runner/work/jellyfin-web/jellyfin-web/node_modules/eslint/lib/api.js

Artifacts

Produced during runtime
Name Size Digest
frontend
33.8 MB
sha256:259c6a947f1d66db2973a4ccae334ba7184235f8aa0c7edc1bc2f73606a661d2