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

feat(composer): Implement data overlay container #595

Merged
merged 1 commit into from
Feb 22, 2023

Conversation

sheilaXu
Copy link
Contributor

Overview

Implement the container of data overlay component with simple text content rendering

overlay-container

Verifying Changes

Scene Composer

For scene-composer package changes specifically, you can preview the component in the published storybook artifact. To do this, wait for the Publish Storybook action to complete below.

  • Click on the workflow details
  • Select the Summary item on the left
  • Download the zip file

To run the storybook build locally, you need a local static web server:

npm install -g httpserver
cd <Extracted Zip Directory>
httpserver

Then open the website http://localhost:8080 to run the doc site.

Legal

This project is available under the Apache 2.0 License.

@sheilaXu sheilaXu force-pushed the data-overlay-m1-container branch from 5a476a6 to 82d0ebb Compare February 17, 2023 01:06
@sheilaXu sheilaXu marked this pull request as ready for review February 17, 2023 01:38
@sheilaXu sheilaXu requested a review from MO-Elmu February 17, 2023 19:28
@@ -16,7 +16,7 @@ const useCallbackWhenNotPanning = (callback, deps, acceptableDriftDistance = 1)
const lastPointerDownLocation = useRef<[number, number] | null>(null);

const onPointerUp = useCallback(
(e: ThreeEvent<MouseEvent>) => {
(e: ThreeEvent<MouseEvent> | PointerEvent) => {
Copy link
Contributor

Choose a reason for hiding this comment

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

Not sure if it is necessary right now but should we use ThreeEvent<PointerEvent> vs ThreeEvent<MouseEvent>? Not your change just thinking out loud

@sheilaXu sheilaXu force-pushed the data-overlay-m1-container branch from 82d0ebb to fe72d13 Compare February 20, 2023 23:07
@sheilaXu sheilaXu force-pushed the data-overlay-m1-container branch from fe72d13 to b35830f Compare February 22, 2023 18:31
@sheilaXu sheilaXu merged this pull request into data-overlay-m1 Feb 22, 2023
@sheilaXu sheilaXu deleted the data-overlay-m1-container branch February 22, 2023 19:12
sheilaXu added a commit that referenced this pull request Mar 27, 2023
feat(composer): Implement data overlay container (#595)

feat(composer): data overlay editor & render markdown

feat(composer): add overlay panel config editor

feat(composer): data binding variable support for data overlay

feat(composer): implement add overlay from menu
sheilaXu added a commit that referenced this pull request Mar 27, 2023
feat(composer): Implement data overlay container (#595)

feat(composer): data overlay editor & render markdown

feat(composer): add overlay panel config editor

feat(composer): data binding variable support for data overlay

feat(composer): implement add overlay from menu
@github-actions github-actions bot mentioned this pull request Mar 27, 2023
sheilaXu added a commit that referenced this pull request Mar 27, 2023
feat(composer): Implement data overlay container (#595)

feat(composer): data overlay editor & render markdown

feat(composer): add overlay panel config editor

feat(composer): data binding variable support for data overlay

feat(composer): implement add overlay from menu
sheilaXu added a commit that referenced this pull request Mar 27, 2023
feat(composer): Implement data overlay container (#595)

feat(composer): data overlay editor & render markdown

feat(composer): add overlay panel config editor

feat(composer): data binding variable support for data overlay

feat(composer): implement add overlay from menu
@github-actions github-actions bot mentioned this pull request Mar 27, 2023
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.

3 participants