Skip to content

Stop long urls from overflowing on booking page.#5240

Merged
pumfleet merged 1 commit intomainfrom
fix/stop-long-urls-from-overflowing
Oct 27, 2022
Merged

Stop long urls from overflowing on booking page.#5240
pumfleet merged 1 commit intomainfrom
fix/stop-long-urls-from-overflowing

Conversation

@JeroenReumkens
Copy link
Copy Markdown
Contributor

What does this PR do?

Prevents long urls (or other very long texts for that matter) from overflowing on the public booking page. It will only break long words if they're longer than one sentence.

It's not the most beautiful thing, but for edge cases this works and is way better than the overflowing. Probably down the line we can add the HTML editor @CarinaWolli is working on here as well, and give people to option to add real anchors?

Before

CleanShot 2022-10-27 at 09 49 34@2x

After

CleanShot 2022-10-27 at 09 49 04@2x

Environment: Staging(main branch)

Type of change

  • Bug fix (non-breaking change which fixes an issue)

How should this be tested?

  • Create an event with a long discription with either a very long word or url in there (without dashes, . etc, since it will break there too), and notice that it now breaks it over multiple lines.

@vercel
Copy link
Copy Markdown

vercel Bot commented Oct 27, 2022

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

Name Status Preview Updated
cal ✅ Ready (Inspect) Visit Preview Oct 27, 2022 at 7:54AM (UTC)

@harshsinghatz
Copy link
Copy Markdown
Contributor

Tested on mobile viewport?

@JeroenReumkens
Copy link
Copy Markdown
Contributor Author

@harshsinghatz Jep!
On wider mobiles it will even show more of it:

CleanShot 2022-10-27 at 09 57 44@2x

On smaller screens it overflows the same:
CleanShot 2022-10-27 at 09 58 31@2x

@pumfleet pumfleet merged commit f009321 into main Oct 27, 2022
@pumfleet pumfleet deleted the fix/stop-long-urls-from-overflowing branch October 27, 2022 08:04
haffla pushed a commit to tourlane/cal.com that referenced this pull request Nov 22, 2022
haffla pushed a commit to tourlane/cal.com that referenced this pull request Nov 22, 2022
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.

3 participants