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

Adds ... if the current message is not completely displayed on the screen #6

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Adds ... if the current message is not completely displayed on the screen #6

wants to merge 1 commit into from

Conversation

accognet
Copy link
Contributor

Like that :
image

@piRGoif
Copy link
Contributor

piRGoif commented Sep 24, 2024

Could text-overflow: ellipsis be a simplier solution ? (ref : https://developer.mozilla.org/fr/docs/Web/CSS/text-overflow)

@accognet
Copy link
Contributor Author

No, it does not work with multi-line text

@piRGoif
Copy link
Contributor

piRGoif commented Sep 24, 2024 •
edited
Loading

Could text-overflow: ellipsis be a simplier solution

No, it does not work with multi-line text

Oh ok
Seems indeed there are workarounds, one is simple but only works on Webkit :(

@accognet accognet self-assigned this Jan 27, 2025
@rquetiez rquetiez assigned steffunky and unassigned steffunky Jan 28, 2025
bdalsass
bdalsass approved these changes Jan 29, 2025
View reviewed changes
Copy link
Contributor

@bdalsass bdalsass left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe a more comprehensive indication, like" ...click to read more"
Not sure data-communication-id is used ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bdalsass bdalsass bdalsass approved these changes

@steffunky steffunky Awaiting requested review from steffunky

Assignees

@accognet accognet

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@accognet @piRGoif @bdalsass @steffunky