Skip to content

fix(quick-tools): add visible close button to search when toggler is off - #2570

Merged
bajrangCoder merged 2 commits into
mainfrom
fix/quick-tools-search-close
Aug 2, 2026
Merged

fix(quick-tools): add visible close button to search when toggler is off#2570
bajrangCoder merged 2 commits into
mainfrom
fix/quick-tools-search-close

Conversation

@bajrangCoder

Copy link
Copy Markdown
Member

No description provided.

@greptile-apps

greptile-apps Bot commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

Adds an inline search close button and keeps its visibility synchronized with the floating close control.

  • Adds the close control to the first search row.
  • Displays the inline control whenever the floating toggler is unavailable or hidden.
  • Observes toggler visibility and mounting changes while search is active, then disconnects during cleanup.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains; the current observer covers the settings, file-visibility, and floating-button transitions underlying the previous stale-close issue.

Important Files Changed

Filename Overview
src/components/quickTools/footer.js Adds the inline close button to the search toolbar.
src/components/quickTools/style.scss Hides the inline close button by default and reveals it through the synchronized state class.
src/handlers/quickTools.js Synchronizes inline-close visibility with all current floating toggler class and DOM-presence transitions.

Reviews (2): Last reviewed commit: "fix" | Re-trigger Greptile

Comment thread src/handlers/quickTools.js Outdated
@bajrangCoder

This comment was marked as outdated.

@bajrangCoder
bajrangCoder added this pull request to the merge queue Aug 2, 2026
Merged via the queue into main with commit 5167445 Aug 2, 2026
13 checks passed
@github-project-automation github-project-automation Bot moved this from Backlog to Done in The Code Board - Acode Aug 2, 2026
@bajrangCoder
bajrangCoder deleted the fix/quick-tools-search-close branch August 2, 2026 08:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant