Skip to content

Conversation

@sixsage
Copy link
Contributor

@sixsage sixsage commented Sep 24, 2024

Purpose

Address #2810 #2809 with a different solution from #2834
With the solution in #2834, it is hard to use the workspace-editor components elsewhere as it is based on vh/vw styling.

Changes

Revoke the styling changes made in #2834 (back to using 100% instead of vh/vw and remove overflow: hidden)
Dispatch "resize" event when workflow editor is loaded to invoke all the relevant resize handlers.

Demo

Demo is the same as #2834 as they fix the same bug
Before (before #2834)
workflow_editor_size_bug_master
After
workflow_editor_size_bug_branch

@sixsage sixsage self-assigned this Sep 24, 2024
@sixsage sixsage marked this pull request as ready for review September 25, 2024 07:07
Copy link
Contributor

@aglinxinyuan aglinxinyuan left a comment

Choose a reason for hiding this comment

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

LGTM!

@sixsage sixsage merged commit 3b39085 into master Sep 25, 2024
@sixsage sixsage deleted the fix-workspace-editor-sizing-bug branch September 25, 2024 07:49
PurelyBlank pushed a commit that referenced this pull request Dec 4, 2024
…2867)

## Purpose
Address #2810 #2809 with a different solution from #2834
With the solution in #2834, it is hard to use the workspace-editor
components elsewhere as it is based on vh/vw styling.
## Changes
Revoke the styling changes made in #2834 (back to using 100% instead of
vh/vw and remove overflow: hidden)
Dispatch "resize" event when workflow editor is loaded to invoke all the
relevant resize handlers.

## Demo
Demo is the same as #2834 as they fix the same bug
Before (before #2834)

![workflow_editor_size_bug_master](https://github.com/user-attachments/assets/a3e5a50b-00ac-4c20-9fdf-afb256e0947c)
After

![workflow_editor_size_bug_branch](https://github.com/user-attachments/assets/7c51624d-bce5-4e6f-adb7-ce4ec54e3137)
@Yicong-Huang Yicong-Huang added fix and removed bug labels Oct 4, 2025
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.

3 participants