Skip to content

Fix search input in list views in the Console#7833

Open
ryaplots wants to merge 1 commit intov3.35from
fix/list-search-focus
Open

Fix search input in list views in the Console#7833
ryaplots wants to merge 1 commit intov3.35from
fix/list-search-focus

Conversation

@ryaplots
Copy link
Contributor

Summary

References https://github.com/TheThingsIndustries/lorawan-stack-support/issues/1415

Changes

  • Remove disabling

Testing

Steps

Follow the steps in cybereye issue.

Regressions
  • Removing the disabling will mean the the field won't be disabled while the fetching for new entries is happening

Notes for Reviewers

There were two ways of fixing this:

  1. Make the field not disabled at the risk that the user will change the input while the fetch is still happening
  2. Adding more logic to the fetch table which is already huge to track that field and check is it was focused before, refocus it again.

I went for option 1. Let me know what you think


Checklist

  • Scope: The referenced issue is addressed, there are no unrelated changes.
  • Compatibility: The changes are backwards compatible with existing API, storage, configuration and CLI, according to the compatibility commitments in README.md for the chosen target branch.
  • Documentation: Relevant documentation is added or updated.
  • Testing: The steps/process to test this feature are clearly explained including testing for regressions.
  • Infrastructure: If infrastructural changes (e.g., new RPC, configuration) are needed, a separate issue is created in the infrastructural repositories.
  • Changelog: Significant features, behavior changes, deprecations and fixes are added to CHANGELOG.md.
  • Commits: Commit messages follow guidelines in CONTRIBUTING.md, there are no fixup commits left.

@ryaplots ryaplots self-assigned this Feb 18, 2026
@ryaplots ryaplots requested a review from a team as a code owner February 18, 2026 12:40
@ryaplots ryaplots added the ui/web This is related to a web interface label Feb 18, 2026
@nicholaspcr nicholaspcr added this to the v3.36.0 milestone Feb 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ui/web This is related to a web interface

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments