Skip to content

chore(tui): remove stale TCSS selectors - #1266

Merged
Kohei-Wada merged 1 commit into
mainfrom
chore/remove-stale-tcss
Oct 17, 2026
Merged

Kohei-Wada merged 1 commit into
mainfrom
chore/remove-stale-tcss

Conversation

@Kohei-Wada

Copy link
Copy Markdown
Owner

Removes item 3 of #1160: #filterable-task-table (and its :focus-within rule) in components.tcss, and #table-title in main.tcss. No widget in taskdog-ui/src or its tests uses either id (grep, including dynamically built ids).

Verification: make test-ui passes, including test_keybindings, which boots the full TaskdogTUI via run_test() and therefore parses the app's TCSS.

Items 1–2 of #1160 remain open.

Refs #1160

#filterable-task-table and #table-title match no widget id anywhere in
the codebase.

Refs #1160

@amazon-q-developer amazon-q-developer Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review Summary

This PR removes three stale TCSS selectors (#filterable-task-table, #filterable-task-table:focus-within, and #table-title) that are no longer referenced by any widget in the codebase. The changes are pure dead-code deletion with no logic, security, or functional concerns.

Both files are clean after the removal, and the remaining selectors are unaffected. No blocking defects found — this PR is safe to merge.


You can now have the agent implement changes and create commits directly on your pull request's source branch. Simply comment with /q followed by your request in natural language to ask the agent to make changes.

@Kohei-Wada
Kohei-Wada merged commit 0308fcf into main Oct 17, 2026
18 checks passed
@Kohei-Wada
Kohei-Wada deleted the chore/remove-stale-tcss branch October 17, 2026 01:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant