-
Notifications
You must be signed in to change notification settings - Fork 13k
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(editor): Make the left sidebar in Expressions editor draggable #11838
feat(editor): Make the left sidebar in Expressions editor draggable #11838
Conversation
7a9fb33
to
5b076a5
Compare
Codecov ReportAttention: Patch coverage is 📢 Thoughts on this report? Let us know! |
13e424c
to
0ab7782
Compare
e8c7903
to
673fe92
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.
LGTM 🎉
|
n8n
|
Project |
n8n
|
Branch Review |
node-1924-expression-modal-allow-changing-width-of-sections
|
Run status |
|
Run duration | 04m 40s |
Commit |
|
Committer | Dana Lee |
View all properties for this run ↗︎ |
Test results | |
---|---|
|
0
|
|
3
|
|
0
|
|
0
|
|
478
|
View all changes introduced in this branch ↗︎ |
|
This will prevent long import statements in later commits.
This encourages more type safety.
This makes it more uniformed with the Expressions editor. In addition, throttle is the more accurate since we want resize to be called during a certain period rather than delaying the call to resize. This also allows us to save calls to resizing.
This will make the draggable cursor more visible when it is in the middle of two Expression Editor panels.
673fe92
to
3d8a14b
Compare
✅ All Cypress E2E specs passed |
Got released with |
Summary
This PR introduces a feature request in the Expressions editor to allow for the left sidebar to be resized for better readability
Related Linear tickets, Github issues, and Community forum posts
Review / Merge checklist
release/backport
(if the PR is an urgent fix that needs to be backported)