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

Docs - add content component tooltip #3458

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

Inbal-Tish
Copy link
Collaborator

Description

Docs - add content component tooltip

Changelog

Docs - add content component tooltip

Additional info

https://wix.atlassian.net/browse/MADS-4558

@Inbal-Tish Inbal-Tish requested a review from ethanshar December 12, 2024 07:59
@Inbal-Tish Inbal-Tish added this to the Docs milestone Dec 12, 2024
@ethanshar
Copy link
Collaborator

Screen.Recording.2024-12-24.at.16.51.36.mov

The experience is not great IMO.
The tooltip doesn't show the whole code but also I think that showing it when hovering the example is a little annoying.

Let's instead add a button at the right-top of the cell that user can click on. It will show the tooltip and maybe clicking on the tooltip will just copy the code

@Inbal-Tish Inbal-Tish assigned ethanshar and unassigned Inbal-Tish Dec 31, 2024
@Inbal-Tish
Copy link
Collaborator Author

@ethanshar I replaced the hover with a button. Copy was already implemented on a tooltip click

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

Successfully merging this pull request may close these issues.

2 participants