@univerjs/sheets-conditional-formatting-ui 1.0.0-alpha.2 → 1.0.0-alpha.4

package/lib/es/index.js

@@ -1,14 +1,14 @@
 import { BooleanNumber, ColorKit, CommandType, DependentOn, Disposable, DisposableCollection, ICommandService, IConfigService, IUniverInstanceService, Inject, Injector, InterceptorEffectEnum, InterceptorManager, LocaleService, ObjectMatrix, Plugin, Range, Rectangle, Tools, UniverInstanceType, createInterceptorKey, generateRandomId, get, merge, registerDependencies, set, toDisposable, touchDependencies } from "@univerjs/core";
 import { AFTER_CELL_EDIT, AUTO_FILL_APPLY_TYPE, AutoFillTools, ClearSelectionAllCommand, ClearSelectionFormatCommand, IAutoFillService, INTERCEPTOR_POINT, RangeProtectionPermissionEditPoint, RemoveSheetMutation, SetSelectionsOperation, SetWorksheetActiveOperation, SheetInterceptorService, SheetPermissionCheckController, SheetsSelectionsService, WorkbookEditablePermission, WorksheetEditPermission, WorksheetSetCellStylePermission, checkRangesEditablePermission, getSheetCommandTarget, rangeToDiscreteRange, setEndForRange } from "@univerjs/sheets";
 import { AddCfCommand, AddConditionalRuleMutation, AddConditionalRuleMutationUndoFactory, CFNumberOperator, CFRuleType, CFSubRuleType, CFTextOperator, CFTimePeriodOperator, CFValueType, CONDITIONAL_FORMATTING_VIEWPORT_CACHE_LENGTH, ClearRangeCfCommand, ClearWorksheetCfCommand, ConditionalFormattingRangeIndexModel, ConditionalFormattingRangeTransformService, ConditionalFormattingRuleModel, ConditionalFormattingService, ConditionalFormattingViewModel, DEFAULT_BG_COLOR, DEFAULT_FONT_COLOR, DEFAULT_PADDING, DEFAULT_WIDTH, DeleteCfCommand, DeleteConditionalRuleMutation, DeleteConditionalRuleMutationUndoFactory, IIconSetType, MoveCfCommand, MoveConditionalRuleMutation, SHEET_CONDITIONAL_FORMATTING_PLUGIN, SetCfCommand, SetConditionalRuleMutation, UniverSheetsConditionalFormattingPlugin, compareWithNumber, createCfId, createDefaultRule, createDefaultValue, createDefaultValueByValueType, defaultDataBarNativeColor, defaultDataBarPositiveColor, getColorScaleFromValue, getOppositeOperator, iconGroup, iconMap, removeUndefinedAttr, setConditionalRuleMutationUndoFactory } from "@univerjs/sheets-conditional-formatting";
-import { ComponentManager, ILayoutService, IMenuManagerService, ISidebarService, IconManager, MenuItemType, RibbonDataGroup, getMenuHiddenObservable, useDependency, useObservable, useScrollYOverContainer, useSidebarClick } from "@univerjs/ui";
-import { FormulaRefRangeService, UniverSheetsFormulaPlugin } from "@univerjs/sheets-formula";
+import { ComponentManager, IMenuManagerService, ISidebarService, IconManager, MenuItemType, RibbonDataGroup, RibbonInsertGroup, getMenuHiddenObservable, useDependency, useObservable, useSidebarClick } from "@univerjs/ui";
 import { COPY_TYPE, FormatPainterStatus, IFormatPainterService, ISheetClipboardService, PREDEFINED_HOOK_NAME_PASTE, SheetSkeletonManagerService, getCurrentRangeDisable$, getRepeatRange, useHighlightRange, virtualizeDiscreteRanges } from "@univerjs/sheets-ui";
+import { Observable, debounceTime, filter, map, merge as merge$1, share, startWith } from "rxjs";
+import { bufferTime, debounceTime as debounceTime$1, filter as filter$1 } from "rxjs/operators";
+import { FormulaRefRangeService, UniverSheetsFormulaPlugin } from "@univerjs/sheets-formula";
 import { IRenderManagerService } from "@univerjs/engine-render";
-import { Observable, debounceTime, merge as merge$1 } from "rxjs";
-import { bufferTime, debounceTime as debounceTime$1, filter } from "rxjs/operators";
-import { BoldIcon, ConditionsDoubleIcon, DeleteIcon, FontColorDoubleIcon, GripVerticalIcon, IncreaseIcon, ItalicIcon, MoreDownIcon, PaintBucketDoubleIcon, SlashDoubleIcon, StrikethroughIcon, UnderlineIcon } from "@univerjs/icons";
-import { forwardRef, useEffect, useMemo, useRef, useState } from "react";
+import { BoldIcon, ConditionsDoubleIcon, DataBarIcon, DeleteIcon, FontColorDoubleIcon, GripVerticalIcon, IncreaseIcon, ItalicIcon, MoreDownIcon, PaintBucketDoubleIcon, RatingIcon, SlashDoubleIcon, StrikethroughIcon, UnderlineIcon } from "@univerjs/icons";
+import { useEffect, useMemo, useRef, useState } from "react";
 import { Button, Checkbox, ColorPicker, DraggableList, Dropdown, Input, InputNumber, Radio, RadioGroup, Select, Tooltip, borderClassName, clsx } from "@univerjs/design";
 import { LexerTreeBuilder, deserializeRangeWithSheet, operatorToken, serializeRange } from "@univerjs/engine-formula";
 import { FormulaEditor, RangeSelector } from "@univerjs/sheets-formula-ui";
@@ -81,8 +81,11 @@ const AddDataBarConditionalRuleCommand = {
 	type: CommandType.COMMAND,
 	id: "sheet.command.add-data-bar-conditional-rule",
 	handler(accessor, params) {
+		var _params$ranges;
 		if (!params) return false;
-		const { ranges, min, max, nativeColor, positiveColor, isGradient, stopIfTrue, isShowValue } = params;
+		const { min, max, nativeColor, positiveColor, isGradient, stopIfTrue, isShowValue } = params;
+		const ranges = (_params$ranges = params.ranges) !== null && _params$ranges !== void 0 ? _params$ranges : accessor.get(SheetsSelectionsService).getCurrentSelections().map((selection) => selection.range);
+		if (!ranges.length) return false;
 		const conditionalFormattingRuleModel = accessor.get(ConditionalFormattingRuleModel);
 		const target = getSheetCommandTarget(accessor.get(IUniverInstanceService));
 		if (!target) return false;
@@ -143,6 +146,37 @@ const AddDuplicateValuesCfCommand = {
 	}
 };
 
+//#endregion
+//#region src/commands/commands/add-icon-set-cf.command.ts
+const AddIconSetConditionalRuleCommand = {
+	type: CommandType.COMMAND,
+	id: "sheet.command.add-icon-set-conditional-rule",
+	handler(accessor, params) {
+		var _params$ranges;
+		if (!params) return false;
+		const ranges = (_params$ranges = params.ranges) !== null && _params$ranges !== void 0 ? _params$ranges : accessor.get(SheetsSelectionsService).getCurrentSelections().map((selection) => selection.range);
+		if (!ranges.length) return false;
+		const target = getSheetCommandTarget(accessor.get(IUniverInstanceService));
+		if (!target) return false;
+		const { unitId, subUnitId } = target;
+		const rule = {
+			ranges,
+			cfId: accessor.get(ConditionalFormattingRuleModel).createCfId(unitId, subUnitId),
+			stopIfTrue: !!params.stopIfTrue,
+			rule: {
+				type: CFRuleType.iconSet,
+				config: params.config,
+				isShowValue: params.isShowValue
+			}
+		};
+		return accessor.get(ICommandService).executeCommand(AddConditionalRuleMutation.id, {
+			unitId,
+			subUnitId,
+			rule
+		});
+	}
+};
+
 //#endregion
 //#region src/commands/commands/add-number-cf.command.ts
 const AddNumberCfCommand = {
@@ -393,11 +427,9 @@ function __decorate(decorators, target, key, desc) {
 //#region src/controllers/cf.panel.controller.ts
 const CF_PANEL_KEY$1 = "sheet.conditional.formatting.panel";
 let ConditionalFormattingPanelController = class ConditionalFormattingPanelController extends Disposable {
-	constructor(_univerInstanceService, _injector, _componentManager, _sidebarService, _localeService) {
+	constructor(_univerInstanceService, _sidebarService, _localeService) {
 		super();
 		this._univerInstanceService = _univerInstanceService;
-		this._injector = _injector;
-		this._componentManager = _componentManager;
 		this._sidebarService = _sidebarService;
 		this._localeService = _localeService;
 		_defineProperty(this, "_sidebarDisposable", null);
@@ -429,10 +461,8 @@ let ConditionalFormattingPanelController = class ConditionalFormattingPanelContr
 };
 ConditionalFormattingPanelController = __decorate([
 	__decorateParam(0, IUniverInstanceService),
-	__decorateParam(1, Inject(Injector)),
-	__decorateParam(2, Inject(ComponentManager)),
-	__decorateParam(3, Inject(ISidebarService)),
-	__decorateParam(4, Inject(LocaleService))
+	__decorateParam(1, Inject(ISidebarService)),
+	__decorateParam(2, Inject(LocaleService))
 ], ConditionalFormattingPanelController);
 
 //#endregion
@@ -637,10 +667,317 @@ function generateClearCfMutations(injector, allRules, ranges, unitId, subUnitId)
 	};
 }
 
+//#endregion
+//#region src/menu/manage-rule.ts
+const commandList = [
+	SetWorksheetActiveOperation.id,
+	AddConditionalRuleMutation.id,
+	SetConditionalRuleMutation.id,
+	DeleteConditionalRuleMutation.id,
+	MoveConditionalRuleMutation.id
+];
+const commonSelections = [
+	{
+		label: {
+			name: "sheets-conditional-formatting-ui.ruleType.highlightCell",
+			selectable: false
+		},
+		value: 3
+	},
+	{
+		label: {
+			name: "sheets-conditional-formatting-ui.panel.rankAndAverage",
+			selectable: false
+		},
+		value: 4
+	},
+	{
+		label: {
+			name: "sheets-conditional-formatting-ui.ruleType.formula",
+			selectable: false
+		},
+		value: 5
+	},
+	{
+		label: {
+			name: "sheets-conditional-formatting-ui.ruleType.colorScale",
+			selectable: false
+		},
+		value: 6
+	},
+	{
+		label: {
+			name: "sheets-conditional-formatting-ui.ruleType.dataBar",
+			selectable: false
+		},
+		value: 7
+	},
+	{
+		label: {
+			name: "sheets-conditional-formatting-ui.ruleType.iconSet",
+			selectable: false
+		},
+		value: 8
+	},
+	{
+		label: {
+			name: "sheets-conditional-formatting-ui.menu.manageConditionalFormatting",
+			selectable: false
+		},
+		value: 2
+	},
+	{
+		label: {
+			name: "sheets-conditional-formatting-ui.menu.createConditionalFormatting",
+			selectable: false
+		},
+		value: 1
+	},
+	{
+		label: {
+			name: "sheets-conditional-formatting-ui.menu.clearRangeRules",
+			selectable: false
+		},
+		value: 9,
+		disabled: false
+	},
+	{
+		label: {
+			name: "sheets-conditional-formatting-ui.menu.clearWorkSheetRules",
+			selectable: false
+		},
+		value: 10
+	}
+];
+function FactoryManageConditionalFormattingRule(accessor) {
+	const selectionManagerService = accessor.get(SheetsSelectionsService);
+	const commandService = accessor.get(ICommandService);
+	const univerInstanceService = accessor.get(IUniverInstanceService);
+	const conditionalFormattingRuleModel = accessor.get(ConditionalFormattingRuleModel);
+	const clearRangeEnable$ = new Observable((subscriber) => merge$1(selectionManagerService.selectionMoveEnd$, selectionManagerService.selectionSet$, new Observable((commandSubscribe) => {
+		const disposable = commandService.onCommandExecuted((commandInfo) => {
+			var _univerInstanceServic;
+			const { id, params } = commandInfo;
+			const unitId = (_univerInstanceServic = univerInstanceService.getCurrentUnitOfType(UniverInstanceType.UNIVER_SHEET)) === null || _univerInstanceServic === void 0 ? void 0 : _univerInstanceServic.getUnitId();
+			if (commandList.includes(id) && params.unitId === unitId) commandSubscribe.next(null);
+		});
+		return () => disposable.dispose();
+	})).pipe(debounceTime$1(16)).subscribe(() => {
+		var _selectionManagerServ;
+		const ranges = ((_selectionManagerServ = selectionManagerService.getCurrentSelections()) === null || _selectionManagerServ === void 0 ? void 0 : _selectionManagerServ.map((selection) => selection.range)) || [];
+		const workbook = univerInstanceService.getCurrentUnitOfType(UniverInstanceType.UNIVER_SHEET);
+		if (!workbook) return;
+		const worksheet = workbook.getActiveSheet();
+		if (!worksheet) return;
+		const hasPermission = (conditionalFormattingRuleModel.getSubunitRules(workbook.getUnitId(), worksheet.getSheetId()) || []).filter((rule) => rule.ranges.some((ruleRange) => ranges.some((range) => Rectangle.intersects(range, ruleRange)))).map((rule) => rule.ranges).every((ranges) => {
+			return checkRangesEditablePermission(accessor, workbook.getUnitId(), worksheet.getSheetId(), ranges);
+		});
+		subscriber.next(hasPermission);
+	}));
+	const clearSheetEnable$ = new Observable((subscriber) => new Observable((commandSubscribe) => {
+		const disposable = commandService.onCommandExecuted((commandInfo) => {
+			var _univerInstanceServic2;
+			const { id, params } = commandInfo;
+			const unitId = (_univerInstanceServic2 = univerInstanceService.getCurrentUnitOfType(UniverInstanceType.UNIVER_SHEET)) === null || _univerInstanceServic2 === void 0 ? void 0 : _univerInstanceServic2.getUnitId();
+			if (commandList.includes(id) && params.unitId === unitId) commandSubscribe.next(null);
+		});
+		return () => disposable.dispose();
+	}).pipe(debounceTime$1(16)).subscribe(() => {
+		const workbook = univerInstanceService.getCurrentUnitOfType(UniverInstanceType.UNIVER_SHEET);
+		if (!workbook) return;
+		const worksheet = workbook.getActiveSheet();
+		if (!worksheet) return;
+		const allRule = conditionalFormattingRuleModel.getSubunitRules(workbook.getUnitId(), worksheet.getSheetId()) || [];
+		if (!allRule.length) {
+			subscriber.next(false);
+			return false;
+		}
+		const hasPermission = allRule.map((rule) => rule.ranges).every((ranges) => {
+			return checkRangesEditablePermission(accessor, workbook.getUnitId(), worksheet.getSheetId(), ranges);
+		});
+		subscriber.next(hasPermission);
+	}));
+	const selections$ = new Observable((subscriber) => {
+		clearRangeEnable$.subscribe((v) => {
+			const item = commonSelections.find((item) => item.value === 9);
+			if (item) {
+				item.disabled = !v;
+				subscriber.next(commonSelections);
+			}
+		});
+		clearSheetEnable$.subscribe((v) => {
+			const item = commonSelections.find((item) => item.value === 10);
+			if (item) {
+				item.disabled = !v;
+				subscriber.next(commonSelections);
+			}
+		});
+		subscriber.next(commonSelections);
+	});
+	return {
+		id: OpenConditionalFormattingOperator.id,
+		type: MenuItemType.SELECTOR,
+		icon: "ConditionsDoubleIcon",
+		tooltip: "sheets-conditional-formatting-ui.title",
+		selections: selections$,
+		hidden$: getMenuHiddenObservable(accessor, UniverInstanceType.UNIVER_SHEET),
+		disabled$: getCurrentRangeDisable$(accessor, {
+			workbookTypes: [WorkbookEditablePermission],
+			worksheetTypes: [WorksheetSetCellStylePermission, WorksheetEditPermission],
+			rangeTypes: [RangeProtectionPermissionEditPoint]
+		})
+	};
+}
+
+//#endregion
+//#region src/menu/quick-insert.menu.ts
+const DATA_BAR_PARAMS = {
+	min: { type: CFValueType.min },
+	max: { type: CFValueType.max },
+	nativeColor: defaultDataBarNativeColor,
+	positiveColor: defaultDataBarPositiveColor,
+	isGradient: false,
+	isShowValue: true
+};
+function createIconSetParams(iconType) {
+	return {
+		config: [
+			{
+				iconType,
+				iconId: "0",
+				operator: CFNumberOperator.greaterThanOrEqual,
+				value: {
+					type: CFValueType.percent,
+					value: 67
+				}
+			},
+			{
+				iconType,
+				iconId: "1",
+				operator: CFNumberOperator.greaterThanOrEqual,
+				value: {
+					type: CFValueType.percent,
+					value: 33
+				}
+			},
+			{
+				iconType,
+				iconId: "2",
+				operator: CFNumberOperator.lessThanOrEqual,
+				value: {
+					type: CFValueType.percent,
+					value: Number.MAX_SAFE_INTEGER
+				}
+			}
+		],
+		isShowValue: true
+	};
+}
+function getQuickConditionalFormattingMenuState(accessor) {
+	return {
+		hidden$: getMenuHiddenObservable(accessor, UniverInstanceType.UNIVER_SHEET),
+		disabled$: getCurrentRangeDisable$(accessor, {
+			workbookTypes: [WorkbookEditablePermission],
+			worksheetTypes: [WorksheetSetCellStylePermission, WorksheetEditPermission],
+			rangeTypes: [RangeProtectionPermissionEditPoint]
+		})
+	};
+}
+function quickDataBarMenuFactory(accessor) {
+	return {
+		id: AddDataBarConditionalRuleCommand.id,
+		type: MenuItemType.BUTTON_SELECTOR,
+		icon: "DataBarIcon",
+		tooltip: "sheets-conditional-formatting-ui.ruleType.dataBar",
+		params: DATA_BAR_PARAMS,
+		selectionsCommandId: AddDataBarConditionalRuleCommand.id,
+		selections: [
+			{
+				label: "sheets-conditional-formatting-ui.ruleType.dataBar",
+				value: CFRuleType.dataBar,
+				params: DATA_BAR_PARAMS
+			},
+			{
+				id: OpenConditionalFormattingOperator.id,
+				label: "sheets-conditional-formatting-ui.menu.createConditionalFormatting",
+				value: 7,
+				params: { value: 7 }
+			},
+			{
+				id: ClearRangeCfCommand.id,
+				label: "sheets-conditional-formatting-ui.menu.clearRangeRules",
+				value: CFRuleType.dataBar,
+				params: { types: [CFRuleType.dataBar] }
+			}
+		],
+		...getQuickConditionalFormattingMenuState(accessor)
+	};
+}
+function quickIconSetMenuFactory(accessor) {
+	const ratingParams = createIconSetParams(IIconSetType.threeStars);
+	return {
+		id: AddIconSetConditionalRuleCommand.id,
+		type: MenuItemType.BUTTON_SELECTOR,
+		icon: "RatingIcon",
+		tooltip: "sheets-conditional-formatting-ui.ruleType.iconSet",
+		params: ratingParams,
+		selectionsCommandId: AddIconSetConditionalRuleCommand.id,
+		selections: [
+			{
+				label: "sheets-conditional-formatting-ui.iconSet.rank",
+				value: IIconSetType.threeStars,
+				params: ratingParams
+			},
+			{
+				label: "sheets-conditional-formatting-ui.iconSet.shape",
+				value: IIconSetType.threeTrafficLights1,
+				params: createIconSetParams(IIconSetType.threeTrafficLights1)
+			},
+			{
+				label: "sheets-conditional-formatting-ui.iconSet.direction",
+				value: IIconSetType.threeArrows,
+				params: createIconSetParams(IIconSetType.threeArrows)
+			},
+			{
+				id: OpenConditionalFormattingOperator.id,
+				label: "sheets-conditional-formatting-ui.menu.createConditionalFormatting",
+				value: 8,
+				params: { value: 8 }
+			},
+			{
+				id: ClearRangeCfCommand.id,
+				label: "sheets-conditional-formatting-ui.menu.clearRangeRules",
+				value: CFRuleType.iconSet,
+				params: { types: [CFRuleType.iconSet] }
+			}
+		],
+		...getQuickConditionalFormattingMenuState(accessor)
+	};
+}
+
+//#endregion
+//#region src/menu/schema.ts
+const menuSchema = {
+	[RibbonInsertGroup.CELL]: {
+		[AddDataBarConditionalRuleCommand.id]: {
+			order: 3,
+			menuItemFactory: quickDataBarMenuFactory
+		},
+		[AddIconSetConditionalRuleCommand.id]: {
+			order: 4,
+			menuItemFactory: quickIconSetMenuFactory
+		}
+	},
+	[RibbonDataGroup.RULES]: { [OpenConditionalFormattingOperator.id]: {
+		order: 1,
+		menuItemFactory: FactoryManageConditionalFormattingRule
+	} }
+};
+
 //#endregion
 //#region package.json
 var name = "@univerjs/sheets-conditional-formatting-ui";
-var version = "1.0.0-alpha.2";
+var version = "1.0.0-alpha.4";
 
 //#endregion
 //#region src/config/config.ts
@@ -1166,24 +1503,68 @@ let SheetsCfRenderController = class SheetsCfRenderController extends Disposable
 		_defineProperty(this, "_ruleChangeCacheMap", /* @__PURE__ */ new Map());
 		this._initViewModelInterceptor();
 		this._initSkeleton();
+		queueMicrotask(() => this._markActiveSheetRulesDirty());
 		this.disposeWithMe(() => {
 			this._ruleChangeCacheMap.clear();
 		});
 	}
-	_markDirtySkeleton() {
-		var _this$_renderManagerS, _this$_renderManagerS2;
-		const unitId = this._univerInstanceService.getCurrentUnitOfType(UniverInstanceType.UNIVER_SHEET).getUnitId();
-		(_this$_renderManagerS = this._renderManagerService.getRenderById(unitId)) === null || _this$_renderManagerS === void 0 || _this$_renderManagerS.with(SheetSkeletonManagerService).reCalculate();
-		(_this$_renderManagerS2 = this._renderManagerService.getRenderById(unitId)) === null || _this$_renderManagerS2 === void 0 || (_this$_renderManagerS2 = _this$_renderManagerS2.mainComponent) === null || _this$_renderManagerS2 === void 0 || _this$_renderManagerS2.makeDirty();
+	_collectDirtyRanges(items, unitId, subUnitId) {
+		const ranges = [];
+		items.forEach((item) => {
+			var _item$rule, _rule$ranges;
+			if (item.unitId !== unitId || item.subUnitId !== subUnitId) return;
+			const rule = (_item$rule = item.rule) !== null && _item$rule !== void 0 ? _item$rule : item.cfId ? this._conditionalFormattingRuleModel.getRule(unitId, subUnitId, item.cfId) : null;
+			if (rule === null || rule === void 0 || (_rule$ranges = rule.ranges) === null || _rule$ranges === void 0 ? void 0 : _rule$ranges.length) ranges.push(...rule.ranges);
+		});
+		return ranges;
+	}
+	_intersectDirtyRangesWithRenderedRange(ranges, renderedRange) {
+		if (!renderedRange || renderedRange.startRow < 0 || renderedRange.startColumn < 0) return [];
+		return ranges.map((range) => ({
+			startRow: Math.max(range.startRow, renderedRange.startRow),
+			endRow: Math.min(range.endRow, renderedRange.endRow),
+			startColumn: Math.max(range.startColumn, renderedRange.startColumn),
+			endColumn: Math.min(range.endColumn, renderedRange.endColumn)
+		})).filter((range) => range.startRow <= range.endRow && range.startColumn <= range.endColumn);
+	}
+	_markDirtySkeleton(items) {
+		var _render$mainComponent;
+		const workbook = this._univerInstanceService.getCurrentUnitOfType(UniverInstanceType.UNIVER_SHEET);
+		const worksheet = workbook === null || workbook === void 0 ? void 0 : workbook.getActiveSheet();
+		if (!workbook || !worksheet) return;
+		const unitId = workbook.getUnitId();
+		const subUnitId = worksheet.getSheetId();
+		const render = this._renderManagerService.getRenderById(unitId);
+		const skeletonManagerService = render === null || render === void 0 ? void 0 : render.with(SheetSkeletonManagerService);
+		const currentSkeleton = skeletonManagerService === null || skeletonManagerService === void 0 ? void 0 : skeletonManagerService.getCurrentSkeleton();
+		const dirtyRanges = this._intersectDirtyRangesWithRenderedRange(this._collectDirtyRanges(items, unitId, subUnitId), currentSkeleton === null || currentSkeleton === void 0 ? void 0 : currentSkeleton.rowColumnSegment);
+		if (dirtyRanges.length) currentSkeleton === null || currentSkeleton === void 0 || currentSkeleton.resetRangeCache(dirtyRanges);
+		skeletonManagerService === null || skeletonManagerService === void 0 || skeletonManagerService.reCalculate();
+		render === null || render === void 0 || (_render$mainComponent = render.mainComponent) === null || _render$mainComponent === void 0 || _render$mainComponent.makeDirty();
+	}
+	_markActiveSheetRulesDirty() {
+		var _this$_conditionalFor;
+		const workbook = this._univerInstanceService.getCurrentUnitOfType(UniverInstanceType.UNIVER_SHEET);
+		const worksheet = workbook === null || workbook === void 0 ? void 0 : workbook.getActiveSheet();
+		if (!workbook || !worksheet) return;
+		const unitId = workbook.getUnitId();
+		const subUnitId = worksheet.getSheetId();
+		const rules = (_this$_conditionalFor = this._conditionalFormattingRuleModel.getSubunitRules(unitId, subUnitId)) !== null && _this$_conditionalFor !== void 0 ? _this$_conditionalFor : [];
+		if (!rules.length) return;
+		this._markDirtySkeleton(rules.map((rule) => ({
+			rule,
+			subUnitId,
+			unitId
+		})));
 	}
 	_initSkeleton() {
-		this.disposeWithMe(merge$1(this._conditionalFormattingRuleModel.$ruleChange, this._conditionalFormattingViewModel.markDirty$).pipe(bufferTime(16), filter((v) => !!v.length), filter((v) => {
+		this.disposeWithMe(merge$1(this._conditionalFormattingRuleModel.$ruleChange, this._conditionalFormattingViewModel.markDirty$).pipe(bufferTime(16), filter$1((v) => !!v.length), filter$1((v) => {
 			const workbook = this._univerInstanceService.getCurrentUnitOfType(UniverInstanceType.UNIVER_SHEET);
 			if (!workbook) return false;
 			const worksheet = workbook.getActiveSheet();
 			if (!worksheet) return false;
 			return v.some((item) => item.unitId === workbook.getUnitId() && item.subUnitId === worksheet.getSheetId());
-		})).subscribe(() => this._markDirtySkeleton()));
+		})).subscribe((items) => this._markDirtySkeleton(items)));
 	}
 	_initViewModelInterceptor() {
 		this.disposeWithMe(this._sheetInterceptorService.intercept(INTERCEPTOR_POINT.CELL_CONTENT, {
@@ -1249,6 +1630,7 @@ let UniverSheetsConditionalFormattingMobileUIPlugin = class UniverSheetsConditio
 			AddAverageCfCommand,
 			AddColorScaleConditionalRuleCommand,
 			AddDataBarConditionalRuleCommand,
+			AddIconSetConditionalRuleCommand,
 			AddDuplicateValuesCfCommand,
 			AddNumberCfCommand,
 			AddRankCfCommand,
@@ -1879,11 +2261,10 @@ const ColorPicker$1 = (props) => {
 
 //#endregion
 //#region src/views/Preview.tsx
-const Preview = (props) => {
-	const rule = props.rule;
-	if (!rule) return null;
+function Preview(props) {
+	const { rule } = props;
 	const colorList = useMemo(() => {
-		if (rule.type === CFRuleType.colorScale) {
+		if ((rule === null || rule === void 0 ? void 0 : rule.type) === CFRuleType.colorScale) {
 			const config = rule.config.map((c, index) => ({
 				color: new ColorKit(c.color),
 				value: index
@@ -1894,11 +2275,12 @@ const Preview = (props) => {
 		return null;
 	}, [rule]);
 	const iconSet = useMemo(() => {
-		if (rule.type === CFRuleType.iconSet) return rule.config.map((item) => {
+		if ((rule === null || rule === void 0 ? void 0 : rule.type) === CFRuleType.iconSet) return rule.config.map((item) => {
 			const iconList = iconMap[item.iconType];
 			return iconList && iconList[Number(item.iconId)];
 		});
 	}, [rule]);
+	if (!rule) return null;
 	const previewClassName = "univer-pointer-events-none univer-flex univer-h-5 univer-min-w-[72px] univer-items-center univer-justify-center univer-text-xs";
 	switch (rule.type) {
 		case CFRuleType.dataBar: {
@@ -1964,7 +2346,7 @@ const Preview = (props) => {
 			});
 		}
 	}
-};
+}
 
 //#endregion
 //#region src/views/panel/rule-edit/styles.ts
@@ -1987,8 +2369,8 @@ const previewClassName = clsx("univer-mt-5 univer-px-1 univer-py-2 univer-rounde
 
 //#endregion
 //#region src/views/panel/rule-edit/ColorScale.tsx
-const createOptionItem$2 = (text, localeService) => ({
-	label: localeService.t(`sheets-conditional-formatting-ui.valueType.${text}`),
+const createOptionItem$2 = (text) => ({
+	label: `sheets-conditional-formatting-ui.valueType.${text}`,
 	value: text
 });
 const TextInput$1 = (props) => {
@@ -2048,15 +2430,24 @@ const ColorScaleStyleEditor = (props) => {
 	const { interceptorManager } = props;
 	const localeService = useDependency(LocaleService);
 	const rule = ((_props$rule = props.rule) === null || _props$rule === void 0 ? void 0 : _props$rule.type) === CFRuleType.colorScale ? props.rule : void 0;
-	const commonOptions = [
-		createOptionItem$2(CFValueType.num, localeService),
-		createOptionItem$2(CFValueType.percent, localeService),
-		createOptionItem$2(CFValueType.percentile, localeService),
-		createOptionItem$2(CFValueType.formula, localeService)
+	const commonOptionDefinitions = [
+		createOptionItem$2(CFValueType.num),
+		createOptionItem$2(CFValueType.percent),
+		createOptionItem$2(CFValueType.percentile),
+		createOptionItem$2(CFValueType.formula)
 	];
-	const minOptions = [createOptionItem$2(CFValueType.min, localeService), ...commonOptions];
-	const medianOptions = [createOptionItem$2("none", localeService), ...commonOptions];
-	const maxOptions = [createOptionItem$2(CFValueType.max, localeService), ...commonOptions];
+	const minOptions = [createOptionItem$2(CFValueType.min), ...commonOptionDefinitions].map((option) => ({
+		...option,
+		label: localeService.t(option.label)
+	}));
+	const medianOptions = [createOptionItem$2("none"), ...commonOptionDefinitions].map((option) => ({
+		...option,
+		label: localeService.t(option.label)
+	}));
+	const maxOptions = [createOptionItem$2(CFValueType.max), ...commonOptionDefinitions].map((option) => ({
+		...option,
+		label: localeService.t(option.label)
+	}));
 	const [minType, setMinType] = useState(() => {
 		var _rule$config$;
 		const defaultV = CFValueType.min;
@@ -2202,7 +2593,7 @@ const ColorScaleStyleEditor = (props) => {
 			children: localeService.t("sheets-conditional-formatting-ui.valueType.min")
 		}),
 		/* @__PURE__ */ jsxs("div", {
-			className: "univer-mt-3 univer-flex univer-h-8 univer-items-center",
+			className: "univer-mt-3 univer-flex univer-h-8 univer-items-center univer-gap-2",
 			children: [
 				/* @__PURE__ */ jsx(Select, {
 					className: "univer-flex-shrink-0",
@@ -2270,7 +2661,7 @@ const ColorScaleStyleEditor = (props) => {
 			children: localeService.t("sheets-conditional-formatting-ui.panel.medianValue")
 		}),
 		/* @__PURE__ */ jsxs("div", {
-			className: "univer-mt-3 univer-flex univer-h-8 univer-items-center",
+			className: "univer-mt-3 univer-flex univer-h-8 univer-items-center univer-gap-2",
 			children: [
 				/* @__PURE__ */ jsx(Select, {
 					className: "univer-flex-shrink-0",
@@ -2338,7 +2729,7 @@ const ColorScaleStyleEditor = (props) => {
 			children: localeService.t("sheets-conditional-formatting-ui.valueType.max")
 		}),
 		/* @__PURE__ */ jsxs("div", {
-			className: "univer-mt-3 univer-flex univer-h-8 univer-items-center",
+			className: "univer-mt-3 univer-flex univer-h-8 univer-items-center univer-gap-2",
 			children: [
 				/* @__PURE__ */ jsx(Select, {
 					className: "univer-flex-shrink-0",
@@ -2363,7 +2754,6 @@ const ColorScaleStyleEditor = (props) => {
 				}),
 				/* @__PURE__ */ jsx(TextInput$1, {
 					id: "max",
-					className: "univer-ml-1",
 					value: maxValue,
 					type: maxType,
 					onChange: (v) => {
@@ -2382,7 +2772,6 @@ const ColorScaleStyleEditor = (props) => {
 					}
 				}),
 				/* @__PURE__ */ jsx(ColorPicker$1, {
-					className: "univer-ml-1",
 					color: maxColor,
 					onChange: (v) => {
 						setMaxColor(v);
@@ -2406,8 +2795,8 @@ const ColorScaleStyleEditor = (props) => {
 
 //#endregion
 //#region src/views/panel/rule-edit/DataBar.tsx
-const createOptionItem$1 = (text, localeService) => ({
-	label: localeService.t(`sheets-conditional-formatting-ui.valueType.${text}`),
+const createOptionItem$1 = (text) => ({
+	label: `sheets-conditional-formatting-ui.valueType.${text}`,
 	value: text
 });
 const InputText = (props) => {
@@ -2436,7 +2825,7 @@ const InputText = (props) => {
 	if (type === CFValueType.formula) {
 		const v = String(_value.current).startsWith("=") ? String(_value.current) || "" : "=";
 		return /* @__PURE__ */ jsx("div", {
-			className: "univer-ml-3 univer-w-full",
+			className: "univer-w-full",
 			children: /* @__PURE__ */ jsx(FormulaEditor, {
 				ref: formulaEditorRef,
 				className: clsx("univer-box-border univer-h-8 univer-w-full univer-cursor-pointer univer-items-center univer-rounded-lg univer-bg-white univer-pt-2 univer-transition-colors hover:univer-border-primary-600 dark:!univer-bg-gray-700 dark:!univer-text-white [&>div:first-child]:univer-px-2.5 [&>div]:univer-h-5 [&>div]:univer-ring-transparent", borderClassName),
@@ -2482,14 +2871,20 @@ const DataBarStyleEditor = (props) => {
 		if (!rule) return defaultDataBarNativeColor;
 		return ((_rule$config3 = rule.config) === null || _rule$config3 === void 0 ? void 0 : _rule$config3.nativeColor) || defaultDataBarNativeColor;
 	});
-	const commonOptions = [
-		createOptionItem$1(CFValueType.num, localeService),
-		createOptionItem$1(CFValueType.percent, localeService),
-		createOptionItem$1(CFValueType.percentile, localeService),
-		createOptionItem$1(CFValueType.formula, localeService)
+	const commonOptionDefinitions = [
+		createOptionItem$1(CFValueType.num),
+		createOptionItem$1(CFValueType.percent),
+		createOptionItem$1(CFValueType.percentile),
+		createOptionItem$1(CFValueType.formula)
 	];
-	const minOptions = [createOptionItem$1(CFValueType.min, localeService), ...commonOptions];
-	const maxOptions = [createOptionItem$1(CFValueType.max, localeService), ...commonOptions];
+	const minOptions = [createOptionItem$1(CFValueType.min), ...commonOptionDefinitions].map((option) => ({
+		...option,
+		label: localeService.t(option.label)
+	}));
+	const maxOptions = [createOptionItem$1(CFValueType.max), ...commonOptionDefinitions].map((option) => ({
+		...option,
+		label: localeService.t(option.label)
+	}));
 	const [minValueType, setMinValueType] = useState(() => {
 		var _rule$config4;
 		const defaultV = minOptions[0].value;
@@ -2596,7 +2991,7 @@ const DataBarStyleEditor = (props) => {
 		});
 	};
 	const isShowInput = (type) => {
-		return commonOptions.map((item) => item.value).includes(type);
+		return commonOptionDefinitions.map((item) => item.value).includes(type);
 	};
 	return /* @__PURE__ */ jsxs("div", { children: [
 		/* @__PURE__ */ jsx("div", {
@@ -2620,7 +3015,7 @@ const DataBarStyleEditor = (props) => {
 			className: "univer-mt-3 univer-text-sm univer-text-gray-600 dark:!univer-text-gray-200",
 			children: localeService.t("sheets-conditional-formatting-ui.panel.fillType")
 		}), /* @__PURE__ */ jsxs("div", {
-			className: "univer-ml-1 univer-mt-3 univer-flex univer-items-center",
+			className: "univer-mt-3 univer-flex univer-items-center univer-gap-3",
 			children: [/* @__PURE__ */ jsxs(RadioGroup, {
 				value: isGradient,
 				onChange: (v) => {
@@ -2650,7 +3045,7 @@ const DataBarStyleEditor = (props) => {
 					})
 				})]
 			}), /* @__PURE__ */ jsxs("div", {
-				className: "univer-ml-6 univer-flex univer-items-center univer-text-xs",
+				className: "univer-flex univer-items-center univer-text-xs",
 				children: [/* @__PURE__ */ jsx(Checkbox, {
 					checked: !isShowValue,
 					onChange: (v) => {
@@ -2673,7 +3068,7 @@ const DataBarStyleEditor = (props) => {
 			className: "univer-mt-3 univer-text-sm univer-text-gray-600 dark:!univer-text-gray-200",
 			children: localeService.t("sheets-conditional-formatting-ui.panel.colorSet")
 		}), /* @__PURE__ */ jsxs("div", {
-			className: "univer-ml-1 univer-mt-3 univer-flex univer-items-center",
+			className: "univer-mt-3 univer-flex univer-items-center univer-gap-2",
 			children: [/* @__PURE__ */ jsxs("div", {
 				className: "univer-flex univer-items-center",
 				children: [/* @__PURE__ */ jsx("div", {
@@ -2684,7 +3079,7 @@ const DataBarStyleEditor = (props) => {
 					onChange: handleNativeColorChange
 				})]
 			}), /* @__PURE__ */ jsxs("div", {
-				className: "univer-ml-3 univer-flex univer-items-center",
+				className: "univer-flex univer-items-center univer-gap-3",
 				children: [/* @__PURE__ */ jsx("div", {
 					className: "univer-text-xs",
 					children: localeService.t("sheets-conditional-formatting-ui.panel.positive")
@@ -2700,7 +3095,7 @@ const DataBarStyleEditor = (props) => {
 				children: localeService.t("sheets-conditional-formatting-ui.valueType.min")
 			}),
 			/* @__PURE__ */ jsxs("div", {
-				className: "univer-mt-3 univer-flex univer-items-center",
+				className: "univer-mt-3 univer-flex univer-items-center univer-gap-2",
 				children: [/* @__PURE__ */ jsx(Select, {
 					className: "univer-w-1/2 univer-flex-shrink-0",
 					options: minOptions,
@@ -2722,7 +3117,6 @@ const DataBarStyleEditor = (props) => {
 					}
 				}), /* @__PURE__ */ jsx(InputText, {
 					id: "min",
-					className: "univer-ml-3",
 					disabled: !isShowInput(minValueType),
 					type: minValueType,
 					value: minValue,
@@ -2746,7 +3140,7 @@ const DataBarStyleEditor = (props) => {
 				children: localeService.t("sheets-conditional-formatting-ui.valueType.max")
 			}),
 			/* @__PURE__ */ jsxs("div", {
-				className: "univer-mt-3 univer-flex univer-items-center",
+				className: "univer-mt-3 univer-flex univer-items-center univer-gap-2",
 				children: [/* @__PURE__ */ jsx(Select, {
 					className: "univer-w-1/2 univer-flex-shrink-0",
 					options: maxOptions,
@@ -2767,7 +3161,6 @@ const DataBarStyleEditor = (props) => {
 						});
 					}
 				}), /* @__PURE__ */ jsx(InputText, {
-					className: "univer-ml-3",
 					disabled: !isShowInput(maxValueType),
 					id: "max",
 					type: maxValueType,
@@ -3013,8 +3406,14 @@ const WrapperError = (props) => {
 
 //#endregion
 //#region src/views/panel/rule-edit/HighlightCell.tsx
-const createOptionItem = (text, localeService) => ({
-	label: localeService.t(`sheets-conditional-formatting-ui.operator.${text}`),
+const textOperators = new Set(Object.values(CFTextOperator));
+const numberOperators = new Set(Object.values(CFNumberOperator));
+const timePeriodOperators = new Set(Object.values(CFTimePeriodOperator));
+const isTextOperator = (value) => typeof value === "string" && textOperators.has(value);
+const isNumberOperator = (value) => typeof value === "string" && numberOperators.has(value);
+const isTimePeriodOperator = (value) => typeof value === "string" && timePeriodOperators.has(value);
+const createOptionItem = (text) => ({
+	label: `sheets-conditional-formatting-ui.operator.${text}`,
 	value: text
 });
 function HighlightCellInput(props) {
@@ -3175,42 +3574,43 @@ function HighlightCellInput(props) {
 	}
 	return null;
 }
-const getOperatorOptions = (type, localeService) => {
+const getOperatorOptions = (type) => {
 	switch (type) {
 		case CFSubRuleType.text: return [
-			createOptionItem(CFTextOperator.containsText, localeService),
-			createOptionItem(CFTextOperator.notContainsText, localeService),
-			createOptionItem(CFTextOperator.beginsWith, localeService),
-			createOptionItem(CFTextOperator.endsWith, localeService),
-			createOptionItem(CFTextOperator.equal, localeService),
-			createOptionItem(CFTextOperator.notEqual, localeService),
-			createOptionItem(CFTextOperator.containsBlanks, localeService),
-			createOptionItem(CFTextOperator.notContainsBlanks, localeService),
-			createOptionItem(CFTextOperator.containsErrors, localeService),
-			createOptionItem(CFTextOperator.notContainsErrors, localeService)
+			createOptionItem(CFTextOperator.containsText),
+			createOptionItem(CFTextOperator.notContainsText),
+			createOptionItem(CFTextOperator.beginsWith),
+			createOptionItem(CFTextOperator.endsWith),
+			createOptionItem(CFTextOperator.equal),
+			createOptionItem(CFTextOperator.notEqual),
+			createOptionItem(CFTextOperator.containsBlanks),
+			createOptionItem(CFTextOperator.notContainsBlanks),
+			createOptionItem(CFTextOperator.containsErrors),
+			createOptionItem(CFTextOperator.notContainsErrors)
 		];
 		case CFSubRuleType.number: return [
-			createOptionItem(CFNumberOperator.between, localeService),
-			createOptionItem(CFNumberOperator.notBetween, localeService),
-			createOptionItem(CFNumberOperator.equal, localeService),
-			createOptionItem(CFNumberOperator.notEqual, localeService),
-			createOptionItem(CFNumberOperator.greaterThan, localeService),
-			createOptionItem(CFNumberOperator.greaterThanOrEqual, localeService),
-			createOptionItem(CFNumberOperator.lessThan, localeService),
-			createOptionItem(CFNumberOperator.lessThanOrEqual, localeService)
+			createOptionItem(CFNumberOperator.between),
+			createOptionItem(CFNumberOperator.notBetween),
+			createOptionItem(CFNumberOperator.equal),
+			createOptionItem(CFNumberOperator.notEqual),
+			createOptionItem(CFNumberOperator.greaterThan),
+			createOptionItem(CFNumberOperator.greaterThanOrEqual),
+			createOptionItem(CFNumberOperator.lessThan),
+			createOptionItem(CFNumberOperator.lessThanOrEqual)
 		];
 		case CFSubRuleType.timePeriod: return [
-			createOptionItem(CFTimePeriodOperator.yesterday, localeService),
-			createOptionItem(CFTimePeriodOperator.today, localeService),
-			createOptionItem(CFTimePeriodOperator.tomorrow, localeService),
-			createOptionItem(CFTimePeriodOperator.last7Days, localeService),
-			createOptionItem(CFTimePeriodOperator.lastWeek, localeService),
-			createOptionItem(CFTimePeriodOperator.thisWeek, localeService),
-			createOptionItem(CFTimePeriodOperator.nextWeek, localeService),
-			createOptionItem(CFTimePeriodOperator.lastMonth, localeService),
-			createOptionItem(CFTimePeriodOperator.thisMonth, localeService),
-			createOptionItem(CFTimePeriodOperator.nextMonth, localeService)
+			createOptionItem(CFTimePeriodOperator.yesterday),
+			createOptionItem(CFTimePeriodOperator.today),
+			createOptionItem(CFTimePeriodOperator.tomorrow),
+			createOptionItem(CFTimePeriodOperator.last7Days),
+			createOptionItem(CFTimePeriodOperator.lastWeek),
+			createOptionItem(CFTimePeriodOperator.thisWeek),
+			createOptionItem(CFTimePeriodOperator.nextWeek),
+			createOptionItem(CFTimePeriodOperator.lastMonth),
+			createOptionItem(CFTimePeriodOperator.thisMonth),
+			createOptionItem(CFTimePeriodOperator.nextMonth)
 		];
+		default: return [];
 	}
 };
 const HighlightCellStyleEditor = (props) => {
@@ -3245,77 +3645,74 @@ const HighlightCellStyleEditor = (props) => {
 			label: localeService.t("sheets-conditional-formatting-ui.subRuleType.uniqueValues")
 		}
 	];
-	const operatorOptions = useMemo(() => getOperatorOptions(subType, localeService), [subType]);
+	const operatorOptions = useMemo(() => getOperatorOptions(subType).map((option) => ({
+		...option,
+		label: localeService.t(option.label)
+	})), [localeService, subType]);
 	const [operator, setOperator] = useState(() => {
-		const defaultV = operatorOptions ? operatorOptions[0].value : void 0;
+		var _operatorOptions$;
+		const defaultV = (_operatorOptions$ = operatorOptions[0]) === null || _operatorOptions$ === void 0 ? void 0 : _operatorOptions$.value;
 		if (!rule) return defaultV;
-		return rule.operator || defaultV;
+		return "operator" in rule ? rule.operator : defaultV;
 	});
 	const [value, setValue] = useState(() => {
-		var _value2;
 		const defaultV = "";
 		if (!rule) return defaultV;
-		return (_value2 = rule.value) !== null && _value2 !== void 0 ? _value2 : createDefaultValue(rule.subType, rule.operator);
+		if ("value" in rule && rule.value !== void 0) return rule.value;
+		return "operator" in rule ? createDefaultValue(rule.subType, rule.operator) : defaultV;
 	});
 	const [style, setStyle] = useState({});
 	const getResult = useMemo(() => (option) => {
-		var _option$subType4, _option$operator4, _option$style4;
-		switch (option.subType || subType) {
-			case CFSubRuleType.text:
-				if ([
-					CFTextOperator.beginsWith,
-					CFTextOperator.endsWith,
-					CFTextOperator.containsText,
-					CFTextOperator.notContainsText,
-					CFTextOperator.equal,
-					CFTextOperator.notEqual
-				].includes(operator)) {
-					var _option$subType, _option$operator, _option$style, _option$value;
-					return {
-						type: CFRuleType.highlightCell,
-						subType: (_option$subType = option.subType) !== null && _option$subType !== void 0 ? _option$subType : subType,
-						operator: (_option$operator = option.operator) !== null && _option$operator !== void 0 ? _option$operator : operator,
-						style: (_option$style = option.style) !== null && _option$style !== void 0 ? _option$style : style,
-						value: (_option$value = option.value) !== null && _option$value !== void 0 ? _option$value : value
-					};
-				}
-				break;
-			case CFSubRuleType.number:
-				if ([
-					CFNumberOperator.equal,
-					CFNumberOperator.notEqual,
-					CFNumberOperator.greaterThan,
-					CFNumberOperator.greaterThanOrEqual,
-					CFNumberOperator.lessThan,
-					CFNumberOperator.lessThanOrEqual
-				].includes(operator)) {
-					var _option$subType2, _option$operator2, _option$style2, _option$value2;
-					return {
-						type: CFRuleType.highlightCell,
-						subType: (_option$subType2 = option.subType) !== null && _option$subType2 !== void 0 ? _option$subType2 : subType,
-						operator: (_option$operator2 = option.operator) !== null && _option$operator2 !== void 0 ? _option$operator2 : operator,
-						style: (_option$style2 = option.style) !== null && _option$style2 !== void 0 ? _option$style2 : style,
-						value: (_option$value2 = option.value) !== null && _option$value2 !== void 0 ? _option$value2 : value
-					};
-				}
-				if ([CFNumberOperator.between, CFNumberOperator.notBetween].includes(operator)) {
-					var _option$subType3, _option$operator3, _option$style3, _option$value3;
+		var _option$subType, _option$operator, _option$value, _option$style;
+		const nextSubType = (_option$subType = option.subType) !== null && _option$subType !== void 0 ? _option$subType : subType;
+		const nextOperator = (_option$operator = option.operator) !== null && _option$operator !== void 0 ? _option$operator : operator;
+		const nextValue = (_option$value = option.value) !== null && _option$value !== void 0 ? _option$value : value;
+		const nextStyle = (_option$style = option.style) !== null && _option$style !== void 0 ? _option$style : style;
+		switch (nextSubType) {
+			case CFSubRuleType.duplicateValues: return {
+				type: CFRuleType.highlightCell,
+				subType: CFSubRuleType.duplicateValues,
+				style: nextStyle
+			};
+			case CFSubRuleType.uniqueValues: return {
+				type: CFRuleType.highlightCell,
+				subType: CFSubRuleType.uniqueValues,
+				style: nextStyle
+			};
+			case CFSubRuleType.text: return {
+				type: CFRuleType.highlightCell,
+				subType: CFSubRuleType.text,
+				operator: isTextOperator(nextOperator) ? nextOperator : CFTextOperator.containsText,
+				style: nextStyle,
+				value: typeof nextValue === "string" ? nextValue : void 0
+			};
+			case CFSubRuleType.timePeriod: return {
+				type: CFRuleType.highlightCell,
+				subType: CFSubRuleType.timePeriod,
+				operator: isTimePeriodOperator(nextOperator) ? nextOperator : CFTimePeriodOperator.today,
+				style: nextStyle
+			};
+			case CFSubRuleType.number: {
+				const numberOperator = isNumberOperator(nextOperator) ? nextOperator : CFNumberOperator.equal;
+				if (numberOperator === CFNumberOperator.between || numberOperator === CFNumberOperator.notBetween) {
+					const range = Array.isArray(nextValue) ? nextValue : [0, 100];
 					return {
 						type: CFRuleType.highlightCell,
-						subType: (_option$subType3 = option.subType) !== null && _option$subType3 !== void 0 ? _option$subType3 : subType,
-						operator: (_option$operator3 = option.operator) !== null && _option$operator3 !== void 0 ? _option$operator3 : operator,
-						style: (_option$style3 = option.style) !== null && _option$style3 !== void 0 ? _option$style3 : style,
-						value: (_option$value3 = option.value) !== null && _option$value3 !== void 0 ? _option$value3 : value
+						subType: CFSubRuleType.number,
+						operator: numberOperator,
+						style: nextStyle,
+						value: range
 					};
 				}
-				break;
+				return {
+					type: CFRuleType.highlightCell,
+					subType: CFSubRuleType.number,
+					operator: numberOperator,
+					style: nextStyle,
+					value: typeof nextValue === "number" ? nextValue : void 0
+				};
+			}
 		}
-		return {
-			type: CFRuleType.highlightCell,
-			subType: (_option$subType4 = option.subType) !== null && _option$subType4 !== void 0 ? _option$subType4 : subType,
-			operator: (_option$operator4 = option.operator) !== null && _option$operator4 !== void 0 ? _option$operator4 : operator,
-			style: (_option$style4 = option.style) !== null && _option$style4 !== void 0 ? _option$style4 : style
-		};
 	}, [
 		subType,
 		operator,
@@ -3331,9 +3728,13 @@ const HighlightCellStyleEditor = (props) => {
 		if (!typeOptions.some((item) => item.value === subType)) setSubType(typeOptions[0].value);
 	}, [typeOptions]);
 	const onTypeChange = (v) => {
-		const _subType = v;
-		const operatorList = getOperatorOptions(_subType, localeService);
-		const _operator = operatorList && operatorList[0].value;
+		var _typeOptions$find, _operatorList$;
+		const _subType = (_typeOptions$find = typeOptions.find((item) => item.value === v)) === null || _typeOptions$find === void 0 ? void 0 : _typeOptions$find.value;
+		if (!_subType) return;
+		const _operator = (_operatorList$ = getOperatorOptions(_subType).map((option) => ({
+			...option,
+			label: localeService.t(option.label)
+		}))[0]) === null || _operatorList$ === void 0 ? void 0 : _operatorList$.value;
 		setSubType(_subType);
 		setOperator(_operator);
 		_operator && setValue(createDefaultValue(_subType, _operator));
@@ -3343,7 +3744,9 @@ const HighlightCellStyleEditor = (props) => {
 		}));
 	};
 	const onOperatorChange = (v) => {
-		const _operator = v;
+		var _operatorOptions$find;
+		const _operator = (_operatorOptions$find = operatorOptions.find((item) => item.value === v)) === null || _operatorOptions$find === void 0 ? void 0 : _operatorOptions$find.value;
+		if (!_operator) return;
 		setOperator(_operator);
 		onChange(getResult({ operator: _operator }));
 	};
@@ -3351,9 +3754,6 @@ const HighlightCellStyleEditor = (props) => {
 		setValue(value);
 		onChange(getResult({ value }));
 	};
-	const inputRenderKey = useMemo(() => {
-		return `${subType}_${operator}_${Math.random()}`;
-	}, [subType, operator]);
 	return /* @__PURE__ */ jsxs("div", { children: [
 		/* @__PURE__ */ jsx("div", {
 			className: "univer-mt-4 univer-text-sm univer-text-gray-600 dark:!univer-text-gray-200",
@@ -3380,7 +3780,7 @@ const HighlightCellStyleEditor = (props) => {
 			operator,
 			rule,
 			onChange: onInputChange
-		}, inputRenderKey),
+		}, `${subType}_${operator}`),
 		/* @__PURE__ */ jsx("div", {
 			className: previewClassName,
 			children: /* @__PURE__ */ jsx(Preview, { rule: getResult({}) })
@@ -3448,32 +3848,34 @@ const createDefaultConfigItem = (iconType, index, list) => ({
 	iconType,
 	iconId: String(index)
 });
-const IconGroupList = forwardRef((props, ref) => {
-	const { onClick } = props;
+const IconGroupList = (props) => {
+	const { iconType, onClick } = props;
 	const localeService = useDependency(LocaleService);
-	const handleClick = (iconType) => {
-		onClick(iconType);
-	};
 	return /* @__PURE__ */ jsx("div", {
-		ref,
+		dir: useObservable(localeService.direction$, localeService.getDirection()),
 		className: "univer-w-80",
-		children: iconGroup.map((group, index) => {
+		children: iconGroup.map((group) => {
 			return /* @__PURE__ */ jsxs("div", {
-				className: "univer-mb-3",
+				className: "univer-mb-4 last:univer-mb-0",
 				children: [/* @__PURE__ */ jsx("div", {
-					className: "univer-mb-1 univer-text-sm",
+					className: "univer-mb-2 univer-px-1 univer-text-xs univer-font-medium univer-text-gray-500 dark:!univer-text-gray-400",
 					children: localeService.t(group.title)
 				}), /* @__PURE__ */ jsx("div", {
-					className: "univer-flex univer-flex-wrap",
-					children: group.group.map((groupItem) => {
-						return /* @__PURE__ */ jsx("div", {
-							className: "univer-mb-1 univer-flex univer-w-1/2 univer-items-center",
+					className: "univer-grid univer-grid-cols-2 univer-gap-1",
+					children: group.group.map((groupItem, groupItemIndex) => {
+						const selected = groupItem.name === iconType;
+						return /* @__PURE__ */ jsx("button", {
+							type: "button",
+							"aria-label": `${localeService.t(group.title)} ${groupItemIndex + 1}`,
+							"aria-pressed": selected,
+							className: clsx("univer-flex univer-h-8 univer-w-full univer-cursor-pointer univer-items-center univer-rounded-md univer-border-0 univer-bg-transparent univer-px-2 univer-outline-none univer-transition-colors focus-visible:univer-ring-2 focus-visible:univer-ring-primary-600 rtl:univer-flex-row-reverse", selected ? "univer-bg-primary-50 univer-ring-1 univer-ring-primary-600 dark:!univer-bg-primary-900" : "hover:univer-bg-gray-100 dark:hover:!univer-bg-gray-700"),
 							onClick: () => {
-								handleClick(groupItem.name);
+								onClick(groupItem.name);
 							},
-							children: /* @__PURE__ */ jsx("a", {
-								className: "univer-flex univer-cursor-pointer univer-rounded hover:univer-bg-gray-100 dark:hover:!univer-bg-gray-700",
+							children: /* @__PURE__ */ jsx("span", {
+								className: "univer-flex univer-items-center univer-gap-0.5",
 								children: groupItem.list.map((base64, index) => /* @__PURE__ */ jsx("img", {
+									alt: "",
 									className: "univer-size-5",
 									src: base64,
 									draggable: false
@@ -3482,12 +3884,14 @@ const IconGroupList = forwardRef((props, ref) => {
 						}, groupItem.name);
 					})
 				})]
-			}, index);
+			}, group.title);
 		})
 	});
-});
+};
 const IconItemList = (props) => {
 	const { onClick } = props;
+	const localeService = useDependency(LocaleService);
+	const direction = useObservable(localeService.direction$, localeService.getDirection());
 	const list = useMemo(() => {
 		const result = [];
 		for (const key in iconMap) {
@@ -3505,29 +3909,30 @@ const IconItemList = (props) => {
 	const handleClick = (item) => {
 		onClick(item.iconType, item.iconId);
 	};
-	return /* @__PURE__ */ jsxs("div", { children: [/* @__PURE__ */ jsxs("div", {
-		className: "univer-mb-2.5 univer-flex univer-cursor-pointer univer-items-center univer-pl-1",
-		onClick: () => handleClick({
-			iconType: IIconSetType.empty,
-			iconId: "",
-			base64: ""
-		}),
-		children: [/* @__PURE__ */ jsx(SlashDoubleIcon, { className: "univer-size-5" }), /* @__PURE__ */ jsx("span", {
-			className: "univer-ml-2",
-			children: "无单元格图标"
+	return /* @__PURE__ */ jsxs("div", {
+		dir: direction,
+		children: [/* @__PURE__ */ jsx("div", {
+			"data-u-comp": "cf-icon-set-no-icon-option",
+			className: "univer-mb-2.5 univer-flex univer-cursor-pointer univer-items-center univer-gap-2 univer-text-sm",
+			onClick: () => handleClick({
+				iconType: IIconSetType.empty,
+				iconId: "",
+				base64: ""
+			}),
+			children: localeService.t("sheets-conditional-formatting-ui.iconSet.noCellIcon")
+		}), /* @__PURE__ */ jsx("div", {
+			className: "univer-flex univer-w-64 univer-flex-wrap univer-gap-2",
+			children: list.map((item) => /* @__PURE__ */ jsx("div", {
+				className: "univer-mb-2 univer-flex univer-cursor-pointer univer-items-center univer-justify-center univer-rounded hover:univer-bg-gray-100 dark:hover:!univer-bg-gray-700",
+				children: /* @__PURE__ */ jsx("img", {
+					className: "univer-size-5",
+					src: item.base64,
+					draggable: false,
+					onClick: () => handleClick(item)
+				})
+			}, `${item.iconType}_${item.iconId}`))
 		})]
-	}), /* @__PURE__ */ jsx("div", {
-		className: "univer-flex univer-w-64 univer-flex-wrap",
-		children: list.map((item) => /* @__PURE__ */ jsx("div", {
-			className: "univer-mb-2 univer-mr-2 univer-flex univer-cursor-pointer univer-items-center univer-justify-center univer-rounded hover:univer-bg-gray-100 dark:hover:!univer-bg-gray-700",
-			children: /* @__PURE__ */ jsx("img", {
-				className: "univer-size-5",
-				src: item.base64,
-				draggable: false,
-				onClick: () => handleClick(item)
-			})
-		}, `${item.iconType}_${item.iconId}`))
-	})] });
+	});
 };
 const IconSetRuleEdit = (props) => {
 	const { onChange, configList, errorMap = {} } = props;
@@ -3615,7 +4020,8 @@ const IconSetRuleEdit = (props) => {
 						})]
 					}),
 					/* @__PURE__ */ jsxs("div", {
-						className: "univer-mt-3 univer-grid univer-grid-cols-2 univer-gap-4",
+						"data-u-comp": "cf-icon-set-rule-grid",
+						className: "univer-mt-3 univer-grid univer-grid-cols-2 univer-gap-4 rtl:univer-grid-flow-dense",
 						children: [/* @__PURE__ */ jsx("div", {
 							className: "univer-flex univer-items-center",
 							children: /* @__PURE__ */ jsx(Dropdown, {
@@ -3633,7 +4039,7 @@ const IconSetRuleEdit = (props) => {
 										src: icon,
 										className: "univer-size-4",
 										draggable: false
-									}) : /* @__PURE__ */ jsx(SlashDoubleIcon, { className: "univer-size-4" }), /* @__PURE__ */ jsx(MoreDownIcon, {})]
+									}) : /* @__PURE__ */ jsx(SlashDoubleIcon, {}), /* @__PURE__ */ jsx(MoreDownIcon, {})]
 								})
 							})
 						}), !isEnd ? /* @__PURE__ */ jsx(Select, {
@@ -3686,6 +4092,7 @@ const IconSet = (props) => {
 	const { interceptorManager } = props;
 	const rule = ((_props$rule = props.rule) === null || _props$rule === void 0 ? void 0 : _props$rule.type) === CFRuleType.iconSet ? props.rule : void 0;
 	const localeService = useDependency(LocaleService);
+	const direction = useObservable(localeService.direction$, localeService.getDirection());
 	const [errorMap, setErrorMap] = useState({});
 	const [currentIconType, setCurrentIconType] = useState(() => {
 		const defaultV = Object.keys(iconMap)[0];
@@ -3717,7 +4124,8 @@ const IconSet = (props) => {
 	});
 	const previewIcon = useMemo(() => {
 		return /* @__PURE__ */ jsx("div", {
-			className: "univer-flex univer-items-center",
+			"data-u-comp": "cf-icon-set-preview",
+			className: "univer-flex univer-items-center rtl:univer-flex-row-reverse",
 			children: configList.map((item) => {
 				return getIcon(item.iconType, item.iconId);
 			}).map((icon, index) => icon ? /* @__PURE__ */ jsx("img", {
@@ -3817,54 +4225,55 @@ const IconSet = (props) => {
 		});
 		setConfigList([...configList]);
 	};
-	const layoutService = useDependency(ILayoutService);
-	const [iconGroupListEl, setIconGroupListEl] = useState();
-	useScrollYOverContainer(iconGroupListEl, layoutService.rootContainerElement);
-	return /* @__PURE__ */ jsxs("div", { children: [
-		/* @__PURE__ */ jsx("div", {
-			className: "univer-mt-4 univer-text-sm univer-text-gray-600",
-			children: localeService.t("sheets-conditional-formatting-ui.panel.styleRule")
-		}),
-		/* @__PURE__ */ jsx("div", {
-			className: "univer-mt-3",
-			children: /* @__PURE__ */ jsx(Dropdown, {
-				overlay: /* @__PURE__ */ jsx("div", {
-					className: "univer-rounded-lg univer-p-4",
-					children: /* @__PURE__ */ jsx(IconGroupList, {
-						ref: (el) => {
-							!iconGroupListEl && el && setIconGroupListEl(el);
-						},
-						iconType: currentIconType,
-						onClick: handleClickIconList
+	return /* @__PURE__ */ jsxs("div", {
+		"data-u-comp": "cf-icon-set-editor",
+		dir: direction,
+		children: [
+			/* @__PURE__ */ jsx("div", {
+				className: "univer-mt-4 univer-text-sm univer-text-gray-600",
+				children: localeService.t("sheets-conditional-formatting-ui.panel.styleRule")
+			}),
+			/* @__PURE__ */ jsx("div", {
+				className: "univer-mt-3",
+				children: /* @__PURE__ */ jsx(Dropdown, {
+					overlay: /* @__PURE__ */ jsx("div", {
+						dir: direction,
+						className: "univer-rounded-lg univer-p-3",
+						children: /* @__PURE__ */ jsx(IconGroupList, {
+							iconType: currentIconType,
+							onClick: handleClickIconList
+						})
+					}),
+					children: /* @__PURE__ */ jsxs("div", {
+						className: clsx("univer-box-border univer-flex univer-h-8 univer-w-full univer-items-center univer-justify-between univer-rounded-md univer-bg-white univer-px-4 univer-py-2 univer-text-xs univer-text-gray-600 univer-transition-all hover:univer-border-primary-600", borderClassName),
+						children: [previewIcon, /* @__PURE__ */ jsx(MoreDownIcon, {})]
 					})
-				}),
-				children: /* @__PURE__ */ jsxs("div", {
-					className: clsx("univer-box-border univer-flex univer-h-8 univer-w-full univer-items-center univer-justify-between univer-rounded-md univer-bg-white univer-px-4 univer-py-2 univer-text-xs univer-text-gray-600 univer-transition-all hover:univer-border-primary-600", borderClassName),
-					children: [previewIcon, /* @__PURE__ */ jsx(MoreDownIcon, {})]
 				})
+			}),
+			/* @__PURE__ */ jsxs("div", {
+				className: "univer-mt-3 univer-flex univer-items-center univer-gap-2 univer-text-xs rtl:univer-flex-row-reverse rtl:univer-justify-end",
+				children: [/* @__PURE__ */ jsxs("div", {
+					"data-u-comp": "cf-icon-set-option-row",
+					className: "univer-flex univer-items-center univer-text-xs",
+					children: [/* @__PURE__ */ jsx(Checkbox, { onChange: reverseIcon }), localeService.t("sheets-conditional-formatting-ui.iconSet.reverseIconOrder")]
+				}), /* @__PURE__ */ jsxs("div", {
+					"data-u-comp": "cf-icon-set-option-row",
+					className: "univer-flex univer-items-center univer-text-xs",
+					children: [/* @__PURE__ */ jsx(Checkbox, {
+						checked: !isShowValue,
+						onChange: (v) => {
+							setIsShowValue(!v);
+						}
+					}), localeService.t("sheets-conditional-formatting-ui.iconSet.onlyShowIcon")]
+				})]
+			}),
+			/* @__PURE__ */ jsx(IconSetRuleEdit, {
+				errorMap,
+				onChange: handleChange,
+				configList
 			})
-		}),
-		/* @__PURE__ */ jsxs("div", {
-			className: "univer-mt-3 univer-flex univer-items-center univer-text-xs",
-			children: [/* @__PURE__ */ jsxs("div", {
-				className: "univer-flex univer-items-center univer-text-xs",
-				children: [/* @__PURE__ */ jsx(Checkbox, { onChange: reverseIcon }), localeService.t("sheets-conditional-formatting-ui.iconSet.reverseIconOrder")]
-			}), /* @__PURE__ */ jsxs("div", {
-				className: "univer-ml-6 univer-flex univer-items-center univer-text-xs",
-				children: [/* @__PURE__ */ jsx(Checkbox, {
-					checked: !isShowValue,
-					onChange: (v) => {
-						setIsShowValue(!v);
-					}
-				}), localeService.t("sheets-conditional-formatting-ui.iconSet.onlyShowIcon")]
-			})]
-		}),
-		/* @__PURE__ */ jsx(IconSetRuleEdit, {
-			errorMap,
-			onChange: handleChange,
-			configList
-		})
-	] });
+		]
+	});
 };
 
 //#endregion
@@ -3933,34 +4342,36 @@ const RankStyleEditor = (props) => {
 	const [style, setStyle] = useState({});
 	const getResult = (config) => {
 		const { type, isPercent, value, style } = config;
-		if (type === "isNotBottom") return {
-			type: CFRuleType.highlightCell,
-			subType: CFSubRuleType.rank,
-			isPercent,
-			isBottom: false,
-			value,
-			style
-		};
-		if (type === "isBottom") return {
-			type: CFRuleType.highlightCell,
-			subType: CFSubRuleType.rank,
-			isPercent,
-			isBottom: true,
-			value,
-			style
-		};
-		if (type === "greaterThanAverage") return {
-			type: CFRuleType.highlightCell,
-			subType: CFSubRuleType.average,
-			operator: CFNumberOperator.greaterThan,
-			style
-		};
-		if (type === "lessThanAverage") return {
-			type: CFRuleType.highlightCell,
-			subType: CFSubRuleType.average,
-			operator: CFNumberOperator.lessThan,
-			style
-		};
+		switch (type) {
+			case "isNotBottom": return {
+				type: CFRuleType.highlightCell,
+				subType: CFSubRuleType.rank,
+				isPercent,
+				isBottom: false,
+				value,
+				style
+			};
+			case "isBottom": return {
+				type: CFRuleType.highlightCell,
+				subType: CFSubRuleType.rank,
+				isPercent,
+				isBottom: true,
+				value,
+				style
+			};
+			case "greaterThanAverage": return {
+				type: CFRuleType.highlightCell,
+				subType: CFSubRuleType.average,
+				operator: CFNumberOperator.greaterThan,
+				style
+			};
+			case "lessThanAverage": return {
+				type: CFRuleType.highlightCell,
+				subType: CFSubRuleType.average,
+				operator: CFNumberOperator.lessThan,
+				style
+			};
+		}
 	};
 	useEffect(() => {
 		return interceptorManager.intercept(interceptorManager.getInterceptPoints().submit, { handler() {
@@ -3991,17 +4402,21 @@ const RankStyleEditor = (props) => {
 			value: type,
 			options,
 			onChange: (v) => {
-				setType(v);
-				_onChange({
-					type: v,
-					isPercent,
-					value,
-					style
-				});
+				var _options$find;
+				const nextType = (_options$find = options.find((option) => option.value === v)) === null || _options$find === void 0 ? void 0 : _options$find.value;
+				if (nextType) {
+					setType(nextType);
+					_onChange({
+						type: nextType,
+						isPercent,
+						value,
+						style
+					});
+				}
 			}
 		}),
 		["isNotBottom", "isBottom"].includes(type) && /* @__PURE__ */ jsxs("div", {
-			className: "univer-mt-3 univer-flex univer-items-center",
+			className: "univer-mt-3 univer-flex univer-items-center univer-gap-2",
 			children: [/* @__PURE__ */ jsx(InputNumber, {
 				min: 1,
 				max: 1e3,
@@ -4017,7 +4432,7 @@ const RankStyleEditor = (props) => {
 					});
 				}
 			}), /* @__PURE__ */ jsxs("div", {
-				className: "univer-ml-3 univer-flex univer-items-center univer-text-xs",
+				className: "univer-flex univer-items-center univer-text-xs",
 				children: [/* @__PURE__ */ jsx(Checkbox, {
 					checked: isPercent,
 					onChange: (v) => {
@@ -4275,12 +4690,11 @@ const RuleEdit = (props) => {
 			onChange: onStyleChange
 		}),
 		/* @__PURE__ */ jsxs("div", {
-			className: "univer-mt-4 univer-flex univer-justify-end",
+			className: "univer-mt-4 univer-flex univer-justify-end univer-gap-2",
 			children: [/* @__PURE__ */ jsx(Button, {
 				onClick: handleCancel,
 				children: localeService.t("sheets-conditional-formatting-ui.panel.cancel")
 			}), /* @__PURE__ */ jsx(Button, {
-				className: "univer-ml-3",
 				variant: "primary",
 				onClick: handleSubmit,
 				children: localeService.t("sheets-conditional-formatting-ui.panel.submit")
@@ -4324,7 +4738,7 @@ const getRuleDescribe = (rule, localeService) => {
 		}
 	}
 };
-const RuleList = (props) => {
+function RuleList(props) {
 	const { onClick } = props;
 	const conditionalFormattingRuleModel = useDependency(ConditionalFormattingRuleModel);
 	const univerInstanceService = useDependency(IUniverInstanceService);
@@ -4335,12 +4749,9 @@ const RuleList = (props) => {
 	const conditionalFormattingI18nController = useDependency(ConditionalFormattingI18nController);
 	const workbook = useObservable(() => univerInstanceService.getCurrentTypeOfUnit$(UniverInstanceType.UNIVER_SHEET), void 0, void 0, []);
 	const unitId = workbook.getUnitId();
-	const worksheet = workbook.getActiveSheet();
-	if (!worksheet) throw new Error("No active sheet found");
-	const subUnitId = worksheet.getSheetId();
+	const subUnitId = workbook.getActiveSheet().getSheetId();
 	const [currentRuleRanges, setCurrentRuleRanges] = useState([]);
 	const [selectValue, setSelectValue] = useState("2");
-	const [fetchRuleListId, setFetchRuleListId] = useState(0);
 	const [draggingId, setDraggingId] = useState("");
 	const selectOption = [{
 		label: localeService.t("sheets-conditional-formatting-ui.panel.workSheet"),
@@ -4362,25 +4773,8 @@ const RuleList = (props) => {
 		} else if (selectValue === "2") return [...ruleList];
 		return [];
 	};
-	const [ruleList, setRuleList] = useState(getRuleList);
-	useHighlightRange(currentRuleRanges);
-	useEffect(() => {
-		const disposable = commandService.onCommandExecuted((commandInfo) => {
-			if (commandInfo.id === SetWorksheetActiveOperation.id) setFetchRuleListId(Math.random());
-		});
-		return () => disposable.dispose();
-	});
-	useEffect(() => {
-		setRuleList(getRuleList);
-	}, [
-		selectValue,
-		fetchRuleListId,
-		unitId,
-		subUnitId
-	]);
-	useEffect(() => {
-		if (selectValue === "2") return;
-		const subscription = new Observable((commandSubscribe) => {
+	const ruleList = useObservable(() => {
+		const commandEvent$ = new Observable((subscriber) => {
 			const commandList = [
 				SetSelectionsOperation.id,
 				AddConditionalRuleMutation.id,
@@ -4389,29 +4783,25 @@ const RuleList = (props) => {
 				MoveConditionalRuleMutation.id
 			];
 			const disposable = commandService.onCommandExecuted((commandInfo) => {
-				const { id, params } = commandInfo;
-				const unitId = univerInstanceService.getCurrentUnitOfType(UniverInstanceType.UNIVER_SHEET).getUnitId();
-				if (commandList.includes(id) && params.unitId === unitId) commandSubscribe.next(null);
+				var _commandInfo$params;
+				if (commandInfo.id === SetWorksheetActiveOperation.id) {
+					subscriber.next("immediate");
+					return;
+				}
+				const commandUnitId = (_commandInfo$params = commandInfo.params) === null || _commandInfo$params === void 0 ? void 0 : _commandInfo$params.unitId;
+				if (selectValue === "1" && commandList.includes(commandInfo.id) && commandUnitId === unitId) subscriber.next("debounced");
 			});
 			return () => disposable.dispose();
-		}).pipe(debounceTime(16)).subscribe(() => {
-			setRuleList(getRuleList);
-		});
-		return () => {
-			subscription.unsubscribe();
-		};
-	}, [
-		univerInstanceService,
+		}).pipe(share());
+		return merge$1(conditionalFormattingRuleModel.$ruleChange, commandEvent$.pipe(filter((event) => event === "immediate")), commandEvent$.pipe(filter((event) => event === "debounced"), debounceTime(16))).pipe(map(getRuleList), startWith(getRuleList()));
+	}, [], false, [
+		commandService,
+		conditionalFormattingRuleModel,
 		selectValue,
-		unitId,
-		subUnitId
+		subUnitId,
+		unitId
 	]);
-	useEffect(() => {
-		const dispose = conditionalFormattingRuleModel.$ruleChange.subscribe(() => {
-			setFetchRuleListId(Math.random());
-		});
-		return () => dispose.unsubscribe();
-	}, [conditionalFormattingRuleModel]);
+	useHighlightRange(currentRuleRanges);
 	const handleDelete = (rule) => {
 		var _getActiveSheet;
 		const unitId = univerInstanceService.getCurrentUnitOfType(UniverInstanceType.UNIVER_SHEET).getUnitId();
@@ -4563,7 +4953,7 @@ const RuleList = (props) => {
 			]
 		})
 	}) })] });
-};
+}
 
 //#endregion
 //#region src/views/ConditionFormattingPanel.tsx
@@ -4623,180 +5013,15 @@ let ComponentsController = class ComponentsController extends Disposable {
 		this.disposeWithMe(this._componentManager.register(CF_PANEL_KEY, ConditionFormattingPanel));
 	}
 	_registerIcons() {
-		this.disposeWithMe(this._iconManager.register({ ConditionsDoubleIcon }));
+		this.disposeWithMe(this._iconManager.register({
+			ConditionsDoubleIcon,
+			DataBarIcon,
+			RatingIcon
+		}));
 	}
 };
 ComponentsController = __decorate([__decorateParam(0, Inject(ComponentManager)), __decorateParam(1, Inject(IconManager))], ComponentsController);
 
-//#endregion
-//#region src/menu/manage-rule.ts
-const commandList = [
-	SetWorksheetActiveOperation.id,
-	AddConditionalRuleMutation.id,
-	SetConditionalRuleMutation.id,
-	DeleteConditionalRuleMutation.id,
-	MoveConditionalRuleMutation.id
-];
-const commonSelections = [
-	{
-		label: {
-			name: "sheets-conditional-formatting-ui.ruleType.highlightCell",
-			selectable: false
-		},
-		value: 3
-	},
-	{
-		label: {
-			name: "sheets-conditional-formatting-ui.panel.rankAndAverage",
-			selectable: false
-		},
-		value: 4
-	},
-	{
-		label: {
-			name: "sheets-conditional-formatting-ui.ruleType.formula",
-			selectable: false
-		},
-		value: 5
-	},
-	{
-		label: {
-			name: "sheets-conditional-formatting-ui.ruleType.colorScale",
-			selectable: false
-		},
-		value: 6
-	},
-	{
-		label: {
-			name: "sheets-conditional-formatting-ui.ruleType.dataBar",
-			selectable: false
-		},
-		value: 7
-	},
-	{
-		label: {
-			name: "sheets-conditional-formatting-ui.ruleType.iconSet",
-			selectable: false
-		},
-		value: 8
-	},
-	{
-		label: {
-			name: "sheets-conditional-formatting-ui.menu.manageConditionalFormatting",
-			selectable: false
-		},
-		value: 2
-	},
-	{
-		label: {
-			name: "sheets-conditional-formatting-ui.menu.createConditionalFormatting",
-			selectable: false
-		},
-		value: 1
-	},
-	{
-		label: {
-			name: "sheets-conditional-formatting-ui.menu.clearRangeRules",
-			selectable: false
-		},
-		value: 9,
-		disabled: false
-	},
-	{
-		label: {
-			name: "sheets-conditional-formatting-ui.menu.clearWorkSheetRules",
-			selectable: false
-		},
-		value: 10
-	}
-];
-function FactoryManageConditionalFormattingRule(accessor) {
-	const selectionManagerService = accessor.get(SheetsSelectionsService);
-	const commandService = accessor.get(ICommandService);
-	const univerInstanceService = accessor.get(IUniverInstanceService);
-	const conditionalFormattingRuleModel = accessor.get(ConditionalFormattingRuleModel);
-	const clearRangeEnable$ = new Observable((subscriber) => merge$1(selectionManagerService.selectionMoveEnd$, selectionManagerService.selectionSet$, new Observable((commandSubscribe) => {
-		const disposable = commandService.onCommandExecuted((commandInfo) => {
-			var _univerInstanceServic;
-			const { id, params } = commandInfo;
-			const unitId = (_univerInstanceServic = univerInstanceService.getCurrentUnitOfType(UniverInstanceType.UNIVER_SHEET)) === null || _univerInstanceServic === void 0 ? void 0 : _univerInstanceServic.getUnitId();
-			if (commandList.includes(id) && params.unitId === unitId) commandSubscribe.next(null);
-		});
-		return () => disposable.dispose();
-	})).pipe(debounceTime$1(16)).subscribe(() => {
-		var _selectionManagerServ;
-		const ranges = ((_selectionManagerServ = selectionManagerService.getCurrentSelections()) === null || _selectionManagerServ === void 0 ? void 0 : _selectionManagerServ.map((selection) => selection.range)) || [];
-		const workbook = univerInstanceService.getCurrentUnitOfType(UniverInstanceType.UNIVER_SHEET);
-		if (!workbook) return;
-		const worksheet = workbook.getActiveSheet();
-		if (!worksheet) return;
-		const hasPermission = (conditionalFormattingRuleModel.getSubunitRules(workbook.getUnitId(), worksheet.getSheetId()) || []).filter((rule) => rule.ranges.some((ruleRange) => ranges.some((range) => Rectangle.intersects(range, ruleRange)))).map((rule) => rule.ranges).every((ranges) => {
-			return checkRangesEditablePermission(accessor, workbook.getUnitId(), worksheet.getSheetId(), ranges);
-		});
-		subscriber.next(hasPermission);
-	}));
-	const clearSheetEnable$ = new Observable((subscriber) => new Observable((commandSubscribe) => {
-		const disposable = commandService.onCommandExecuted((commandInfo) => {
-			var _univerInstanceServic2;
-			const { id, params } = commandInfo;
-			const unitId = (_univerInstanceServic2 = univerInstanceService.getCurrentUnitOfType(UniverInstanceType.UNIVER_SHEET)) === null || _univerInstanceServic2 === void 0 ? void 0 : _univerInstanceServic2.getUnitId();
-			if (commandList.includes(id) && params.unitId === unitId) commandSubscribe.next(null);
-		});
-		return () => disposable.dispose();
-	}).pipe(debounceTime$1(16)).subscribe(() => {
-		const workbook = univerInstanceService.getCurrentUnitOfType(UniverInstanceType.UNIVER_SHEET);
-		if (!workbook) return;
-		const worksheet = workbook.getActiveSheet();
-		if (!worksheet) return;
-		const allRule = conditionalFormattingRuleModel.getSubunitRules(workbook.getUnitId(), worksheet.getSheetId()) || [];
-		if (!allRule.length) {
-			subscriber.next(false);
-			return false;
-		}
-		const hasPermission = allRule.map((rule) => rule.ranges).every((ranges) => {
-			return checkRangesEditablePermission(accessor, workbook.getUnitId(), worksheet.getSheetId(), ranges);
-		});
-		subscriber.next(hasPermission);
-	}));
-	const selections$ = new Observable((subscriber) => {
-		clearRangeEnable$.subscribe((v) => {
-			const item = commonSelections.find((item) => item.value === 9);
-			if (item) {
-				item.disabled = !v;
-				subscriber.next(commonSelections);
-			}
-		});
-		clearSheetEnable$.subscribe((v) => {
-			const item = commonSelections.find((item) => item.value === 10);
-			if (item) {
-				item.disabled = !v;
-				subscriber.next(commonSelections);
-			}
-		});
-		subscriber.next(commonSelections);
-	});
-	return {
-		id: OpenConditionalFormattingOperator.id,
-		type: MenuItemType.SELECTOR,
-		icon: "ConditionsDoubleIcon",
-		tooltip: "sheets-conditional-formatting-ui.title",
-		selections: selections$,
-		hidden$: getMenuHiddenObservable(accessor, UniverInstanceType.UNIVER_SHEET),
-		disabled$: getCurrentRangeDisable$(accessor, {
-			workbookTypes: [WorkbookEditablePermission],
-			worksheetTypes: [WorksheetSetCellStylePermission, WorksheetEditPermission],
-			rangeTypes: [RangeProtectionPermissionEditPoint]
-		})
-	};
-}
-
-//#endregion
-//#region src/menu/schema.ts
-const menuSchema = { [RibbonDataGroup.RULES]: { [OpenConditionalFormattingOperator.id]: {
-	order: 1,
-	menuItemFactory: FactoryManageConditionalFormattingRule
-} } };
-
 //#endregion
 //#region src/menu/cf.menu.controller.ts
 /**
@@ -4876,6 +5101,7 @@ let UniverSheetsConditionalFormattingUIPlugin = class UniverSheetsConditionalFor
 			AddAverageCfCommand,
 			AddColorScaleConditionalRuleCommand,
 			AddDataBarConditionalRuleCommand,
+			AddIconSetConditionalRuleCommand,
 			AddDuplicateValuesCfCommand,
 			AddNumberCfCommand,
 			AddRankCfCommand,
@@ -4900,4 +5126,4 @@ UniverSheetsConditionalFormattingUIPlugin = __decorate([
 ], UniverSheetsConditionalFormattingUIPlugin);
 
 //#endregion
-export { AddAverageCfCommand, AddColorScaleConditionalRuleCommand, AddDataBarConditionalRuleCommand, AddDuplicateValuesCfCommand, AddNumberCfCommand, AddRankCfCommand, AddTextCfCommand, AddTimePeriodCfCommand, AddUniqueValuesCfCommand, ConditionalFormattingClearController, OpenConditionalFormattingOperator, UniverSheetsConditionalFormattingMobileUIPlugin, UniverSheetsConditionalFormattingUIPlugin };
+export { AddAverageCfCommand, AddColorScaleConditionalRuleCommand, AddDataBarConditionalRuleCommand, AddDuplicateValuesCfCommand, AddIconSetConditionalRuleCommand, AddNumberCfCommand, AddRankCfCommand, AddTextCfCommand, AddTimePeriodCfCommand, AddUniqueValuesCfCommand, ConditionalFormattingClearController, OpenConditionalFormattingOperator, menuSchema as SheetsConditionalFormattingUIMenuSchema, UniverSheetsConditionalFormattingMobileUIPlugin, UniverSheetsConditionalFormattingUIPlugin };