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

feat(edgeless): support select element below the hollow shape #3212

Merged
merged 6 commits into from
Jun 26, 2023

Conversation

doouding
Copy link
Member

close #2034

Sorry for the long wait. I spent some time investigating several open source projects to find a proper solution. However, I found that all of them require a lot of effort. I believe that we need an overall solution for this type of geometric problem (just like how excalidraw do it), but it may not be our top priority right now. So I just used some simple math and native APIs to fulfill our needs.

@doouding doouding added mod:edgeless Module: related to edgeless mode type:feat Non-bug feature improvements labels Jun 26, 2023
@doouding doouding requested a review from doodlewind June 26, 2023 08:13
@codesandbox
Copy link

codesandbox bot commented Jun 26, 2023

This branch is running in CodeSandbox. Use the links below to review this PR faster.


CodeSandbox logoCodeSandbox logo  Open in CodeSandbox Web Editor | VS Code | VS Code Insiders | Preview

@vercel
Copy link

vercel bot commented Jun 26, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
blocksuite ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 26, 2023 3:27pm
blocksuite-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 26, 2023 3:27pm

@doouding doouding changed the title support select element below the hollow shape feat(edgeless): support select element below the hollow shape Jun 26, 2023
@doodlewind doodlewind removed mod:edgeless Module: related to edgeless mode type:feat Non-bug feature improvements labels Jun 26, 2023
@doodlewind
Copy link
Member

Just saying that you don't have to add module and type labels to PRs, they are mainly for issues :)

@doouding
Copy link
Member Author

It seems break some tests. I'll fix them soon.

Copy link
Member

@doodlewind doodlewind left a comment

Choose a reason for hiding this comment

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

Great to land this, thanks!

@doodlewind doodlewind merged commit 84a7ac0 into master Jun 26, 2023
18 checks passed
@doodlewind doodlewind deleted the feat/select_below_stroke branch June 26, 2023 16:23
fundon added a commit to fundon/blocksuite that referenced this pull request Jun 29, 2023
…editor

* master: (62 commits)
  fix: image resize after switch page mode (toeverything#3238)
  feat(edgeless): add index for other notes when selecting a note (toeverything#3235)
  feat: support import notion database (toeverything#3170)
  style: adjust checkbox style (toeverything#3240)
  fix: the incorrect import path (toeverything#3234)
  fix: console error exporting PNG/PDF in playground (toeverything#3205)
  feat(blocks): add page block service (toeverything#3232)
  refactor(edgeless): rename children container (toeverything#3231)
  fix: edgeless text error when importing from snapshot (toeverything#3229)
  refactor(blocks): split code (toeverything#3228)
  fix(store): handle resource load error when importing snapshot (toeverything#3227)
  feat: use human-readable name in code block (toeverything#3225)
  fix(store): snapshot import and export (toeverything#3223)
  fix(phasor): curveFitting should be enabled on ellipse (toeverything#3224)
  feat: support for adding tags to pages (toeverything#3203)
  fix(store): broadcast channel not defined (toeverything#3222)
  fix(store): bring back remove page for subdoc (toeverything#3221)
  fix(phasor): bound should be calculated using expand (toeverything#3220)
  refactor: migrate image resize manager to block level (toeverything#3218)
  feat(edgeless): support select element below the hollow shape (toeverything#3212)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Archived in project
Development

Successfully merging this pull request may close these issues.

Hope to be able to select other components below the stroke
2 participants