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

Break title in max three lines with expandable on hover #3654

Merged
merged 3 commits into from
Dec 4, 2023

Conversation

sandeepsajan0
Copy link
Member

@sandeepsajan0 sandeepsajan0 commented Nov 10, 2023

Fixes #3631

Right now, big titles look like this on the applicant dashboard:
image

It looks like this after the fix:
On large screen/laptop:
image
on hover(above screen):
image

For smaller screen like tablet/ipad mini:
image

@sandeepsajan0 sandeepsajan0 marked this pull request as ready for review November 12, 2023 10:05
@theskumar
Copy link
Member

Screenshot 2023-11-15 at 7  20 45@2x

@sandeepsajan0 sandeepsajan0 force-pushed the enhancement/gh-3631-title-ui branch 2 times, most recently from 79ef9ce to 2ed2265 Compare November 22, 2023 11:56
@theskumar
Copy link
Member

@sandeepsajan0 Are you able to resolve the conflict?
Rest looks fine.

@sandeepsajan0
Copy link
Member Author

@frjo It is ready for testing.

@frjo frjo added Type: Enhancement This is an improvement of an existing thing (not a new thing, which would be a feature). Type: Patch Mini change, used in release drafter Status: Needs testing Tickets that need testing/qa labels Nov 28, 2023
@sandeepsajan0 sandeepsajan0 changed the title Break title in max two lines with expandable on hover Break title in max three lines with expandable on hover Nov 30, 2023
@wes-otf
Copy link
Contributor

wes-otf commented Dec 1, 2023

In testing it looked like the mobile alignment was still a little off as mentioned by @theskumar. Other than that everything worked as intended and was ready for live!

Screen Shot 2023-12-01 at 10 40 54

@sandeepsajan0
Copy link
Member Author

@wes-otf Thanks for testing it. This PR is more for restricting the titles to max 3 lines so that long titles don't occupy all the space. We have another issue(#3660) for the statuses in mobile size problems.

@wes-otf
Copy link
Contributor

wes-otf commented Dec 2, 2023

Sounds good! This is ready to go then.

@wes-otf wes-otf added Status: Tested - approved for live ✅ and removed Status: Needs testing Tickets that need testing/qa labels Dec 2, 2023
@frjo frjo merged commit fa34eac into main Dec 4, 2023
10 checks passed
@frjo frjo deleted the enhancement/gh-3631-title-ui branch February 12, 2024 11:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Tested - approved for live ✅ Type: Enhancement This is an improvement of an existing thing (not a new thing, which would be a feature). Type: Patch Mini change, used in release drafter
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Dashboard titles and status bar UI
4 participants