Skip to content

Conversation

@SmartManoj
Copy link
Contributor

@SmartManoj SmartManoj commented Mar 30, 2025

Resolves #55

It ignores the background color when pasting.

  • Vibe coded.

@neoneye
Copy link
Owner

neoneye commented Mar 31, 2025

Nice. I just tried out your branch. And the way the entire object gets selected, that feels like a good improvement.

However the selection of the entire object, often selects the top-left area of the object. And there is a TODO in the code.
Screenshot 2025-03-31 at 17 09 23

@SmartManoj
Copy link
Contributor Author

SmartManoj commented Mar 31, 2025

Would you share the task ID?


Task link

Here, the error logic didn't apply.

image

@neoneye
Copy link
Owner

neoneye commented Mar 31, 2025

@neoneye
Copy link
Owner

neoneye commented Mar 31, 2025

How about voice talk on ARC Prize Discord determine what to do about this PR?

@SmartManoj
Copy link
Contributor Author

SmartManoj commented Apr 1, 2025

image

The error is not in expandSelectionToObject.

But in updateDrawCanvas?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Editor: Sparse selection, for manipulating objects

2 participants