Don't show the Indexes pane before there are indexes - #3924
Merged
Conversation
On every page load the key column showed an empty "Indexes" heading under the keys until the first sweep came back and hid it. FT._LIST is part of that sweep, so the pane was on screen for as long as one round trip to the sandbox took: invisible against a local Redis, a clear flash against redis.io — which is why it only showed up on the live site. The section and its divider are now built hidden, which is the state renderIndexes() puts them in when there is nothing to list. Nothing else changes: an FT.CREATE still makes the pane appear, and dropping the index still takes it away. Measured with the backend slowed to 1.2s a request: 5 of 60 sampled frames had the pane on screen before, 0 after. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
|
|
Contributor
Contributor
🧠 Redis MemoryFound 5 related items from repository history (5 new this commit):
Memory updated at e44c2e3 |
andy-stark-redis
approved these changes
Sep 4, 2026
andy-stark-redis
left a comment
andy-stark-redis
left a comment
Contributor
There was a problem hiding this comment.
Tried on geo indexing page - LGTM.
EliShteinman
added a commit
to EliShteinman/docs
that referenced
this pull request
Sep 5, 2026
Builds the workbench's Indexes pane hidden, so an empty heading no longer sits under the keys for the length of the first sweep. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WycKvjyQaCQQBwVEkvSAKT
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
On initial page load the workbench showed an empty indexes section below the keys. This change fixes that.