Skip to content

fix(kanban): use FALLBACK_COLUMN_LABEL in column select-all aria-label#23637

Closed
liuhao1024 wants to merge 1 commit into
NousResearch:mainfrom
liuhao1024:fix/kanban-column-label-aria-23620
Closed

fix(kanban): use FALLBACK_COLUMN_LABEL in column select-all aria-label#23637
liuhao1024 wants to merge 1 commit into
NousResearch:mainfrom
liuhao1024:fix/kanban-column-label-aria-23620

Conversation

@liuhao1024

Copy link
Copy Markdown
Contributor

Summary

Fixes ReferenceError: COLUMN_LABEL is not defined in the Kanban dashboard's column select-all checkbox.

Root Cause

The i18n refactor in commit c39168453 renamed COLUMN_LABEL to FALLBACK_COLUMN_LABEL but missed one reference at line 1901 of plugins/kanban/dashboard/dist/index.js — the aria-label attribute of the column header checkbox.

Fix

Changed COLUMN_LABELFALLBACK_COLUMN_LABEL on the single affected line. All other references (lines 59, 97) already use the correct name.

Testing

  • Verified all COLUMN_LABEL references now use FALLBACK_COLUMN_LABEL
  • Ran kanban Python tests: 168 passed
  • No frontend test framework exists for the dashboard; change is a single variable rename in a template literal

Fixes #23620

The i18n refactor (c391684) renamed COLUMN_LABEL to
FALLBACK_COLUMN_LABEL but missed the aria-label reference at
line 1901 of plugins/kanban/dashboard/dist/index.js.

This caused a ReferenceError: COLUMN_LABEL is not defined when
rendering the column select-all checkbox.

Fixes NousResearch#23620
@alt-glitch alt-glitch added type/bug Something isn't working P3 Low — cosmetic, nice to have comp/plugins Plugin system and bundled plugins labels May 11, 2026
@jroemusic

Copy link
Copy Markdown

Closing as superseded by #23659, which uses the more complete fix (resolves colLabel via getColumnLabel() instead of directly referencing FALLBACK_COLUMN_LABEL). Both fix the same ReferenceError: COLUMN_LABEL is not defined but #23659 follows the current i18n fallback pattern.

@kshitijk4poor

Copy link
Copy Markdown
Collaborator

Merged via salvage PR #23791 using #23659's implementation. Your fix correctly identified the undefined bug and used — but #23659's use of the already-in-scope variable is i18n-aware (it matches the visible column label on line 1911). Thanks for the contribution — your finding helped confirm the bug!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/plugins Plugin system and bundled plugins P3 Low — cosmetic, nice to have type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Kanban dashboard: COLUMN_LABEL undefined in aria-label at column select-all checkbox

4 participants