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

Truncate long breadcrumbs #2787

Closed
ggeisler opened this issue Nov 19, 2021 · 1 comment · Fixed by #2796
Closed

Truncate long breadcrumbs #2787

ggeisler opened this issue Nov 19, 2021 · 1 comment · Fixed by #2796
Labels

Comments

@ggeisler
Copy link
Contributor

It appears that our strategy to truncate long breadcrumbs regressed at some point. The examples below are from DLME but I see similar behavior in Stanford exhibits so I'm assuming it's a core Spotlight issue.

It appears that we fixed this (or thought we did) last January in this PR.

Here's a sample record with a long title.

And this is the record used in PR #2593 to show that it was apparently working then.

Current behavior

Screen Shot 2021-11-19 at 10 21 57 AM


Screen Shot 2021-11-19 at 10 46 40 AM

Desired behavior

Screen Shot 2021-11-19 at 10 49 35 AM


Screen Shot 2021-11-19 at 10 48 50 AM

@ggeisler ggeisler added the bug label Nov 19, 2021
@cbeer
Copy link
Member

cbeer commented Nov 19, 2021

I'd guess this regressed with twbs/bootstrap#31700?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants