{{ message }}
Repository navigation
Commit 4881464
Fix all 7 ESLint errors on develop
- ableplayer-base.js: the descriptionsAudible else-if duplicated the first
condition (no-dupe-else-if), so the documented singular spelling
data-description-audible was never honored. Read the singular attribute
(options.descriptionAudible ?? data.descriptionAudible) in the second
branch, matching the existing comment's intent.
- dialog.js: import AblePlayer for the AblePlayer.getActiveDOMElement()
call (no-undef).
- dragdrop.js: remove unused thisObj locals in handleWindowButtonClick and
handleMenuChoice (no-unused-vars); document the intentionally-empty
Space/Enter branch (no-empty).
- preference.js: remove the write-only $thisLabel variable and a dead
$thisField assignment (no-unused-vars, no-useless-assignment).
npm run lint: 0 problems. jsdom test suites: 55/55 pass. Build artifacts
intentionally omitted per contributing.md.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>1 parent c0a05c9 commit 4881464
4 files changed
Lines changed: 7 additions & 11 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
| 2 | + | |
2 | 3 | | |
3 | 4 | | |
4 | 5 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
287 | 287 | | |
288 | 288 | | |
289 | 289 | | |
290 | | - | |
| 290 | + | |
291 | 291 | | |
292 | | - | |
293 | 292 | | |
294 | 293 | | |
295 | 294 | | |
| |||
309 | 308 | | |
310 | 309 | | |
311 | 310 | | |
| 311 | + | |
312 | 312 | | |
313 | 313 | | |
314 | 314 | | |
| |||
349 | 349 | | |
350 | 350 | | |
351 | 351 | | |
352 | | - | |
| 352 | + | |
353 | 353 | | |
354 | 354 | | |
355 | | - | |
356 | 355 | | |
357 | 356 | | |
358 | 357 | | |
| |||

0 commit comments