blocksuite icon indicating copy to clipboard operation
blocksuite copied to clipboard

refactor(edgeless): make new block element always be on the top

Open L-Sun opened this issue 1 year ago β€’ 7 comments

Close BS-1399

What changes

  • Make new edgeless element be always on the top after refactoring method LayerManager.generateIndex()
  • Update layer related tests.
  • Fix some flaky edgeless e2e tests.

L-Sun avatar Sep 23 '24 10:09 L-Sun

⚠️ No Changeset found

Latest commit: 3bfab06ee3f4621fca26702e03aa3ec6b2a21374

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

changeset-bot[bot] avatar Sep 23 '24 10:09 changeset-bot[bot]

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 Sep 24, 2024 4:33am
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
blocksuite-docs ⬜️ Ignored (Inspect) Visit Preview Sep 24, 2024 4:33am

vercel[bot] avatar Sep 23 '24 10:09 vercel[bot]

Your org has enabled the Graphite merge queue for merging into master

Add the label β€œmerge” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge.

You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link.

graphite-app[bot] avatar Sep 23 '24 10:09 graphite-app[bot]

  • #8437 Graphite πŸ‘ˆ
  • master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @L-Sun and the rest of your teammates on Graphite Graphite

L-Sun avatar Sep 23 '24 10:09 L-Sun

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 3bfab06ee3f4621fca26702e03aa3ec6b2a21374. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

πŸ“‚ See all runs for this CI Pipeline Execution


βœ… Successfully ran 1 target

Sent with πŸ’Œ from NxCloud.

nx-cloud[bot] avatar Sep 23 '24 10:09 nx-cloud[bot]

size-limit report πŸ“¦

Path Size
@blocksuite/affine 13 B (0%)
@blocksuite/affine/effects 2.03 MB (+0.03% πŸ”Ί)
@blocksuite/affine/block-std 153.1 KB (-0.05% πŸ”½)
@blocksuite/affine/block-std/gfx 75.86 KB (-0.19% πŸ”½)
@blocksuite/affine/global 13 B (0%)
@blocksuite/affine/global/utils 13.82 KB (0%)
@blocksuite/affine/global/env 231 B (0%)
@blocksuite/affine/global/exceptions 552 B (0%)
@blocksuite/affine/global/di 1.65 KB (0%)
@blocksuite/affine/global/types 13 B (0%)
@blocksuite/affine/store 77.06 KB (0%)
@blocksuite/affine/inline 30.45 KB (0%)
@blocksuite/affine/inline/consts 51 B (0%)
@blocksuite/affine/inline/types 13 B (0%)
@blocksuite/affine/presets 1.83 MB (-0.01% πŸ”½)
@blocksuite/affine/blocks 1.92 MB (-0.05% πŸ”½)
@blocksuite/affine/blocks/schemas 85.39 KB (0%)

github-actions[bot] avatar Sep 23 '24 10:09 github-actions[bot]

What do I need to do ?

Magken91 avatar Sep 23 '24 11:09 Magken91

Merge activity

  • Sep 24, 12:25 AM EDT: The merge label 'merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.
  • Sep 24, 12:25 AM EDT: doodlewind added this pull request to the Graphite merge queue.
  • Sep 24, 12:36 AM EDT: doodlewind merged this pull request with the Graphite merge queue.

graphite-app[bot] avatar Sep 24 '24 04:09 graphite-app[bot]