Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add a tooltip to show full "Replying to @..." text #457

Merged
merged 8 commits into from
Apr 19, 2023
Merged

Add a tooltip to show full "Replying to @..." text #457

merged 8 commits into from
Apr 19, 2023

Conversation

tautautautautau
Copy link
Contributor

A tooltip to view the entire message being replied to, eliminating the need to open the thread to read it.

@Melonify
Copy link
Contributor

Melonify commented Apr 5, 2023

This looks fine, but it could be drastically simplified using the v-tooltip custom directive.

Just setting v-tooltip on any element will setup a tooltip exactly like the one you've created manually

Copy link
Contributor

@AnatoleAM AnatoleAM left a comment

Choose a reason for hiding this comment

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

Not sure I like how it appears in the middle 🤔 looks a bit goofy when in popout mode

@tautautautautau
Copy link
Contributor Author

@AnatoleAM This PR is ready. Could this be merged since it is a QoL that some users have asked for?

@AnatoleAM AnatoleAM enabled auto-merge (squash) April 19, 2023 19:19
@AnatoleAM AnatoleAM merged commit 07a707e into SevenTV:master Apr 19, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants