Skip to content

lasso #730

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
Closed
wants to merge 14 commits into from
Closed

lasso #730

wants to merge 14 commits into from

Conversation

@Fil
Copy link
Contributor

@Fil Fil commented Jan 31, 2022 •
edited
Loading

@Fil Fil marked this pull request as ready for review January 31, 2022 16:40
@Fil Fil requested a review from mbostock January 31, 2022 16:40
@mbostock
Copy link
Member

mbostock commented Jan 31, 2022

Fun! Let’s continue this discussion: #721 (comment)

mbostock
mbostock reviewed Jan 31, 2022
View reviewed changes
@Fil
Copy link
Contributor Author

Fil commented Jan 31, 2022

There is an issue sometimes where the pointer is not released appropriately.

@Fil Fil changed the base branch from main to mbostock/pointer February 2, 2022 15:15
@Fil
Copy link
Contributor Author

Fil commented Jul 18, 2022

Closing due to inactivity.

@Fil Fil closed this Jul 18, 2022
@Fil Fil mentioned this pull request May 29, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mbostock mbostock mbostock left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Fil @mbostock