Skip to content

bug fix#17

Merged
devandapaige merged 1 commit intomainfrom
feat-search
Nov 12, 2025
Merged

bug fix#17
devandapaige merged 1 commit intomainfrom
feat-search

Conversation

@devandapaige
Copy link
Copy Markdown
Owner

@devandapaige devandapaige commented Nov 12, 2025

added trim


Note

Trim search input so whitespace-only queries are ignored across filtering, URL params, and related UI.

  • Blog search (src/components/blog/BlogContent.tsx):
    • Trim searchQuery before:
      • Syncing URL params (?search=) with router.replace.
      • Rendering the clear button and "Found N posts" message.
      • Determining the "No posts found" helper text.
    • Ensures whitespace-only input is treated as empty for filtering and UI state.

Written by Cursor Bugbot for commit cf69664. This will update automatically on new commits. Configure here.

added trim
@devandapaige devandapaige merged commit cd8c065 into main Nov 12, 2025
2 checks passed
@netlify
Copy link
Copy Markdown

netlify Bot commented Nov 12, 2025

Deploy Preview for pythoness-programmer ready!

Name Link
🔨 Latest commit cf69664
🔍 Latest deploy log https://app.netlify.com/projects/pythoness-programmer/deploys/6914e0800916f50008116394
😎 Deploy Preview https://deploy-preview-17--pythoness-programmer.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

This is the final PR Bugbot will review for you during this billing cycle

Your free Bugbot reviews will reset on December 30

Details

You are on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

<path d="m21 21-4.35-4.35" />
</svg>
{searchQuery && (
{searchQuery.trim() && (
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Bug: UI Inconsistency: Clear Button Missing

The clear button is now hidden when the search input contains only whitespace characters because the condition checks searchQuery.trim() while the input value is bound to the untrimmed searchQuery. Users typing only spaces see text in the input but no way to clear it, creating an inconsistent UI state.

Fix in Cursor Fix in Web

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