refactor(editor): remove stable feature flags (#10547)
This commit is contained in:
@@ -6,7 +6,6 @@ import {
|
|||||||
EMBED_CARD_HEIGHT,
|
EMBED_CARD_HEIGHT,
|
||||||
EMBED_CARD_WIDTH,
|
EMBED_CARD_WIDTH,
|
||||||
} from '@blocksuite/affine-shared/consts';
|
} from '@blocksuite/affine-shared/consts';
|
||||||
import { FeatureFlagService } from '@blocksuite/affine-shared/services';
|
|
||||||
import {
|
import {
|
||||||
cloneReferenceInfoWithoutAliases,
|
cloneReferenceInfoWithoutAliases,
|
||||||
isNewTabTrigger,
|
isNewTabTrigger,
|
||||||
@@ -23,14 +22,6 @@ export class EmbedEdgelessLinkedDocBlockComponent extends toEdgelessEmbedBlock(
|
|||||||
override convertToEmbed = () => {
|
override convertToEmbed = () => {
|
||||||
const { id, doc, caption, xywh } = this.model;
|
const { id, doc, caption, xywh } = this.model;
|
||||||
|
|
||||||
// synced doc entry controlled by flag
|
|
||||||
const isSyncedDocEnabled = doc
|
|
||||||
.get(FeatureFlagService)
|
|
||||||
.getFlag('enable_synced_doc_block');
|
|
||||||
if (!isSyncedDocEnabled) {
|
|
||||||
return;
|
|
||||||
}
|
|
||||||
|
|
||||||
const style = 'syncedDoc';
|
const style = 'syncedDoc';
|
||||||
const bound = Bound.deserialize(xywh);
|
const bound = Bound.deserialize(xywh);
|
||||||
bound.w = EMBED_CARD_WIDTH[style];
|
bound.w = EMBED_CARD_WIDTH[style];
|
||||||
|
|||||||
@@ -14,7 +14,6 @@ import {
|
|||||||
import {
|
import {
|
||||||
DocDisplayMetaProvider,
|
DocDisplayMetaProvider,
|
||||||
DocModeProvider,
|
DocModeProvider,
|
||||||
FeatureFlagService,
|
|
||||||
OpenDocExtensionIdentifier,
|
OpenDocExtensionIdentifier,
|
||||||
type OpenDocMode,
|
type OpenDocMode,
|
||||||
ThemeProvider,
|
ThemeProvider,
|
||||||
@@ -131,14 +130,6 @@ export class EmbedLinkedDocBlockComponent extends EmbedBlockComponent<EmbedLinke
|
|||||||
|
|
||||||
const { doc, caption } = this.model;
|
const { doc, caption } = this.model;
|
||||||
|
|
||||||
// synced doc entry controlled by flag
|
|
||||||
const isSyncedDocEnabled = doc
|
|
||||||
.get(FeatureFlagService)
|
|
||||||
.getFlag('enable_synced_doc_block');
|
|
||||||
if (!isSyncedDocEnabled) {
|
|
||||||
return;
|
|
||||||
}
|
|
||||||
|
|
||||||
const parent = doc.getParent(this.model);
|
const parent = doc.getParent(this.model);
|
||||||
if (!parent) {
|
if (!parent) {
|
||||||
return;
|
return;
|
||||||
|
|||||||
@@ -43,7 +43,6 @@ import {
|
|||||||
import {
|
import {
|
||||||
EmbedOptionProvider,
|
EmbedOptionProvider,
|
||||||
type EmbedOptions,
|
type EmbedOptions,
|
||||||
FeatureFlagService,
|
|
||||||
GenerateDocUrlProvider,
|
GenerateDocUrlProvider,
|
||||||
type GenerateDocUrlService,
|
type GenerateDocUrlService,
|
||||||
type LinkEventType,
|
type LinkEventType,
|
||||||
@@ -419,16 +418,6 @@ export class EdgelessChangeEmbedCardButton extends WithDisposable(LitElement) {
|
|||||||
private get _canConvertToEmbedView() {
|
private get _canConvertToEmbedView() {
|
||||||
const block = this._blockComponent;
|
const block = this._blockComponent;
|
||||||
|
|
||||||
// synced doc entry controlled by awareness flag
|
|
||||||
if (!!block && isEmbedLinkedDocBlock(block.model)) {
|
|
||||||
const isSyncedDocEnabled = block.doc
|
|
||||||
.get(FeatureFlagService)
|
|
||||||
.getFlag('enable_synced_doc_block');
|
|
||||||
if (!isSyncedDocEnabled) {
|
|
||||||
return false;
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
return (
|
return (
|
||||||
(block && 'convertToEmbed' in block) ||
|
(block && 'convertToEmbed' in block) ||
|
||||||
this._embedOptions?.viewType === 'embed'
|
this._embedOptions?.viewType === 'embed'
|
||||||
|
|||||||
@@ -44,7 +44,6 @@ import {
|
|||||||
import {
|
import {
|
||||||
EmbedOptionProvider,
|
EmbedOptionProvider,
|
||||||
type EmbedOptions,
|
type EmbedOptions,
|
||||||
FeatureFlagService,
|
|
||||||
GenerateDocUrlProvider,
|
GenerateDocUrlProvider,
|
||||||
type GenerateDocUrlService,
|
type GenerateDocUrlService,
|
||||||
type LinkEventType,
|
type LinkEventType,
|
||||||
@@ -236,16 +235,6 @@ export class EmbedCardToolbar extends WidgetComponent<
|
|||||||
cloneGroups(BUILT_IN_GROUPS);
|
cloneGroups(BUILT_IN_GROUPS);
|
||||||
|
|
||||||
private get _canConvertToEmbedView() {
|
private get _canConvertToEmbedView() {
|
||||||
// synced doc entry controlled by awareness flag
|
|
||||||
if (this.focusModel && isEmbedLinkedDocBlock(this.focusModel)) {
|
|
||||||
const isSyncedDocEnabled = this.doc
|
|
||||||
.get(FeatureFlagService)
|
|
||||||
.getFlag('enable_synced_doc_block');
|
|
||||||
if (!isSyncedDocEnabled) {
|
|
||||||
return false;
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
if (!this.focusBlock) return false;
|
if (!this.focusBlock) return false;
|
||||||
|
|
||||||
return (
|
return (
|
||||||
|
|||||||
@@ -1,6 +1,5 @@
|
|||||||
import type { ReferenceInfo } from '@blocksuite/affine-model';
|
import type { ReferenceInfo } from '@blocksuite/affine-model';
|
||||||
import {
|
import {
|
||||||
FeatureFlagService,
|
|
||||||
GenerateDocUrlProvider,
|
GenerateDocUrlProvider,
|
||||||
type LinkEventType,
|
type LinkEventType,
|
||||||
OpenDocExtensionIdentifier,
|
OpenDocExtensionIdentifier,
|
||||||
@@ -320,10 +319,6 @@ export class ReferencePopup extends WithDisposable(LitElement) {
|
|||||||
}
|
}
|
||||||
|
|
||||||
private _viewSelector() {
|
private _viewSelector() {
|
||||||
// synced doc entry controlled by flag
|
|
||||||
const isSyncedDocEnabled = this.doc
|
|
||||||
.get(FeatureFlagService)
|
|
||||||
.getFlag('enable_synced_doc_block');
|
|
||||||
const buttons = [];
|
const buttons = [];
|
||||||
|
|
||||||
buttons.push({
|
buttons.push({
|
||||||
@@ -338,17 +333,13 @@ export class ReferencePopup extends WithDisposable(LitElement) {
|
|||||||
disabled: this.doc.readonly,
|
disabled: this.doc.readonly,
|
||||||
});
|
});
|
||||||
|
|
||||||
if (isSyncedDocEnabled) {
|
buttons.push({
|
||||||
buttons.push({
|
type: 'embed',
|
||||||
type: 'embed',
|
label: 'Embed view',
|
||||||
label: 'Embed view',
|
action: () => this._convertToEmbedView(),
|
||||||
action: () => this._convertToEmbedView(),
|
disabled:
|
||||||
disabled:
|
this.doc.readonly || this.isLinkedNode || this._embedViewButtonDisabled,
|
||||||
this.doc.readonly ||
|
});
|
||||||
this.isLinkedNode ||
|
|
||||||
this._embedViewButtonDisabled,
|
|
||||||
});
|
|
||||||
}
|
|
||||||
|
|
||||||
return html`
|
return html`
|
||||||
<editor-menu-button
|
<editor-menu-button
|
||||||
|
|||||||
@@ -2,8 +2,6 @@ import { type Store, StoreExtension } from '@blocksuite/store';
|
|||||||
import { type Signal, signal } from '@preact/signals-core';
|
import { type Signal, signal } from '@preact/signals-core';
|
||||||
|
|
||||||
export interface BlockSuiteFlags {
|
export interface BlockSuiteFlags {
|
||||||
enable_synced_doc_block: boolean;
|
|
||||||
enable_pie_menu: boolean;
|
|
||||||
enable_database_number_formatting: boolean;
|
enable_database_number_formatting: boolean;
|
||||||
enable_database_attachment_note: boolean;
|
enable_database_attachment_note: boolean;
|
||||||
enable_database_full_width: boolean;
|
enable_database_full_width: boolean;
|
||||||
@@ -24,8 +22,6 @@ export class FeatureFlagService extends StoreExtension {
|
|||||||
static override key = 'feature-flag-server';
|
static override key = 'feature-flag-server';
|
||||||
|
|
||||||
private readonly _flags: Signal<BlockSuiteFlags> = signal({
|
private readonly _flags: Signal<BlockSuiteFlags> = signal({
|
||||||
enable_synced_doc_block: true,
|
|
||||||
enable_pie_menu: false,
|
|
||||||
enable_database_number_formatting: false,
|
enable_database_number_formatting: false,
|
||||||
enable_database_attachment_note: false,
|
enable_database_attachment_note: false,
|
||||||
enable_database_full_width: false,
|
enable_database_full_width: false,
|
||||||
|
|||||||
@@ -41,8 +41,6 @@ function createCollectionOptions() {
|
|||||||
schema,
|
schema,
|
||||||
idGenerator,
|
idGenerator,
|
||||||
defaultFlags: {
|
defaultFlags: {
|
||||||
enable_synced_doc_block: true,
|
|
||||||
enable_pie_menu: true,
|
|
||||||
readonly: {
|
readonly: {
|
||||||
'doc:home': false,
|
'doc:home': false,
|
||||||
},
|
},
|
||||||
|
|||||||
@@ -53,16 +53,6 @@ export const AFFINE_FLAGS = {
|
|||||||
'com.affine.settings.workspace.experimental-features.enable-block-query.description',
|
'com.affine.settings.workspace.experimental-features.enable-block-query.description',
|
||||||
configurable: isCanaryBuild,
|
configurable: isCanaryBuild,
|
||||||
},
|
},
|
||||||
enable_synced_doc_block: {
|
|
||||||
category: 'blocksuite',
|
|
||||||
bsFlag: 'enable_synced_doc_block',
|
|
||||||
displayName:
|
|
||||||
'com.affine.settings.workspace.experimental-features.enable-synced-doc-block.name',
|
|
||||||
description:
|
|
||||||
'com.affine.settings.workspace.experimental-features.enable-synced-doc-block.description',
|
|
||||||
configurable: false,
|
|
||||||
defaultState: true,
|
|
||||||
},
|
|
||||||
enable_edgeless_text: {
|
enable_edgeless_text: {
|
||||||
category: 'blocksuite',
|
category: 'blocksuite',
|
||||||
bsFlag: 'enable_edgeless_text',
|
bsFlag: 'enable_edgeless_text',
|
||||||
|
|||||||
Reference in New Issue
Block a user