Skip to content

Docs - LiveCode in table cell - block interaction #3610

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

Merged
merged 6 commits into from
Mar 16, 2025

Conversation

Inbal-Tish
Copy link
Collaborator

Description

Docs - LiveCode in table cell - block interaction

Changelog

Docs - LiveCode in table cell - block interaction

Additional info

return (
<div className={`${styles.componentItemContainer} ${!showCode ? styles.componentSpotlightStyle : ''}`}>
{content}
{interactionBlocker}
Copy link
Collaborator

@nitzanyiz nitzanyiz Mar 13, 2025

Choose a reason for hiding this comment

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

Because this div is over the content, when the content is the codePreview, the user can't select the code or copy it. It blocks the interactions with it. Perhaps we can just wrap the componentPreview with a div that has pointerEvents: none. Wdyt?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I tried that but it didn't work, now it is so probably didn't implemented well. Anyways, changed.
Good catch!

@nitzanyiz nitzanyiz assigned Inbal-Tish and unassigned nitzanyiz Mar 13, 2025
@Inbal-Tish Inbal-Tish assigned nitzanyiz and unassigned Inbal-Tish Mar 13, 2025
@Inbal-Tish Inbal-Tish requested a review from nitzanyiz March 13, 2025 10:43
<LiveProvider code={code} scope={ReactLiveScope}>
<LivePreview/>
</LiveProvider>
<div style={{pointerEvents: 'none'}}>
Copy link
Collaborator

Choose a reason for hiding this comment

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

You could add this as a class in the styles file. Wdyt?

@nitzanyiz nitzanyiz assigned Inbal-Tish and unassigned nitzanyiz Mar 13, 2025
@Inbal-Tish Inbal-Tish assigned nitzanyiz and unassigned Inbal-Tish Mar 13, 2025
@Inbal-Tish Inbal-Tish requested a review from nitzanyiz March 13, 2025 10:49
@nitzanyiz nitzanyiz merged commit 4158f3f into master Mar 16, 2025
1 check passed
@nitzanyiz nitzanyiz deleted the docs/fix_line_interaction branch March 16, 2025 07:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants