Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Bug fixed - Added placeholder in search bar in apps section #11930

Closed
wants to merge 4 commits into from
Closed

fix: Bug fixed - Added placeholder in search bar in apps section #11930

wants to merge 4 commits into from

Conversation

kilbha
Copy link

@kilbha kilbha commented Oct 17, 2023

Added place holder in search bar in apps section.
Fixed: #11927

@vercel
Copy link

vercel bot commented Oct 17, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
ui ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 17, 2023 9:46am

@CLAassistant
Copy link

CLAassistant commented Oct 17, 2023

CLA assistant check
All committers have signed the CLA.

@vercel
Copy link

vercel bot commented Oct 17, 2023

@kilbha is attempting to deploy a commit to the cal Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions
Copy link
Contributor

github-actions bot commented Oct 17, 2023

Thank you for following the naming conventions! 🙏 Feel free to join our discord and post your PR link to collect XP and win prizes!

@github-actions github-actions bot added 🐛 bug Something isn't working ❗️ .env changes contains changes to env variables labels Oct 17, 2023
@kilbha kilbha changed the title Fixed Issue: 11927 Fix: Added placeholder in seachbar in apps section Oct 17, 2023
@kilbha kilbha changed the title Fix: Added placeholder in seachbar in apps section Added placeholder in search bar in apps section Oct 17, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Oct 17, 2023

📦 Next.js Bundle Analysis for @calcom/web

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

@kilbha kilbha changed the title Added placeholder in search bar in apps section Fix: Bug Fix - Added placeholder in search bar in apps section Oct 17, 2023
@kilbha kilbha changed the title Fix: Bug Fix - Added placeholder in search bar in apps section fix: Bug fixed - Added placeholder in search bar in apps section Oct 17, 2023
@PeerRich
Copy link
Member

@PeerRich PeerRich removed the ❗️ .env changes contains changes to env variables label Oct 17, 2023
@PeerRich
Copy link
Member

not needed imo

@PeerRich PeerRich closed this Oct 17, 2023
@kilbha
Copy link
Author

kilbha commented Oct 17, 2023

Thanks PeerRich

@@ -54,6 +54,7 @@ function AppsSearch({
type="search"
autoComplete="false"
onChange={onChange}
placeholder="Search apps"
Copy link
Member

Choose a reason for hiding this comment

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

missing i18n, use t("search_apps")

@PeerRich
Copy link
Member

added correctly here #11944

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Placeholder in search field in "apps" section
3 participants