Skip to content

Implement viewport selection #178

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

Merged
merged 14 commits into from
Jul 4, 2021
Merged

Implement viewport selection #178

merged 14 commits into from
Jul 4, 2021

Conversation

pkupper
Copy link
Contributor

@pkupper pkupper commented Jun 10, 2021

Closes #140 and closes #145

  • Ellipse layer intersection
  • Line layer intersection
  • Polyline layer intersection
  • Rectangle layer intersection
  • Shape layer intersection
  • Select tool drag box
  • Select tool point selection
  • Select tool box selection
  • Update layer panel when deselecting
  • Consistent tolerance when selecting lines

This change is Reviewable

@Keavon Keavon added P-High and removed P-High labels Jun 13, 2021
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Jun 13, 2021

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 5f0eb5f
Status: ✅  Deploy successful!
Preview URL: https://0118ed3c.graphite-master.pages.dev

View logs

@pkupper pkupper marked this pull request as ready for review June 27, 2021 16:56
@pkupper pkupper requested review from TrueDoctor and Keavon July 4, 2021 19:42
@Keavon
Copy link
Member

Keavon commented Jul 4, 2021

I'm going to merge this now @TrueDoctor but then we can go back and make changes after you have time to code review it. I'd really like to get this submitted.

@Keavon Keavon merged commit ad06460 into master Jul 4, 2021
@Keavon Keavon deleted the viewport-selection branch July 4, 2021 23:05
Keavon added a commit that referenced this pull request Jun 16, 2022
* Begin implementing viewport selection

* Implement viewport click and drag selection for ellipse and rectangle

* Begin implementing line selection

* Remove debug prints

* Run cargo format

* Use DVec2 instead of kurbo::Point

* Line and polyline intersection

* Run cargo format

* Add fix for missing layer panel update

* Replace point selection with box selection

* Formatting

Co-authored-by: Keavon Chambers <[email protected]>
Keavon added a commit that referenced this pull request Jul 30, 2023
* Begin implementing viewport selection

* Implement viewport click and drag selection for ellipse and rectangle

* Begin implementing line selection

* Remove debug prints

* Run cargo format

* Use DVec2 instead of kurbo::Point

* Line and polyline intersection

* Run cargo format

* Add fix for missing layer panel update

* Replace point selection with box selection

* Formatting

Co-authored-by: Keavon Chambers <[email protected]>
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.

Select Tool: Viewport box selection Select Tool: Viewport layer click selection
2 participants