Skip to content

feat(query-bar): add remove button to query history autocomplete - #6916

Closed
Anemy wants to merge 1 commit into
mainfrom
query-history-autocomplete-remove-button
Closed

Anemy wants to merge 1 commit into
mainfrom
query-history-autocomplete-remove-button

Conversation

@Anemy

@Anemy Anemy commented May 13, 2025

Copy link
Copy Markdown
Member
Screenshot 2025-05-12 at 8 03 02 PM

Opened as a draft since we're getting this error when deleting:
Screenshot 2025-05-12 at 8 03 49 PM
Things work as expected, it's likely a race condition in updating the auto completion items and destroying the old info. If we delete it ourselves however, it closes the autocompletion, so maybe there's a way to avoid this I haven't found yet.

@Anemy Anemy added release notes no-title-validation Skips validation of PR titles (conventional commit adherence + JIRA ticket inclusion) labels May 13, 2025
@github-actions github-actions Bot added the feat label May 13, 2025
type: query.type === 'recent' ? 'query-history' : 'favorite',
detail: formatDate(query.lastExecuted.getTime()),
info: () => createInfo(query, theme).dom,
info: () =>

@Anemy Anemy May 19, 2025 •

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

We could give a react tree here somehow.

@Anemy

Anemy commented Sep 3, 2025

Copy link
Copy Markdown
Member Author

Closing as this is stale and I haven't found time to fix it yet. There's that code mirror race condition edit change that I don't think is easily caught, although there is likely a workaround we can do.

@Anemy Anemy closed this Sep 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feat no-title-validation Skips validation of PR titles (conventional commit adherence + JIRA ticket inclusion) release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant