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.

Sign up for GitHub

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

Jump to bottom

Node created while another node is selected is not placed under. #9689

Closed
farmaazon opened this issue Apr 12, 2024 · 0 comments · Fixed by #9696
Closed

Node created while another node is selected is not placed under. #9689

farmaazon opened this issue Apr 12, 2024 · 0 comments · Fixed by #9696
Assignees
@vitvakatu
Labels
--bug Type: bug --regression Important: regression -gui d-easy Difficulty: little prior knowledge required p-high Should be completed in the next sprint
Milestone
Beta Release

Comments

@farmaazon
Copy link
Contributor

Screencast.from.2024-04-12.16-27-19.webm
@farmaazon farmaazon added d-easy Difficulty: little prior knowledge required p-high Should be completed in the next sprint --bug Type: bug --regression Important: regression -gui labels Apr 12, 2024
@vitvakatu vitvakatu self-assigned this Apr 12, 2024
@vitvakatu vitvakatu mentioned this issue Apr 15, 2024
Merged
5 tasks
@mergify mergify bot closed this as completed in #9696 Apr 15, 2024
mergify bot pushed a commit that referenced this issue Apr 15, 2024
@vitvakatu
Fix node placement (#9696)
59a9c8e 
Fixes #9689

We should consider visualization size in all cases. `visibleArea` has multiple usages, but they all seem to benefit from the change (e.g. zoom to selected will also consider visualization sizes).

https://github.com/enso-org/enso/assets/6566674/3c9ae4a9-27f5-4736-83b5-4b3a500564e7
@AdRiley AdRiley added this to the Beta Release milestone Apr 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@vitvakatu vitvakatu

Labels
--bug Type: bug --regression Important: regression -gui d-easy Difficulty: little prior knowledge required p-high Should be completed in the next sprint
Projects
Issues Board
Status: 🗄️ Archived
Milestone
Beta Release
Development

Successfully merging a pull request may close this issue.

Fix node placement enso-org/enso
3 participants
@AdRiley @farmaazon @vitvakatu