Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat(component): support image preview by double click #2198

Merged
merged 13 commits into from
May 9, 2023

Conversation

himself65
Copy link
Contributor

@himself65 himself65 commented Apr 28, 2023

Fixes: #1582

@himself65 himself65 temporarily deployed to development April 28, 2023 23:54 — with GitHub Actions Inactive
@vercel
Copy link

vercel bot commented Apr 28, 2023

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

Name Status Preview Comments Updated (UTC)
affine-preview ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 9, 2023 6:18am
affine-storybook ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 9, 2023 6:18am

@himself65 himself65 temporarily deployed to development April 28, 2023 23:54 — with GitHub Actions Inactive
@himself65 himself65 temporarily deployed to development April 28, 2023 23:54 — with GitHub Actions Inactive
@himself65 himself65 temporarily deployed to development April 28, 2023 23:54 — with GitHub Actions Inactive
@himself65 himself65 temporarily deployed to development April 28, 2023 23:54 — with GitHub Actions Inactive
@himself65 himself65 temporarily deployed to development April 28, 2023 23:54 — with GitHub Actions Inactive
@himself65 himself65 temporarily deployed to development April 28, 2023 23:57 — with GitHub Actions Inactive
@codecov
Copy link

codecov bot commented Apr 28, 2023

Codecov Report

Patch coverage: 28.07% and project coverage change: -0.32 ⚠️

Comparison is base (242e074) 71.78% compared to head (4ad7ac1) 71.47%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2198      +/-   ##
==========================================
- Coverage   71.78%   71.47%   -0.32%     
==========================================
  Files         296      298       +2     
  Lines        6593     6650      +57     
  Branches     1068     1078      +10     
==========================================
+ Hits         4733     4753      +20     
- Misses       1663     1698      +35     
- Partials      197      199       +2     
Flag Coverage Δ
e2etest 68.85% <25.92%> (-0.38%) ⬇️
server-test 73.14% <ø> (ø)
storybook-test 55.21% <25.49%> (-1.14%) ⬇️
unittest 36.97% <25.00%> (-0.02%) ⬇️

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

Impacted Files Coverage Δ
apps/web/preset.config.mjs 57.14% <0.00%> (-9.53%) ⬇️
packages/env/src/config.ts 94.28% <ø> (ø)
...onent/src/components/image-preview-modal/index.tsx 11.36% <11.36%> (ø)
.../src/components/image-preview-modal/index.jotai.ts 87.50% <87.50%> (ø)
apps/web/src/providers/modal-provider.tsx 84.00% <100.00%> (ø)
...ponent/src/components/block-suite-editor/index.tsx 92.06% <100.00%> (+0.53%) ⬆️

... and 2 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@vercel vercel bot temporarily deployed to Preview – affine-preview April 28, 2023 23:59 Inactive
@perfsee
Copy link

perfsee bot commented Apr 28, 2023

affine-toeverything

Bundle main

diff ------------------- Bundle Size Diff -------------------------

@@                       EntryPoint: main                        @@
##                     master …image-preview                 +/- ##
===================================================================
< Bundle              3.11 MB        3.14 MB    +33.7 kB(+1.09%)   
< Initial JS          1.86 MB        1.87 MB    +8.24 kB(+0.44%)   
< Initial CSS         16.8 kB        18.8 kB   +2.06 kB(+12.29%)   
#~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~#
< Assets                   41             47                  +6   
< Chunks                   39             43                  +4   
< Packages                132            134                  +2   
= Duplicates                3              3                       
#~~~~~~~~~~~~~~~~~~~~~~~~~~~~ Warnings ~~~~~~~~~~~~~~~~~~~~~~~~~~~#
! Separate mixed content assets files                              
! Avoid non-minified assets                                        
! Avoid cache wasting                                              

@@                    EntryPoint: pages/_app                     @@
##                     master …image-preview                 +/- ##
===================================================================
< Bundle                 3 MB        3.02 MB    +24.3 kB(+0.81%)   
> Initial JS          1.75 MB        1.75 MB    -1.17 kB(-0.07%)   
< Initial CSS         16.8 kB        18.8 kB   +2.06 kB(+12.29%)   
#~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~#
< Assets                   39             45                  +6   
< Chunks                   37             41                  +4   
< Packages                132            134                  +2   
= Duplicates                3              3                       
#~~~~~~~~~~~~~~~~~~~~~~~~~~~~ Warnings ~~~~~~~~~~~~~~~~~~~~~~~~~~~#
! Separate mixed content assets files                              
! Avoid non-minified assets                                        
! Avoid cache wasting                                              

@@                   EntryPoint: pages/_error                    @@
##                     master …image-preview                 +/- ##
===================================================================
= Bundle              16.6 kB        16.6 kB                       
= Initial JS          16.6 kB        16.6 kB                       
= Initial CSS             0 B            0 B                       
#~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~#
= Assets                    2              2                       
= Chunks                    2              2                       
= Packages                  5              5                       
= Duplicates                0              0                       

...and more

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Apr 29, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 4ad7ac1
Status: ✅  Deploy successful!
Preview URL: https://8c9cca39.affine.pages.dev
Branch Preview URL: https://himself65-0428-image-preview.affine.pages.dev

View logs

@himself65 himself65 temporarily deployed to development April 29, 2023 00:02 — with GitHub Actions Inactive
@himself65 himself65 temporarily deployed to development April 29, 2023 00:02 — with GitHub Actions Inactive
@himself65 himself65 temporarily deployed to development April 29, 2023 00:02 — with GitHub Actions Inactive
@himself65 himself65 temporarily deployed to development April 29, 2023 00:02 — with GitHub Actions Inactive
@himself65 himself65 temporarily deployed to development April 29, 2023 00:02 — with GitHub Actions Inactive
@vercel vercel bot temporarily deployed to Preview – affine-storybook April 29, 2023 00:02 Inactive
@himself65 himself65 temporarily deployed to development April 29, 2023 00:05 — with GitHub Actions Inactive
@himself65 himself65 temporarily deployed to development April 29, 2023 00:05 — with GitHub Actions Inactive
@himself65 himself65 temporarily deployed to development April 29, 2023 00:05 — with GitHub Actions Inactive
@himself65 himself65 temporarily deployed to development April 29, 2023 00:05 — with GitHub Actions Inactive
@himself65 himself65 temporarily deployed to development April 29, 2023 00:05 — with GitHub Actions Inactive
@himself65 himself65 temporarily deployed to development April 29, 2023 00:05 — with GitHub Actions Inactive
@himself65 himself65 temporarily deployed to development April 29, 2023 00:07 — with GitHub Actions Inactive
@himself65 himself65 temporarily deployed to development May 9, 2023 05:56 — with GitHub Actions Inactive
@himself65 himself65 temporarily deployed to development May 9, 2023 05:56 — with GitHub Actions Inactive
@himself65 himself65 temporarily deployed to development May 9, 2023 05:56 — with GitHub Actions Inactive
@himself65 himself65 temporarily deployed to development May 9, 2023 05:56 — with GitHub Actions Inactive
@himself65 himself65 temporarily deployed to development May 9, 2023 05:56 — with GitHub Actions Inactive
@himself65 himself65 temporarily deployed to development May 9, 2023 05:56 — with GitHub Actions Inactive
@himself65 himself65 temporarily deployed to development May 9, 2023 05:56 — with GitHub Actions Inactive
@himself65 himself65 temporarily deployed to development May 9, 2023 05:56 — with GitHub Actions Inactive
@himself65 himself65 temporarily deployed to development May 9, 2023 05:57 — with GitHub Actions Inactive
@himself65 himself65 temporarily deployed to development May 9, 2023 05:57 — with GitHub Actions Inactive
@himself65 himself65 temporarily deployed to development May 9, 2023 05:57 — with GitHub Actions Inactive
@himself65 himself65 temporarily deployed to development May 9, 2023 05:57 — with GitHub Actions Inactive
@himself65 himself65 temporarily deployed to development May 9, 2023 05:57 — with GitHub Actions Inactive
@himself65 himself65 temporarily deployed to development May 9, 2023 05:57 — with GitHub Actions Inactive
@himself65 himself65 temporarily deployed to development May 9, 2023 05:59 — with GitHub Actions Inactive
@himself65 himself65 temporarily deployed to development May 9, 2023 06:04 — with GitHub Actions Inactive
@himself65 himself65 temporarily deployed to development May 9, 2023 06:04 — with GitHub Actions Inactive
@himself65 himself65 temporarily deployed to development May 9, 2023 06:04 — with GitHub Actions Inactive
@himself65 himself65 temporarily deployed to development May 9, 2023 06:04 — with GitHub Actions Inactive
@himself65 himself65 temporarily deployed to development May 9, 2023 06:04 — with GitHub Actions Inactive
@himself65 himself65 temporarily deployed to development May 9, 2023 06:04 — with GitHub Actions Inactive
@himself65 himself65 temporarily deployed to development May 9, 2023 06:04 — with GitHub Actions Inactive
@himself65 himself65 temporarily deployed to development May 9, 2023 06:04 — with GitHub Actions Inactive
@himself65 himself65 disabled auto-merge May 9, 2023 06:09
@himself65 himself65 merged commit c41718e into master May 9, 2023
@himself65 himself65 deleted the himself65/0428-image-preview branch May 9, 2023 06:09
@vercel vercel bot temporarily deployed to Preview – affine-storybook May 9, 2023 06:13 Inactive
@vercel vercel bot temporarily deployed to Preview – affine-preview May 9, 2023 06:18 Inactive
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Support image viewer on double click
1 participant