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.

Sign up for GitHub

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

Jump to bottom

fix(ui): step-nav breadcrumbs ellipsis #6344

Merged
merged 3 commits into from
May 14, 2024
Merged

fix(ui): step-nav breadcrumbs ellipsis #6344

merged 3 commits into from
May 14, 2024

Conversation

PatrikKozak
Copy link
Contributor

Description

Fixes this issue in 3.0 demo repo.

Long titles in the breadcrumbs were not properly applying the text-overflow: ellipsis style.

Before:
Screenshot 2024-05-13 at 3 18 58 PM

After:
Screenshot 2024-05-13 at 3 18 25 PM

  • I have read and understand the CONTRIBUTING.md document in this repository.

Type of change

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

Checklist:

  • Existing test suite passes locally with my changes

@PatrikKozak PatrikKozak linked an issue May 13, 2024 that may be closed by this pull request
Long titles are cutoff in the breadcrumb navigation payloadcms/payload-3.0-demo#212
Closed
@PatrikKozak PatrikKozak merged commit 353c2b0 into beta May 14, 2024
34 checks passed
@PatrikKozak PatrikKozak deleted the fix/beta/stepNav-breadcrumbs-ellipsis branch May 14, 2024 15:08
paulpopus pushed a commit that referenced this pull request May 14, 2024
@PatrikKozak @paulpopus
fix(ui): step-nav breadcrumbs ellipsis (#6344)
d0f7fd4
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jacobsfletch jacobsfletch jacobsfletch approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Long titles are cutoff in the breadcrumb navigation
2 participants
@PatrikKozak @jacobsfletch