From 0be62d892d3f057e67b198296edc8cb47d934959 Mon Sep 17 00:00:00 2001
From: JimmFly <447268514@qq.com>
Date: Mon, 26 Feb 2024 08:54:52 +0000
Subject: [PATCH] refactor(core): use custom scrollbar for editor and adjust
shared page style (#5752)
Close TOV-481
- Use a custom scrollbar component for editor
- Modified the header of the share page and added a new footer
---
.../components/auth-components/share.css.ts | 4 +-
.../frontend/component/src/theme/global.css | 25 ----
.../page-history-modal/history-modal.tsx | 15 ++-
.../affine/page-history-modal/styles.css.ts | 1 -
.../block-suite-editor/lit-adaper.tsx | 5 +-
.../block-suite-editor/styles.css.ts | 12 +-
.../cloud/share-header-left-item/index.tsx | 26 ----
.../share-header-left-item/styles.css.ts | 15 ---
.../share-header-left-item/user-avatar.tsx | 45 -------
.../authenticated-item.tsx | 27 ++++-
.../cloud/share-header-right-item/index.tsx | 20 +++-
.../cloud/share-header-right-item/present.tsx | 22 ++--
.../share-header-right-item/styles.css.ts | 113 +++++++++++++++++-
.../share-header-right-item/user-avatar.tsx | 91 ++++++++++++++
.../src/components/page-detail-editor.css.ts | 6 -
.../src/components/page-detail-editor.tsx | 1 -
.../src/pages/share/share-detail-page.css.ts | 58 +++++++++
.../src/pages/share/share-detail-page.tsx | 56 +++++++--
.../core/src/pages/share/share-footer.css.ts | 47 ++++++++
.../core/src/pages/share/share-footer.tsx | 26 ++++
.../core/src/pages/share/share-header.css.ts | 18 +++
.../core/src/pages/share/share-header.tsx | 51 ++++----
.../workspace/detail-page/detail-page.css.ts | 1 -
.../workspace/detail-page/detail-page.tsx | 20 ++--
packages/frontend/i18n/src/resources/en.json | 6 +-
tests/affine-cloud/e2e/collaboration.spec.ts | 2 -
26 files changed, 504 insertions(+), 209 deletions(-)
delete mode 100644 packages/frontend/core/src/components/cloud/share-header-left-item/index.tsx
delete mode 100644 packages/frontend/core/src/components/cloud/share-header-left-item/styles.css.ts
delete mode 100644 packages/frontend/core/src/components/cloud/share-header-left-item/user-avatar.tsx
create mode 100644 packages/frontend/core/src/components/cloud/share-header-right-item/user-avatar.tsx
create mode 100644 packages/frontend/core/src/pages/share/share-detail-page.css.ts
create mode 100644 packages/frontend/core/src/pages/share/share-footer.css.ts
create mode 100644 packages/frontend/core/src/pages/share/share-footer.tsx
create mode 100644 packages/frontend/core/src/pages/share/share-header.css.ts
diff --git a/packages/frontend/component/src/components/auth-components/share.css.ts b/packages/frontend/component/src/components/auth-components/share.css.ts
index f9601f581..d3b482720 100644
--- a/packages/frontend/component/src/components/auth-components/share.css.ts
+++ b/packages/frontend/component/src/components/auth-components/share.css.ts
@@ -4,7 +4,7 @@ export const modalHeaderWrapper = style({});
globalStyle(`${modalHeaderWrapper} .logo`, {
fontSize: cssVar('fontH3'),
fontWeight: 600,
- color: cssVar('blue'),
+ color: cssVar('black'),
marginRight: '6px',
verticalAlign: 'middle',
});
@@ -32,7 +32,7 @@ export const authInputWrapper = style({
});
globalStyle(`${authInputWrapper} label`, {
display: 'block',
- color: 'var(--light-text-color-text-secondary-color, #8E8D91)',
+ color: cssVar('textSecondaryColor'),
marginBottom: '4px',
fontSize: cssVar('fontSm'),
fontWeight: 600,
diff --git a/packages/frontend/component/src/theme/global.css b/packages/frontend/component/src/theme/global.css
index dcd9f41f9..a111fb7d3 100644
--- a/packages/frontend/component/src/theme/global.css
+++ b/packages/frontend/component/src/theme/global.css
@@ -222,30 +222,6 @@ input[type='number']::-webkit-outer-spin-button {
height: 0;
}
-.affine-doc-viewport::-webkit-scrollbar {
- width: 20px;
- /* Chrome Safari */
- height: 20px;
-}
-
-.affine-doc-viewport::-webkit-scrollbar-thumb {
- border-radius: 12px;
- background-clip: padding-box;
- border-style: solid;
- border-width: 7px;
- border-color: transparent;
- transition: all 0.2s;
-}
-
-.affine-doc-viewport:hover::-webkit-scrollbar-thumb {
- background-color: var(--affine-divider-color);
-}
-
-.affine-doc-viewport:hover::-webkit-scrollbar-thumb:hover {
- background-color: var(--affine-icon-color);
- border-width: 5px;
-}
-
.editor-wrapper {
position: relative;
padding-right: 0;
@@ -282,7 +258,6 @@ affine-block-hub {
.editor-wrapper {
width: 100%;
- height: 100%;
}
/* issue: https://github.com/toeverything/AFFiNE/issues/2004 */
diff --git a/packages/frontend/core/src/components/affine/page-history-modal/history-modal.tsx b/packages/frontend/core/src/components/affine/page-history-modal/history-modal.tsx
index 3372fd8f8..deaec3054 100644
--- a/packages/frontend/core/src/components/affine/page-history-modal/history-modal.tsx
+++ b/packages/frontend/core/src/components/affine/page-history-modal/history-modal.tsx
@@ -139,11 +139,16 @@ const HistoryEditorPreview = ({
{snapshotPage ? (