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: updated node toolbar to be more stable on zoom, fix success and error component tooltips, unselect nodes on flow opening #4610

Merged
merged 23 commits into from
Nov 19, 2024

Conversation

lucaseduoli
Copy link
Collaborator

This pull request includes several changes to enhance the user interface, refactor code, and improve the functionality of the NodeToolbarComponent and related components. The most important changes include modifications to the scrollbar styles, refactoring of the NodeToolbarComponent, and updates to the NodeStatus component.

User Interface Enhancements:

  • src/frontend/src/App.css: Updated scrollbar styles, including changing the background color of the scrollbar thumb and hover states.

Code Refactoring:

Component Updates:

Utility and Store Enhancements:

Minor Fixes:

@lucaseduoli lucaseduoli self-assigned this Nov 14, 2024
@lucaseduoli lucaseduoli marked this pull request as ready for review November 14, 2024 17:13
@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Nov 14, 2024
@dosubot dosubot bot added the javascript Pull requests that update Javascript code label Nov 14, 2024
@github-actions github-actions bot added the bug Something isn't working label Nov 14, 2024
@dosubot dosubot bot added size:XL This PR changes 500-999 lines, ignoring generated files. and removed size:L This PR changes 100-499 lines, ignoring generated files. labels Nov 14, 2024
Copy link
Contributor

@anovazzi1 anovazzi1 left a comment

Choose a reason for hiding this comment

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

lgtm

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Nov 14, 2024
@ogabrielluiz ogabrielluiz enabled auto-merge (squash) November 18, 2024 20:14
@ogabrielluiz
Copy link
Contributor

This test passes on all other branches so I can't merge this until it is fixed @lucaseduoli

@anovazzi1 anovazzi1 enabled auto-merge (squash) November 19, 2024 21:41
@ogabrielluiz ogabrielluiz merged commit c9e0d5e into main Nov 19, 2024
18 checks passed
@ogabrielluiz ogabrielluiz deleted the fix/general_bugs_release branch November 19, 2024 21:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working javascript Pull requests that update Javascript code lgtm This PR has been approved by a maintainer size:XL This PR changes 500-999 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants