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

fix(editor): image size and xywh when converting attachment to image #10200

Conversation

fundon
Copy link
Contributor

@fundon fundon commented Feb 14, 2025

In Edgeless, the image size should be read when converting attachment to image:

  • fix size
  • fix xywh

Copy link
Contributor Author

fundon commented Feb 14, 2025


How to use the Graphite Merge Queue

Add either label to this PR to merge it via the merge queue:

  • merge - adds this PR to the back of the merge queue
  • hotfix - for urgent hot fixes, skip the queue and merge this PR next

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

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

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

@fundon fundon marked this pull request as ready for review February 14, 2025 11:12
@fundon fundon requested a review from a team as a code owner February 14, 2025 11:12
@graphite-app graphite-app bot requested review from a team February 14, 2025 11:12
Copy link

codecov bot commented Feb 14, 2025

Codecov Report

Attention: Patch coverage is 25.71429% with 26 lines in your changes missing coverage. Please review.

Project coverage is 54.11%. Comparing base (ff0ce1a) to head (adc0038).
Report is 1 commits behind head on canary.

Files with missing lines Patch % Lines
blocksuite/affine/shared/src/utils/image.ts 0.00% 16 Missing ⚠️
blocksuite/affine/block-attachment/src/embed.ts 47.36% 9 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           canary   #10200      +/-   ##
==========================================
- Coverage   54.12%   54.11%   -0.01%     
==========================================
  Files        2334     2335       +1     
  Lines      107815   107834      +19     
  Branches    17818    17824       +6     
==========================================
+ Hits        58350    58357       +7     
- Misses      48134    48147      +13     
+ Partials     1331     1330       -1     
Flag Coverage Δ
server-test 78.39% <ø> (-0.01%) ⬇️
unittest 31.25% <0.00%> (-0.02%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@fundon fundon force-pushed the fundon/02_15_fix_image_size_when_converting_attachment_to_image branch 2 times, most recently from cd04817 to c1d5cd4 Compare February 14, 2025 12:10
@fundon fundon changed the base branch from fundon/02_15_fix_video_style_in_edgeless to graphite-base/10200 February 15, 2025 05:18
@fundon fundon force-pushed the graphite-base/10200 branch from e1117f8 to 978352c Compare February 15, 2025 05:18
@fundon fundon force-pushed the fundon/02_15_fix_image_size_when_converting_attachment_to_image branch from c1d5cd4 to c8215bd Compare February 15, 2025 05:19
@fundon fundon force-pushed the graphite-base/10200 branch from 978352c to b65cb4a Compare February 15, 2025 06:37
@fundon fundon force-pushed the fundon/02_15_fix_image_size_when_converting_attachment_to_image branch from c8215bd to 64d9e79 Compare February 15, 2025 06:37
@fundon fundon force-pushed the graphite-base/10200 branch from b65cb4a to b4274ec Compare February 17, 2025 01:23
@fundon fundon force-pushed the fundon/02_15_fix_image_size_when_converting_attachment_to_image branch from 64d9e79 to d68b354 Compare February 17, 2025 01:23
@fundon fundon force-pushed the graphite-base/10200 branch from b4274ec to ea94268 Compare February 17, 2025 01:25
@fundon fundon force-pushed the fundon/02_15_fix_image_size_when_converting_attachment_to_image branch from d68b354 to dfc6d57 Compare February 17, 2025 01:25
@fundon fundon force-pushed the graphite-base/10200 branch from ea94268 to 71aa9af Compare February 17, 2025 01:32
@fundon fundon force-pushed the fundon/02_15_fix_image_size_when_converting_attachment_to_image branch 2 times, most recently from 203cac0 to bf5f965 Compare February 17, 2025 03:49
@fundon fundon force-pushed the graphite-base/10200 branch from 71aa9af to 180976c Compare February 17, 2025 03:49
@fundon fundon force-pushed the fundon/02_15_fix_image_size_when_converting_attachment_to_image branch from bf5f965 to 5c90dbf Compare February 17, 2025 04:36
@fundon fundon force-pushed the graphite-base/10200 branch from 180976c to 280a26c Compare February 17, 2025 04:36
@fundon fundon changed the base branch from graphite-base/10200 to fundon/02_15_fix_video_style_in_edgeless February 17, 2025 04:36
@fundon fundon force-pushed the fundon/02_15_fix_image_size_when_converting_attachment_to_image branch from 5c90dbf to d4a96d4 Compare February 17, 2025 04:37
@fundon fundon changed the base branch from fundon/02_15_fix_video_style_in_edgeless to graphite-base/10200 February 17, 2025 04:56
@fundon fundon force-pushed the fundon/02_15_fix_image_size_when_converting_attachment_to_image branch from d4a96d4 to 555f9f5 Compare February 17, 2025 07:26
@fundon fundon force-pushed the graphite-base/10200 branch from 280a26c to 6dc1fef Compare February 17, 2025 07:26
@fundon fundon changed the base branch from graphite-base/10200 to canary February 17, 2025 07:26
@fundon fundon force-pushed the fundon/02_15_fix_image_size_when_converting_attachment_to_image branch from 555f9f5 to 08cd5a3 Compare February 17, 2025 07:26
@fundon fundon force-pushed the fundon/02_15_fix_image_size_when_converting_attachment_to_image branch from 08cd5a3 to 8bfa101 Compare February 20, 2025 04:52
@fundon fundon requested a review from Saul-Mirone February 20, 2025 04:58
Copy link

graphite-app bot commented Feb 20, 2025

Merge activity

…10200)

In Edgeless, the image size should be read when converting attachment to image:

* fix `size`
* fix `xywh`
@graphite-app graphite-app bot force-pushed the fundon/02_15_fix_image_size_when_converting_attachment_to_image branch from 3bd162d to adc0038 Compare February 20, 2025 05:16
@graphite-app graphite-app bot merged commit adc0038 into canary Feb 20, 2025
66 checks passed
@graphite-app graphite-app bot deleted the fundon/02_15_fix_image_size_when_converting_attachment_to_image branch February 20, 2025 05:30
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.

3 participants