Skip to content

Fix focus after search#22

Merged
kdkavanagh merged 6 commits intomasterfrom
fix-focus
Mar 24, 2026
Merged

Fix focus after search#22
kdkavanagh merged 6 commits intomasterfrom
fix-focus

Conversation

@kdkavanagh
Copy link
Copy Markdown
Owner

@kdkavanagh kdkavanagh commented Mar 24, 2026

No description provided.

Claude Code and others added 2 commits March 24, 2026 23:25
Bump the minimum textual version from 6.0.0 to 8.1.1 (latest) and add a
Claude Code skill documenting Textual testing and debugging patterns.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
After submitting a search query, the FilterBox remained mounted and
focused, preventing keyboard navigation (n/N) of search results.
Now the FilterBox is removed and focus is returned to the main table
upon search submission.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@kdkavanagh kdkavanagh changed the title Update textual dependency to >= 8.1.1 and add testing skill Fix focus after search Mar 24, 2026
Claude Code and others added 4 commits March 24, 2026 23:34
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Replace black and isort with ruff for formatting and import sorting
- Add ruff linting (pycodestyle, pyflakes, isort, pyupgrade, bugbear, simplify)
- Migrate all .pylintrc config into pyproject.toml and delete .pylintrc
- Add mypy to dev dependencies with config in pyproject.toml
- Fix all pre-existing ruff lint warnings (unused var, long lines, zip strict, etc.)
- All checks now pass: ruff, pylint (10/10), mypy, pytest

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Remove unused abstract methods (save_state, load_state) from HasState
- Replace Style | None with Style.null() defaults to eliminate union-attr errors
- Add _color_name() helper for safe Color | None access
- Fix type annotations: struct dict params, table_name Path | str, epoch Literal
- Add early return guard in action_iter_search for None-safe queue/idx access
- Use cast() for polars .max() return values in _measure()

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@kdkavanagh kdkavanagh merged commit 10341f4 into master Mar 24, 2026
2 checks passed
@kdkavanagh kdkavanagh deleted the fix-focus branch March 24, 2026 23:53
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