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

Unstable - Long text in the bread-crumb navigation #4653

Closed
pcenov opened this issue Aug 16, 2024 · 6 comments · Fixed by #4677
Closed

Unstable - Long text in the bread-crumb navigation #4653

pcenov opened this issue Aug 16, 2024 · 6 comments · Fixed by #4677

Comments

@pcenov
Copy link
Member

pcenov commented Aug 16, 2024

Observed behavior

See the video - longer text is overlapping the menu items on top of the table.

Expected behavior

To be further discussed

Steps to reproduce the issue

  1. Go to https://unstable.studio.learningequality.org/en/accounts/#/ and sign in
  2. Open a channel with folders and all of the supported resources
  3. Navigate to a folder with a long title
  4. Start shrinking the horizontal size of the page

Additional information

responive-2.mp4

Console logs

no errors observed in the console

Usage Details

@pcenov
Copy link
Member Author

pcenov commented Aug 16, 2024

@radinamatic

@akolson
Copy link
Member

akolson commented Aug 19, 2024

Tagging @jtamiace for thoughts on this. cc @marcellamaki

@jtamiace
Copy link
Member

Is it possible for the title to remain on one line and begin truncation to avoid breaking to that second line and overlapping the other content?

@akolson
Copy link
Member

akolson commented Aug 21, 2024

Yes this should be possible!

@akolson
Copy link
Member

akolson commented Aug 24, 2024

Noting that this is actually a regression from the current version of studio caused by the Icon refactor that happened. Bumping this to critical.

@akolson akolson added P0 - critical Priority: Release blocker or regression and removed P1 - important Priority: High impact on UX labels Aug 24, 2024
@radinamatic
Copy link
Member

Manual QA on the unstable branch confirms the fix.

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

Successfully merging a pull request may close this issue.

4 participants