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

fix: whiteboard zoom on mweb #2424

Closed
wants to merge 1 commit into from
Closed

fix: whiteboard zoom on mweb #2424

wants to merge 1 commit into from

Conversation

eswarclynn
Copy link
Contributor

Details(context, Jira ticket, how was the bug fixed, what does the new feature do)

Choose one of these(put a 'x' in the bracket):

  • The change doesn't require a change to the documentation.
  • The documentation is updated accordingly.

Implementation note, gotchas, related work and Future TODOs (optional)

Copy link

vercel bot commented Jan 16, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
dashboard-app ❌ Failed (Inspect) Jan 16, 2024 5:08am
storybook-app ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 16, 2024 5:08am

Copy link
Contributor

A JIRA Issue ID is missing from your branch name! 🦄

Your branch: fix/whiteboard-mweb

If this is your first time contributing to this repository - welcome!


Please refer to jira-lint to get started.

Without the JIRA Issue ID in your branch name you would lose out on automatic updates to JIRA via SCM; some GitHub status checks might fail.

Valid sample branch names:

‣ feature/shiny-new-feature--mojo-10'
‣ 'chore/changelogUpdate_mojo-123'
‣ 'bugfix/fix-some-strange-bug_GAL-2345'

import { Box } from '../../Layout';
// @ts-ignore: No implicit Any
import { useSetAppDataByKey } from '../components/AppData/useUISettings';
import { APP_DATA } from '../common/constants';

const EmbedComponent = () => {
const { iframeRef } = useWhiteboard();
const isMobile = useMedia(cssConfig.media.md);
Copy link
Collaborator

Choose a reason for hiding this comment

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

we can use this inside the useWhiteboard itself.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

useWhiteboard is in react-sdk and it doesn't have access to config

@eswarclynn eswarclynn closed this Jan 16, 2024
@eswarclynn eswarclynn deleted the fix/whiteboard-mweb branch January 16, 2024 09:32
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