Skip to content

fix(event-types): align URL prefix with input text baseline#29000

Merged
romitg2 merged 2 commits intocalcom:mainfrom
Akash504-ai:fix/url-prefix-alignment
Apr 26, 2026
Merged

fix(event-types): align URL prefix with input text baseline#29000
romitg2 merged 2 commits intocalcom:mainfrom
Akash504-ai:fix/url-prefix-alignment

Conversation

@Akash504-ai
Copy link
Copy Markdown
Contributor

@Akash504-ai Akash504-ai commented Apr 25, 2026

What does this PR do?

Fixes a minor visual misalignment in the Event Type Setup tab URL input.

The URL prefix (e.g. localhost:3000/username/) appeared slightly higher than the input text due to baseline differences between the prefix element and the input field.

Solution

  • Updated the prefix container to use flex alignment for proper vertical centering
  • Applied a minimal 1px vertical offset to match the input text baseline

This results in consistent and visually balanced alignment between the prefix and the input value.

Visual Demo

Before

Before

After

After

Mandatory Tasks

  • I have self-reviewed the code
  • I have updated the developer docs if required (N/A)
  • I confirm automated tests are in place (N/A - visual fix)

How should this be tested?

  1. Navigate to /event-types/:id?tabName=setup
  2. Locate the URL input field in the Basics section
  3. Compare alignment between:
    • URL prefix (e.g. localhost:3000/username/)
    • Input value (e.g. abc)

Expected Result

  • Prefix and input text should be vertically aligned with no visual offset

Checklist

  • My code follows the style guidelines of this project
  • I have commented my code where necessary(N/A)
  • My changes generate no new warnings
  • My PR is small and focused

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 25, 2026

Welcome to Cal.diy, @Akash504-ai! Thanks for opening this pull request.

A few things to keep in mind:

  • This is Cal.diy, not Cal.com. Cal.diy is a community-driven, fully open-source fork of Cal.com licensed under MIT. Your changes here will be part of Cal.diy — they will not be deployed to the Cal.com production app.
  • Please review our Contributing Guidelines if you haven't already.
  • Make sure your PR title follows the Conventional Commits format.

A maintainer will review your PR soon. Thanks for contributing!

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 25, 2026 •
edited
Loading

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 6aa03149-3c90-48ee-a116-2622e3594753

📥 Commits

Reviewing files that changed from the base of the PR and between 07edb53 and 9255e52.

📒 Files selected for processing (1)
  • apps/web/modules/event-types/components/tabs/setup/EventSetupTab.tsx
📝 Walkthrough

Walkthrough

This pull request modifies the styling of the URL/slug prefix wrapper element in the event setup tab component. The change replaces an inline-block display approach with a flex-based layout system. The update also adds explicit height and typography styling along with a small vertical offset adjustment. These modifications affect only the visual presentation and positioning of the prefix element with no changes to component logic or data handling.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately and concisely describes the main change: fixing the alignment of the URL prefix with the input text baseline, which is the primary visual improvement in this PR.
Description check ✅ Passed The description is detailed and directly related to the changeset, explaining the visual misalignment issue, the solution implemented, and providing testing instructions and before/after visual demos.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@romitg2 romitg2 added ready-for-e2e run-ci Approve CI to run for external contributors labels Apr 26, 2026
@romitg2 romitg2 enabled auto-merge (squash) April 26, 2026 11:31
@romitg2 romitg2 merged commit a69c176 into calcom:main Apr 26, 2026
73 of 77 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@romitg2 romitg2 romitg2 approved these changes

Assignees

No one assigned

Labels

ready-for-e2e run-ci Approve CI to run for external contributors size/XS

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Akash504-ai @romitg2