refactor(core): new back&forward button base on workbench

[EYHN]

feature:

In Browser:

• hidden back&forward button in sidebar.
• back and forward is equal with window.history.back() window.history.forward()

In Desktop:

• Back and forward can be controlled through the sidebar, cmdk, and shortcut keys.
• back and forward act on the currently active view.
• buttons change disable&enable style based on current active view history

Refactor:

Move app-sidebar and app-container from @affine/component to @affine/core

[graphite-app]

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

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

You can enable merging using labels in your Graphite merge queue settings.

[EYHN]

• refactor(core): new back&forward button base on workbench #6012 👈
• canary

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

Join @EYHN and the rest of your teammates on Graphite

[nx-cloud]

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 7c76c25. 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 6 targets

• nx test:coverage @affine/monorepo
• nx build @affine/storage -- --target x86_64-unknown-linux-gnu --use-napi-cross
• nx build @affine/native -- --target x86_64-pc-windows-msvc --use-napi-cross
• nx build @affine/native -- --target x86_64-apple-darwin --use-napi-cross
• nx build @affine/native -- --target aarch64-apple-darwin --use-napi-cross
• nx build @affine/native -- --target x86_64-unknown-linux-gnu --use-napi-cross

---

Sent with 💌 from NxCloud.

[codecov]

Codecov Report

Attention: Patch coverage is 2.87770% with 135 lines in your changes are missing coverage. Please review.

Project coverage is 61.64%. Comparing base (b06aeb2) to head (7c76c25).

Files	Patch %	Lines
...kages/frontend/core/src/utils/navigable-history.ts	0.00%	67 Missing ⚠️
...src/modules/navigation/view/navigation-buttons.tsx	3.33%	29 Missing ⚠️
.../core/src/modules/navigation/entities/navigator.ts	0.00%	18 Missing ⚠️
...ontend/core/src/modules/workbench/entities/view.ts	0.00%	7 Missing ⚠️
...d/core/src/modules/workbench/entities/workbench.ts	0.00%	6 Missing ⚠️
...avigation/view/use-register-navigation-commands.ts	0.00%	3 Missing ⚠️
...core/src/modules/workbench/view/desktop-adapter.ts	0.00%	3 Missing ⚠️
...rc/components/app-sidebar/sidebar-header/index.tsx	33.33%	2 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           canary    #6012      +/-   ##
==========================================
- Coverage   62.49%   61.64%   -0.86%     
==========================================
  Files         436      460      +24     
  Lines       20863    21267     +404     
  Branches     1833     1882      +49     
==========================================
+ Hits        13039    13110      +71     
- Misses       7605     7935     +330     
- Partials      219      222       +3     

Flag	Coverage Δ
server-test	71.04% <ø> (-0.13%)	⬇️
unittest	40.61% <2.87%> (-1.22%)	⬇️

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.

[pengx17]

It seems history back/forward with keyboard shortcut does not work well.

[graphite-app]

Merge activity

• Mar 5, 1:56 AM EST: Brooooooklyn added this pull request to the Graphite merge queue.
• Mar 5, 2:11 AM EST: Brooooooklyn merged this pull request with the Graphite merge queue.