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

Overflowing of selected facet badge on search page #2674

Closed
LotteHofstede opened this issue Nov 30, 2023 · 0 comments · Fixed by #2675
Closed

Overflowing of selected facet badge on search page #2674

LotteHofstede opened this issue Nov 30, 2023 · 0 comments · Fixed by #2675
Assignees
Labels
bug component: Discovery related to discovery search or browse system ux User Experience related works
Milestone

Comments

@LotteHofstede
Copy link
Contributor

Describe the bug
When a long facet is selected, the text will overflow its container.

image

To Reproduce
Example of the issue on demo:
https://demo.dspace.org/search?query=test&spc.page=1&f.subject=Lorem%20ipsum%20dolor%20sit%20amet.%20Et%20iusto%20omnis%20qui%20reprehenderit%20illo%20ut%20provident%20cumque%20ab%20perspiciatis%20delectus.%20Eos%20nemo%20mollitia%20qui%20omnis%20nihil%20ut%20galisum%20obcaecati.,equals

Expected behavior
The badge should not overflow its container but wrap around to a new line.

@LotteHofstede LotteHofstede added bug needs triage New issue needs triage and/or scheduling labels Nov 30, 2023
@LotteHofstede LotteHofstede self-assigned this Nov 30, 2023
@tdonohue tdonohue added component: Discovery related to discovery search or browse system and removed needs triage New issue needs triage and/or scheduling labels Apr 29, 2024
@tdonohue tdonohue added this to the 7.6.2 milestone Apr 29, 2024
@tdonohue tdonohue added the ux User Experience related works label Apr 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug component: Discovery related to discovery search or browse system ux User Experience related works
Projects
Development

Successfully merging a pull request may close this issue.

2 participants