Skip to content

fix: event types search placeholder moving on focus#12

Merged
dhairyashiil merged 2 commits into
calcom:mainfrom
mrpaaradox:fix-event-types-search-placeholder-moving
Mar 24, 2026
Merged

fix: event types search placeholder moving on focus#12
dhairyashiil merged 2 commits into
calcom:mainfrom
mrpaaradox:fix-event-types-search-placeholder-moving

Conversation

@mrpaaradox
Copy link
Copy Markdown
Contributor

Summary

Fixes the event types search placeholder shifting when the input is focused (web/Android), and aligns the search bar styling with the Availability screen. The placeholder no longer jumps on tap, and the search header uses the same gray bar and white/dark input as Availability.

Event Types — Search bar (Android)

Placeholder: Resolved moving/jumping placeholder by matching the search bar implementation to Availability (simplified TextInput styling, no extra padding/height/lineHeight that caused layout shift).

Styling: Search header bar uses gray background in light mode (#f3f4f6) and black in dark mode (#000000); search input uses white (#FFFFFF) in light mode and dark gray (#171717) in dark mode, consistent with the “Search schedules” bar on Availability.

Before & After

Before:

Screen.Recording.2026-02-22.at.9.08.41.AM.mov

After:

Screen.Recording.2026-02-22.at.9.15.06.AM.mov

Review & Testing Checklist

  • Event Types (web or Android): In dark mode, open Event Types and tap the search field — placeholder “Search event types” should not move.
  • Event Types (web or Android): In light mode, confirm the search bar has a gray header strip and white input, matching Availability.
  • Light and dark: Confirm search bar and placeholder look correct in both themes with no regressions.

Files touched

  • app/(tabs)/(event-types)/index.tsx

@github-actions github-actions Bot added the mobile Changes to the mobile app (ios/android) label Feb 22, 2026
Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

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

No issues found across 1 file

@mrpaaradox
Copy link
Copy Markdown
Contributor Author

Hey @dhairyashiil, could you take a look at all the PR's I have made ?

@mrpaaradox
Copy link
Copy Markdown
Contributor Author

Hey @volnei, looks like Dhairyashil is busy somewhere. Can you review the PRs I made?

@mrpaaradox mrpaaradox marked this pull request as draft March 14, 2026 13:55
@dhairyashiil dhairyashiil marked this pull request as ready for review March 24, 2026 07:42
@vercel
Copy link
Copy Markdown

vercel Bot commented Mar 24, 2026

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

A member of the Team first needs to authorize it.

Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

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

No issues found across 1 file

@dhairyashiil dhairyashiil merged commit ca90665 into calcom:main Mar 24, 2026
9 of 11 checks passed
devin-ai-integration Bot pushed a commit that referenced this pull request May 23, 2026
- Reject /notify in Telegram groups with DM hint (#3)
- Validate payload fields (title, timeZone, start, end, hosts, attendees) in parseDeliverRequest (#4)
- Wrap deliverNotifications in try/catch, return 422 on formatter crash (#4)
- Add logging across route, service, and delivery modules (#5)
- Add /notify to Slack and Telegram help cards (#7)
- Add error field to DeliverResult for backend retry/unsubscribe decisions (#9)
- Change confirmation message from 'here' to 'via DM' (#10)
- Validate timeZone with Intl.DateTimeFormat before processing (#11)
- Add fallback badge for unknown notificationType (#12)
- Reject /notify in groups with 'check DMs' message (#13)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mobile Changes to the mobile app (ios/android)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants