-
Notifications
You must be signed in to change notification settings - Fork 807
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(whiteboard): support multi window #896
feat(whiteboard): support multi window #896
Conversation
47ce598
to
2c7721c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please make it pass the spell check CI
left: 76px; | ||
z-index: 3; | ||
bottom: 8px; | ||
&.is-disabled { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
&.is-disabled { | |
&.invisible { |
border-radius: 2px; | ||
&:hover { | ||
background: rgba(33, 35, 36, 0.1); | ||
&.is-disabled { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
&.is-disabled { | |
&.invisible { |
<div className="redo-undo-box"> | ||
<div | ||
className={classNames("redo-undo-box", { | ||
"is-disabled": whiteboardStore.isWindowMaximization, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"is-disabled": whiteboardStore.isWindowMaximization, | |
"invisible": whiteboardStore.isWindowMaximization, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We use is-
prefix to indicate state classnames.
</div> | ||
<div | ||
className={classNames("page-controller-box", { | ||
"is-disabled": whiteboardStore.isWindowMaximization, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"is-disabled": whiteboardStore.isWindowMaximization, | |
"invisible": whiteboardStore.isWindowMaximization, |
if (this.room && this.windowManager) { | ||
this.preloadPPTResource.cancel(); | ||
this.windowManager.destroy(); | ||
this.room.callbacks.off(); | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if (this.room && this.windowManager) { | |
this.preloadPPTResource.cancel(); | |
this.windowManager.destroy(); | |
this.room.callbacks.off(); | |
} | |
this.preloadPPTResource.cancel(); | |
this.windowManager?.destroy(); | |
this.room?.callbacks.off(); |
private scenesPathName = (scenePath: string): string => { | ||
const cells = scenePath.split("/"); | ||
const popCell = cells.pop(); | ||
if (popCell === "") { | ||
cells.pop(); | ||
} | ||
return cells.join("/"); | ||
}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What does this mean? Can you give me an example of what the value of scenePath
looks like?
left: 76px; | ||
z-index: 3; | ||
bottom: 8px; | ||
&.is-disabled { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same problem as above
2c7721c
to
5563f32
Compare
5563f32
to
8c6921b
Compare
* feat(whiteboard): support multi window (#896) * refactor(desktop): set aspect ratio to 16/9 (#899) * feat(whiteboard): add containerSizeRatio config in windowManager (#900) * chore(ci): add dev branch to check action (#901) * fix(whiteboard): adjust location of collectorContainer element (#903) * refactor(whiteboard): remove vision control (#907) * refactor(whiteboard): remove Vision control * fix(ci): build web and renderer failed * chore(library): upgrade window-manager (#919) * fix(web): should close local tracks on exiting room (#920) * refactor(whiteboard): adjust class room style of whiteboard (#921) * refactor(whiteboard): adjust class room style of whiteboard * fix(type): type fail Co-authored-by: Cheerego7 <48879533+Cheerego7@users.noreply.github.com> Co-authored-by: Leo <1826234293@qq.com> Co-authored-by: hyrious <hyrious@outlook.com>
No description provided.