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: supports sort all page #2356

Merged
merged 7 commits into from
May 15, 2023
Merged

feat: supports sort all page #2356

merged 7 commits into from
May 15, 2023

Conversation

lawvs
Copy link
Collaborator

@lawvs lawvs commented May 14, 2023

Fix #1752

Screen.Recording.2023-05-15.at.5.12.29.PM.mov

The implementation of function useSorter refers to the following libs:

@codesandbox
Copy link

codesandbox bot commented May 14, 2023

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

@vercel
Copy link

vercel bot commented May 14, 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 15, 2023 7:16am
affine-storybook ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 15, 2023 7:16am

@cloudflare-pages
Copy link

cloudflare-pages bot commented May 14, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: eb8e43b
Status: ✅  Deploy successful!
Preview URL: https://a4db87e5.affine.pages.dev
Branch Preview URL: https://feat-sort-all-page.affine.pages.dev

View logs

@lawvs lawvs temporarily deployed to development May 14, 2023 13:08 — with GitHub Actions Inactive
@lawvs lawvs temporarily deployed to development May 14, 2023 13:08 — with GitHub Actions Inactive
@lawvs lawvs temporarily deployed to development May 14, 2023 13:08 — with GitHub Actions Inactive
@lawvs lawvs temporarily deployed to development May 14, 2023 13:08 — with GitHub Actions Inactive
@lawvs lawvs temporarily deployed to development May 14, 2023 13:08 — with GitHub Actions Inactive
@lawvs lawvs temporarily deployed to development May 14, 2023 13:08 — with GitHub Actions Inactive
@lawvs lawvs marked this pull request as draft May 14, 2023 13:08
@lawvs lawvs requested a review from himself65 May 14, 2023 13:08
@codecov
Copy link

codecov bot commented May 14, 2023

Codecov Report

Patch coverage: 70.17% and project coverage change: -0.03 ⚠️

Comparison is base (2c4db4f) 72.37% compared to head (eb8e43b) 72.34%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2356      +/-   ##
==========================================
- Coverage   72.37%   72.34%   -0.03%     
==========================================
  Files         329      330       +1     
  Lines        7239     7286      +47     
  Branches     1135     1169      +34     
==========================================
+ Hits         5239     5271      +32     
- Misses       1793     1808      +15     
  Partials      207      207              
Flag Coverage Δ
e2etest 66.61% <52.77%> (-0.28%) ⬇️
e2etest-macos-latest-x64 ∅ <ø> (∅)
e2etest-ubuntu-latest-x64 47.52% <5.55%> (-0.33%) ⬇️
e2etest-windows-latest-x64 ∅ <ø> (∅)
server-test 73.12% <ø> (ø)
storybook-test 56.45% <70.83%> (+0.34%) ⬆️
unittest 40.46% <4.16%> (-0.36%) ⬇️

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

Impacted Files Coverage Δ
...es/component/src/components/page-list/all-page.tsx 75.00% <56.25%> (-2.28%) ⬇️
...s/component/src/components/page-list/use-sorter.ts 66.66% <66.66%> (ø)
...ponents/blocksuite/block-suite-page-list/index.tsx 73.01% <100.00%> (+0.88%) ⬆️
packages/component/src/ui/table/styles.ts 100.00% <100.00%> (ø)

... and 1 file 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.

@lawvs lawvs temporarily deployed to development May 14, 2023 13:10 — with GitHub Actions Inactive
@perfsee
Copy link

perfsee bot commented May 14, 2023

affine-toeverything

Bundle main

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

@@                       EntryPoint: main                        @@
##                     master …sort-all-page                 +/- ##
===================================================================
< Bundle              3.11 MB        3.23 MB     +123 kB(+3.94%)   
< Initial JS          1.86 MB        2.05 MB     +186 kB(+9.99%)   
< Initial CSS         16.8 kB        30.5 kB   +13.7 kB(+81.83%)   
#~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~#
< Assets                   41             47                  +6   
< Chunks                   39             41                  +2   
< Packages                132            139                  +7   
= Duplicates                3              3                       
#~~~~~~~~~~~~~~~~~~~~~~~~~~~~ Warnings ~~~~~~~~~~~~~~~~~~~~~~~~~~~#
! Separate mixed content assets files                              
! Avoid cache wasting                                              

@@                    EntryPoint: pages/_app                     @@
##                     master …sort-all-page                 +/- ##
===================================================================
< Bundle                 3 MB        3.11 MB     +113 kB(+3.78%)   
< Initial JS          1.75 MB        1.92 MB    +176 kB(+10.09%)   
< Initial CSS         16.8 kB        30.5 kB   +13.7 kB(+81.83%)   
#~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~#
< Assets                   39             45                  +6   
< Chunks                   37             39                  +2   
< Packages                132            139                  +7   
= Duplicates                3              3                       
#~~~~~~~~~~~~~~~~~~~~~~~~~~~~ Warnings ~~~~~~~~~~~~~~~~~~~~~~~~~~~#
! Separate mixed content assets files                              
! Avoid cache wasting                                              

@@                   EntryPoint: pages/_error                    @@
##                     master …sort-all-page                 +/- ##
===================================================================
= 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

@vercel vercel bot temporarily deployed to Preview – affine-preview May 14, 2023 13:12 Inactive
@lawvs lawvs temporarily deployed to development May 14, 2023 13:12 — with GitHub Actions Inactive
@lawvs lawvs temporarily deployed to development May 14, 2023 13:12 — with GitHub Actions Inactive
@lawvs lawvs temporarily deployed to development May 14, 2023 13:12 — with GitHub Actions Inactive
@lawvs lawvs temporarily deployed to development May 14, 2023 13:12 — with GitHub Actions Inactive
@lawvs lawvs temporarily deployed to development May 14, 2023 13:12 — with GitHub Actions Inactive
@lawvs lawvs temporarily deployed to development May 14, 2023 13:12 — with GitHub Actions Inactive
@lawvs lawvs temporarily deployed to development May 14, 2023 13:12 — with GitHub Actions Inactive
@lawvs lawvs temporarily deployed to development May 14, 2023 13:12 — with GitHub Actions Inactive
@vercel vercel bot temporarily deployed to Preview – affine-storybook May 14, 2023 13:16 Inactive
@lawvs lawvs temporarily deployed to development May 14, 2023 15:12 — with GitHub Actions Inactive
@lawvs lawvs temporarily deployed to development May 14, 2023 15:12 — with GitHub Actions Inactive
@lawvs lawvs temporarily deployed to development May 14, 2023 15:12 — with GitHub Actions Inactive
@lawvs lawvs temporarily deployed to development May 15, 2023 07:13 — with GitHub Actions Inactive
@lawvs lawvs temporarily deployed to development May 15, 2023 07:13 — with GitHub Actions Inactive
@lawvs lawvs temporarily deployed to development May 15, 2023 07:13 — with GitHub Actions Inactive
@lawvs lawvs temporarily deployed to development May 15, 2023 07:13 — with GitHub Actions Inactive
@lawvs lawvs temporarily deployed to development May 15, 2023 07:13 — with GitHub Actions Inactive
@lawvs lawvs temporarily deployed to development May 15, 2023 07:13 — with GitHub Actions Inactive
@lawvs lawvs temporarily deployed to development May 15, 2023 07:13 — with GitHub Actions Inactive
@lawvs lawvs temporarily deployed to development May 15, 2023 07:13 — with GitHub Actions Inactive
@vercel vercel bot temporarily deployed to Preview – affine-storybook May 15, 2023 07:16 Inactive
@lawvs lawvs temporarily deployed to development May 15, 2023 07:39 — with GitHub Actions Inactive
@lawvs lawvs temporarily deployed to development May 15, 2023 07:39 — with GitHub Actions Inactive
@lawvs lawvs temporarily deployed to development May 15, 2023 07:39 — with GitHub Actions Inactive
@lawvs lawvs temporarily deployed to development May 15, 2023 07:39 — with GitHub Actions Inactive
@lawvs lawvs temporarily deployed to development May 15, 2023 07:39 — with GitHub Actions Inactive
@lawvs lawvs temporarily deployed to development May 15, 2023 07:39 — with GitHub Actions Inactive
@lawvs lawvs temporarily deployed to development May 15, 2023 07:40 — with GitHub Actions Inactive
@lawvs lawvs temporarily deployed to development May 15, 2023 07:40 — with GitHub Actions Inactive
@lawvs lawvs temporarily deployed to development May 15, 2023 07:40 — with GitHub Actions Inactive
@lawvs lawvs temporarily deployed to development May 15, 2023 07:40 — with GitHub Actions Inactive
@lawvs lawvs temporarily deployed to development May 15, 2023 07:40 — with GitHub Actions Inactive
@lawvs lawvs temporarily deployed to development May 15, 2023 07:40 — with GitHub Actions Inactive
@lawvs lawvs temporarily deployed to development May 15, 2023 07:40 — with GitHub Actions Inactive
@lawvs lawvs temporarily deployed to development May 15, 2023 07:40 — with GitHub Actions Inactive
@lawvs lawvs temporarily deployed to development May 15, 2023 07:40 — with GitHub Actions Inactive
Copy link
Contributor

@himself65 himself65 left a comment

Choose a reason for hiding this comment

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

Although UI looks a bit strange, we can put it to the next PR to repair

@himself65 himself65 merged commit 9ff7dbf into master May 15, 2023
@himself65 himself65 deleted the feat/sort-all-page branch May 15, 2023 15:50
himself65 pushed a commit that referenced this pull request May 15, 2023
@lawvs lawvs mentioned this pull request May 16, 2023
Brooooooklyn pushed a commit that referenced this pull request May 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

​Sort by create date or update date or a-z by click on the top column
2 participants