Skip to content

fix: align focused-shop search placeholder with marketplace search#336

Merged
calvadev merged 2 commits intoshopstr-eng:mainfrom
23bsm038-Mahi:fix-placeholder-clean-v2
Apr 7, 2026
Merged

fix: align focused-shop search placeholder with marketplace search#336
calvadev merged 2 commits intoshopstr-eng:mainfrom
23bsm038-Mahi:fix-placeholder-clean-v2

Conversation

@23bsm038-Mahi
Copy link
Copy Markdown
Contributor

@23bsm038-Mahi 23bsm038-Mahi commented Apr 4, 2026

Summary

Aligns the focused-shop search placeholder with the main marketplace search to ensure UI consistency.

Changes

  • Updated placeholder text in components/home/marketplace.tsx
  • Fixed minor indentation issues for better readability

Why this matters

Keeps the search experience consistent across different marketplace views and helps users better understand supported search fields.

Scope

  • UI-only change
  • No impact on search logic
  • Fully backward compatible

Testing

  • Verified placeholder text matches marketplace search
  • Confirmed no change in search behavior
Screenshot 2026-04-04 030728 Screenshot 2026-04-04 031046

@vercel
Copy link
Copy Markdown

vercel bot commented Apr 4, 2026

@23bsm038-Mahi is attempting to deploy a commit to the shopstr-eng Team on Vercel.

A member of the Team first needs to authorize it.

@23bsm038-Mahi
Copy link
Copy Markdown
Contributor Author

23bsm038-Mahi commented Apr 4, 2026

Hi @calvadev @GautamBytes !
This is a small UI consistency fix. Please review when you get time

@GautamBytes
Copy link
Copy Markdown
Contributor

@23bsm038-Mahi can you solve merge conflict then its good to merge

@GautamBytes
Copy link
Copy Markdown
Contributor

LGTM!!

@23bsm038-Mahi
Copy link
Copy Markdown
Contributor Author

@GautamBytes
Resolved merge conflicts and updated placeholder while keeping upstream changes. Ready for review.

@calvadev calvadev merged commit 1a24a29 into shopstr-eng:main Apr 7, 2026
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants