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

package/lib/cjs/index.js

@@ -82,8 +82,11 @@ const AddDataBarConditionalRuleCommand = {
 	type: _univerjs_core.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(_univerjs_sheets.SheetsSelectionsService).getCurrentSelections().map((selection) => selection.range);
+		if (!ranges.length) return false;
 		const conditionalFormattingRuleModel = accessor.get(_univerjs_sheets_conditional_formatting.ConditionalFormattingRuleModel);
 		const target = (0, _univerjs_sheets.getSheetCommandTarget)(accessor.get(_univerjs_core.IUniverInstanceService));
 		if (!target) return false;
@@ -144,6 +147,37 @@ const AddDuplicateValuesCfCommand = {
 	}
 };
 
+//#endregion
+//#region src/commands/commands/add-icon-set-cf.command.ts
+const AddIconSetConditionalRuleCommand = {
+	type: _univerjs_core.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(_univerjs_sheets.SheetsSelectionsService).getCurrentSelections().map((selection) => selection.range);
+		if (!ranges.length) return false;
+		const target = (0, _univerjs_sheets.getSheetCommandTarget)(accessor.get(_univerjs_core.IUniverInstanceService));
+		if (!target) return false;
+		const { unitId, subUnitId } = target;
+		const rule = {
+			ranges,
+			cfId: accessor.get(_univerjs_sheets_conditional_formatting.ConditionalFormattingRuleModel).createCfId(unitId, subUnitId),
+			stopIfTrue: !!params.stopIfTrue,
+			rule: {
+				type: _univerjs_sheets_conditional_formatting.CFRuleType.iconSet,
+				config: params.config,
+				isShowValue: params.isShowValue
+			}
+		};
+		return accessor.get(_univerjs_core.ICommandService).executeCommand(_univerjs_sheets_conditional_formatting.AddConditionalRuleMutation.id, {
+			unitId,
+			subUnitId,
+			rule
+		});
+	}
+};
+
 //#endregion
 //#region src/commands/commands/add-number-cf.command.ts
 const AddNumberCfCommand = {
@@ -394,11 +428,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 _univerjs_core.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);
@@ -430,10 +462,8 @@ let ConditionalFormattingPanelController = class ConditionalFormattingPanelContr
 };
 ConditionalFormattingPanelController = __decorate([
 	__decorateParam(0, _univerjs_core.IUniverInstanceService),
-	__decorateParam(1, (0, _univerjs_core.Inject)(_univerjs_core.Injector)),
-	__decorateParam(2, (0, _univerjs_core.Inject)(_univerjs_ui.ComponentManager)),
-	__decorateParam(3, (0, _univerjs_core.Inject)(_univerjs_ui.ISidebarService)),
-	__decorateParam(4, (0, _univerjs_core.Inject)(_univerjs_core.LocaleService))
+	__decorateParam(1, (0, _univerjs_core.Inject)(_univerjs_ui.ISidebarService)),
+	__decorateParam(2, (0, _univerjs_core.Inject)(_univerjs_core.LocaleService))
 ], ConditionalFormattingPanelController);
 
 //#endregion
@@ -800,17 +830,155 @@ function FactoryManageConditionalFormattingRule(accessor) {
 	};
 }
 
+//#endregion
+//#region src/menu/quick-insert.menu.ts
+const DATA_BAR_PARAMS = {
+	min: { type: _univerjs_sheets_conditional_formatting.CFValueType.min },
+	max: { type: _univerjs_sheets_conditional_formatting.CFValueType.max },
+	nativeColor: _univerjs_sheets_conditional_formatting.defaultDataBarNativeColor,
+	positiveColor: _univerjs_sheets_conditional_formatting.defaultDataBarPositiveColor,
+	isGradient: false,
+	isShowValue: true
+};
+function createIconSetParams(iconType) {
+	return {
+		config: [
+			{
+				iconType,
+				iconId: "0",
+				operator: _univerjs_sheets_conditional_formatting.CFNumberOperator.greaterThanOrEqual,
+				value: {
+					type: _univerjs_sheets_conditional_formatting.CFValueType.percent,
+					value: 67
+				}
+			},
+			{
+				iconType,
+				iconId: "1",
+				operator: _univerjs_sheets_conditional_formatting.CFNumberOperator.greaterThanOrEqual,
+				value: {
+					type: _univerjs_sheets_conditional_formatting.CFValueType.percent,
+					value: 33
+				}
+			},
+			{
+				iconType,
+				iconId: "2",
+				operator: _univerjs_sheets_conditional_formatting.CFNumberOperator.lessThanOrEqual,
+				value: {
+					type: _univerjs_sheets_conditional_formatting.CFValueType.percent,
+					value: Number.MAX_SAFE_INTEGER
+				}
+			}
+		],
+		isShowValue: true
+	};
+}
+function getQuickConditionalFormattingMenuState(accessor) {
+	return {
+		hidden$: (0, _univerjs_ui.getMenuHiddenObservable)(accessor, _univerjs_core.UniverInstanceType.UNIVER_SHEET),
+		disabled$: (0, _univerjs_sheets_ui.getCurrentRangeDisable$)(accessor, {
+			workbookTypes: [_univerjs_sheets.WorkbookEditablePermission],
+			worksheetTypes: [_univerjs_sheets.WorksheetSetCellStylePermission, _univerjs_sheets.WorksheetEditPermission],
+			rangeTypes: [_univerjs_sheets.RangeProtectionPermissionEditPoint]
+		})
+	};
+}
+function quickDataBarMenuFactory(accessor) {
+	return {
+		id: AddDataBarConditionalRuleCommand.id,
+		type: _univerjs_ui.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: _univerjs_sheets_conditional_formatting.CFRuleType.dataBar,
+				params: DATA_BAR_PARAMS
+			},
+			{
+				id: OpenConditionalFormattingOperator.id,
+				label: "sheets-conditional-formatting-ui.menu.createConditionalFormatting",
+				value: 7,
+				params: { value: 7 }
+			},
+			{
+				id: _univerjs_sheets_conditional_formatting.ClearRangeCfCommand.id,
+				label: "sheets-conditional-formatting-ui.menu.clearRangeRules",
+				value: _univerjs_sheets_conditional_formatting.CFRuleType.dataBar,
+				params: { types: [_univerjs_sheets_conditional_formatting.CFRuleType.dataBar] }
+			}
+		],
+		...getQuickConditionalFormattingMenuState(accessor)
+	};
+}
+function quickIconSetMenuFactory(accessor) {
+	const ratingParams = createIconSetParams(_univerjs_sheets_conditional_formatting.IIconSetType.threeStars);
+	return {
+		id: AddIconSetConditionalRuleCommand.id,
+		type: _univerjs_ui.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: _univerjs_sheets_conditional_formatting.IIconSetType.threeStars,
+				params: ratingParams
+			},
+			{
+				label: "sheets-conditional-formatting-ui.iconSet.shape",
+				value: _univerjs_sheets_conditional_formatting.IIconSetType.threeTrafficLights1,
+				params: createIconSetParams(_univerjs_sheets_conditional_formatting.IIconSetType.threeTrafficLights1)
+			},
+			{
+				label: "sheets-conditional-formatting-ui.iconSet.direction",
+				value: _univerjs_sheets_conditional_formatting.IIconSetType.threeArrows,
+				params: createIconSetParams(_univerjs_sheets_conditional_formatting.IIconSetType.threeArrows)
+			},
+			{
+				id: OpenConditionalFormattingOperator.id,
+				label: "sheets-conditional-formatting-ui.menu.createConditionalFormatting",
+				value: 8,
+				params: { value: 8 }
+			},
+			{
+				id: _univerjs_sheets_conditional_formatting.ClearRangeCfCommand.id,
+				label: "sheets-conditional-formatting-ui.menu.clearRangeRules",
+				value: _univerjs_sheets_conditional_formatting.CFRuleType.iconSet,
+				params: { types: [_univerjs_sheets_conditional_formatting.CFRuleType.iconSet] }
+			}
+		],
+		...getQuickConditionalFormattingMenuState(accessor)
+	};
+}
+
 //#endregion
 //#region src/menu/schema.ts
-const menuSchema = { [_univerjs_ui.RibbonDataGroup.RULES]: { [OpenConditionalFormattingOperator.id]: {
-	order: 1,
-	menuItemFactory: FactoryManageConditionalFormattingRule
-} } };
+const menuSchema = {
+	[_univerjs_ui.RibbonInsertGroup.CELL]: {
+		[AddDataBarConditionalRuleCommand.id]: {
+			order: 3,
+			menuItemFactory: quickDataBarMenuFactory
+		},
+		[AddIconSetConditionalRuleCommand.id]: {
+			order: 4,
+			menuItemFactory: quickIconSetMenuFactory
+		}
+	},
+	[_univerjs_ui.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.3";
+var version = "1.0.0-alpha.4";
 
 //#endregion
 //#region src/config/config.ts
@@ -1463,6 +1631,7 @@ let UniverSheetsConditionalFormattingMobileUIPlugin = class UniverSheetsConditio
 			AddAverageCfCommand,
 			AddColorScaleConditionalRuleCommand,
 			AddDataBarConditionalRuleCommand,
+			AddIconSetConditionalRuleCommand,
 			AddDuplicateValuesCfCommand,
 			AddNumberCfCommand,
 			AddRankCfCommand,
@@ -2093,11 +2262,10 @@ const ColorPicker = (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 = (0, react.useMemo)(() => {
-		if (rule.type === _univerjs_sheets_conditional_formatting.CFRuleType.colorScale) {
+		if ((rule === null || rule === void 0 ? void 0 : rule.type) === _univerjs_sheets_conditional_formatting.CFRuleType.colorScale) {
 			const config = rule.config.map((c, index) => ({
 				color: new _univerjs_core.ColorKit(c.color),
 				value: index
@@ -2108,11 +2276,12 @@ const Preview = (props) => {
 		return null;
 	}, [rule]);
 	const iconSet = (0, react.useMemo)(() => {
-		if (rule.type === _univerjs_sheets_conditional_formatting.CFRuleType.iconSet) return rule.config.map((item) => {
+		if ((rule === null || rule === void 0 ? void 0 : rule.type) === _univerjs_sheets_conditional_formatting.CFRuleType.iconSet) return rule.config.map((item) => {
 			const iconList = _univerjs_sheets_conditional_formatting.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 _univerjs_sheets_conditional_formatting.CFRuleType.dataBar: {
@@ -2178,7 +2347,7 @@ const Preview = (props) => {
 			});
 		}
 	}
-};
+}
 
 //#endregion
 //#region src/views/panel/rule-edit/styles.ts
@@ -3238,6 +3407,12 @@ const WrapperError = (props) => {
 
 //#endregion
 //#region src/views/panel/rule-edit/HighlightCell.tsx
+const textOperators = new Set(Object.values(_univerjs_sheets_conditional_formatting.CFTextOperator));
+const numberOperators = new Set(Object.values(_univerjs_sheets_conditional_formatting.CFNumberOperator));
+const timePeriodOperators = new Set(Object.values(_univerjs_sheets_conditional_formatting.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
@@ -3479,81 +3654,66 @@ const HighlightCellStyleEditor = (props) => {
 		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] = (0, react.useState)(() => {
-		var _value2;
 		const defaultV = "";
 		if (!rule) return defaultV;
-		return (_value2 = rule.value) !== null && _value2 !== void 0 ? _value2 : (0, _univerjs_sheets_conditional_formatting.createDefaultValue)(rule.subType, rule.operator);
+		if ("value" in rule && rule.value !== void 0) return rule.value;
+		return "operator" in rule ? (0, _univerjs_sheets_conditional_formatting.createDefaultValue)(rule.subType, rule.operator) : defaultV;
 	});
 	const [style, setStyle] = (0, react.useState)({});
 	const getResult = (0, react.useMemo)(() => (option) => {
-		var _option$subType5, _option$operator4, _option$style5;
-		switch (option.subType || subType) {
-			case _univerjs_sheets_conditional_formatting.CFSubRuleType.duplicateValues:
-			case _univerjs_sheets_conditional_formatting.CFSubRuleType.uniqueValues:
-				var _option$subType, _option$style;
-				return {
-					type: _univerjs_sheets_conditional_formatting.CFRuleType.highlightCell,
-					subType: (_option$subType = option.subType) !== null && _option$subType !== void 0 ? _option$subType : subType,
-					style: (_option$style = option.style) !== null && _option$style !== void 0 ? _option$style : style
-				};
-			case _univerjs_sheets_conditional_formatting.CFSubRuleType.text:
-				if ([
-					_univerjs_sheets_conditional_formatting.CFTextOperator.beginsWith,
-					_univerjs_sheets_conditional_formatting.CFTextOperator.endsWith,
-					_univerjs_sheets_conditional_formatting.CFTextOperator.containsText,
-					_univerjs_sheets_conditional_formatting.CFTextOperator.notContainsText,
-					_univerjs_sheets_conditional_formatting.CFTextOperator.equal,
-					_univerjs_sheets_conditional_formatting.CFTextOperator.notEqual
-				].includes(operator)) {
-					var _option$subType2, _option$operator, _option$style2, _option$value;
-					return {
-						type: _univerjs_sheets_conditional_formatting.CFRuleType.highlightCell,
-						subType: (_option$subType2 = option.subType) !== null && _option$subType2 !== void 0 ? _option$subType2 : subType,
-						operator: (_option$operator = option.operator) !== null && _option$operator !== void 0 ? _option$operator : operator,
-						style: (_option$style2 = option.style) !== null && _option$style2 !== void 0 ? _option$style2 : style,
-						value: (_option$value = option.value) !== null && _option$value !== void 0 ? _option$value : value
-					};
-				}
-				break;
-			case _univerjs_sheets_conditional_formatting.CFSubRuleType.number:
-				if ([
-					_univerjs_sheets_conditional_formatting.CFNumberOperator.equal,
-					_univerjs_sheets_conditional_formatting.CFNumberOperator.notEqual,
-					_univerjs_sheets_conditional_formatting.CFNumberOperator.greaterThan,
-					_univerjs_sheets_conditional_formatting.CFNumberOperator.greaterThanOrEqual,
-					_univerjs_sheets_conditional_formatting.CFNumberOperator.lessThan,
-					_univerjs_sheets_conditional_formatting.CFNumberOperator.lessThanOrEqual
-				].includes(operator)) {
-					var _option$subType3, _option$operator2, _option$style3, _option$value2;
-					return {
-						type: _univerjs_sheets_conditional_formatting.CFRuleType.highlightCell,
-						subType: (_option$subType3 = option.subType) !== null && _option$subType3 !== void 0 ? _option$subType3 : subType,
-						operator: (_option$operator2 = option.operator) !== null && _option$operator2 !== void 0 ? _option$operator2 : operator,
-						style: (_option$style3 = option.style) !== null && _option$style3 !== void 0 ? _option$style3 : style,
-						value: (_option$value2 = option.value) !== null && _option$value2 !== void 0 ? _option$value2 : value
-					};
-				}
-				if ([_univerjs_sheets_conditional_formatting.CFNumberOperator.between, _univerjs_sheets_conditional_formatting.CFNumberOperator.notBetween].includes(operator)) {
-					var _option$subType4, _option$operator3, _option$style4, _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 _univerjs_sheets_conditional_formatting.CFSubRuleType.duplicateValues: return {
+				type: _univerjs_sheets_conditional_formatting.CFRuleType.highlightCell,
+				subType: _univerjs_sheets_conditional_formatting.CFSubRuleType.duplicateValues,
+				style: nextStyle
+			};
+			case _univerjs_sheets_conditional_formatting.CFSubRuleType.uniqueValues: return {
+				type: _univerjs_sheets_conditional_formatting.CFRuleType.highlightCell,
+				subType: _univerjs_sheets_conditional_formatting.CFSubRuleType.uniqueValues,
+				style: nextStyle
+			};
+			case _univerjs_sheets_conditional_formatting.CFSubRuleType.text: return {
+				type: _univerjs_sheets_conditional_formatting.CFRuleType.highlightCell,
+				subType: _univerjs_sheets_conditional_formatting.CFSubRuleType.text,
+				operator: isTextOperator(nextOperator) ? nextOperator : _univerjs_sheets_conditional_formatting.CFTextOperator.containsText,
+				style: nextStyle,
+				value: typeof nextValue === "string" ? nextValue : void 0
+			};
+			case _univerjs_sheets_conditional_formatting.CFSubRuleType.timePeriod: return {
+				type: _univerjs_sheets_conditional_formatting.CFRuleType.highlightCell,
+				subType: _univerjs_sheets_conditional_formatting.CFSubRuleType.timePeriod,
+				operator: isTimePeriodOperator(nextOperator) ? nextOperator : _univerjs_sheets_conditional_formatting.CFTimePeriodOperator.today,
+				style: nextStyle
+			};
+			case _univerjs_sheets_conditional_formatting.CFSubRuleType.number: {
+				const numberOperator = isNumberOperator(nextOperator) ? nextOperator : _univerjs_sheets_conditional_formatting.CFNumberOperator.equal;
+				if (numberOperator === _univerjs_sheets_conditional_formatting.CFNumberOperator.between || numberOperator === _univerjs_sheets_conditional_formatting.CFNumberOperator.notBetween) {
+					const range = Array.isArray(nextValue) ? nextValue : [0, 100];
 					return {
 						type: _univerjs_sheets_conditional_formatting.CFRuleType.highlightCell,
-						subType: (_option$subType4 = option.subType) !== null && _option$subType4 !== void 0 ? _option$subType4 : subType,
-						operator: (_option$operator3 = option.operator) !== null && _option$operator3 !== void 0 ? _option$operator3 : operator,
-						style: (_option$style4 = option.style) !== null && _option$style4 !== void 0 ? _option$style4 : style,
-						value: (_option$value3 = option.value) !== null && _option$value3 !== void 0 ? _option$value3 : value
+						subType: _univerjs_sheets_conditional_formatting.CFSubRuleType.number,
+						operator: numberOperator,
+						style: nextStyle,
+						value: range
 					};
 				}
-				break;
+				return {
+					type: _univerjs_sheets_conditional_formatting.CFRuleType.highlightCell,
+					subType: _univerjs_sheets_conditional_formatting.CFSubRuleType.number,
+					operator: numberOperator,
+					style: nextStyle,
+					value: typeof nextValue === "number" ? nextValue : void 0
+				};
+			}
 		}
-		return {
-			type: _univerjs_sheets_conditional_formatting.CFRuleType.highlightCell,
-			subType: (_option$subType5 = option.subType) !== null && _option$subType5 !== void 0 ? _option$subType5 : subType,
-			operator: (_option$operator4 = option.operator) !== null && _option$operator4 !== void 0 ? _option$operator4 : operator,
-			style: (_option$style5 = option.style) !== null && _option$style5 !== void 0 ? _option$style5 : style
-		};
 	}, [
 		subType,
 		operator,
@@ -3569,8 +3729,9 @@ const HighlightCellStyleEditor = (props) => {
 		if (!typeOptions.some((item) => item.value === subType)) setSubType(typeOptions[0].value);
 	}, [typeOptions]);
 	const onTypeChange = (v) => {
-		var _operatorList$;
-		const _subType = v;
+		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)
@@ -3584,7 +3745,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 }));
 	};
@@ -3592,9 +3755,6 @@ const HighlightCellStyleEditor = (props) => {
 		setValue(value);
 		onChange(getResult({ value }));
 	};
-	const inputRenderKey = (0, react.useMemo)(() => {
-		return `${subType}_${operator}_${Math.random()}`;
-	}, [subType, operator]);
 	return /* @__PURE__ */ (0, react_jsx_runtime.jsxs)("div", { children: [
 		/* @__PURE__ */ (0, react_jsx_runtime.jsx)("div", {
 			className: "univer-mt-4 univer-text-sm univer-text-gray-600 dark:!univer-text-gray-200",
@@ -3621,7 +3781,7 @@ const HighlightCellStyleEditor = (props) => {
 			operator,
 			rule,
 			onChange: onInputChange
-		}, inputRenderKey),
+		}, `${subType}_${operator}`),
 		/* @__PURE__ */ (0, react_jsx_runtime.jsx)("div", {
 			className: previewClassName,
 			children: /* @__PURE__ */ (0, react_jsx_runtime.jsx)(Preview, { rule: getResult({}) })
@@ -3689,34 +3849,34 @@ const createDefaultConfigItem = (iconType, index, list) => ({
 	iconType,
 	iconId: String(index)
 });
-const IconGroupList = (0, react.forwardRef)((props, ref) => {
-	const { onClick } = props;
+const IconGroupList = (props) => {
+	const { iconType, onClick } = props;
 	const localeService = (0, _univerjs_ui.useDependency)(_univerjs_core.LocaleService);
-	const direction = (0, _univerjs_ui.useObservable)(localeService.direction$, localeService.getDirection());
-	const handleClick = (iconType) => {
-		onClick(iconType);
-	};
 	return /* @__PURE__ */ (0, react_jsx_runtime.jsx)("div", {
-		ref,
-		dir: direction,
+		dir: (0, _univerjs_ui.useObservable)(localeService.direction$, localeService.getDirection()),
 		className: "univer-w-80",
-		children: _univerjs_sheets_conditional_formatting.iconGroup.map((group, index) => {
+		children: _univerjs_sheets_conditional_formatting.iconGroup.map((group) => {
 			return /* @__PURE__ */ (0, react_jsx_runtime.jsxs)("div", {
-				className: "univer-mb-3",
+				className: "univer-mb-4 last:univer-mb-0",
 				children: [/* @__PURE__ */ (0, react_jsx_runtime.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__ */ (0, react_jsx_runtime.jsx)("div", {
-					className: "univer-flex univer-flex-wrap",
-					children: group.group.map((groupItem) => {
-						return /* @__PURE__ */ (0, react_jsx_runtime.jsx)("div", {
-							className: "univer-mb-1 univer-flex univer-w-1/2 univer-items-center rtl:univer-flex-row-reverse rtl:univer-justify-end",
+					className: "univer-grid univer-grid-cols-2 univer-gap-1",
+					children: group.group.map((groupItem, groupItemIndex) => {
+						const selected = groupItem.name === iconType;
+						return /* @__PURE__ */ (0, react_jsx_runtime.jsx)("button", {
+							type: "button",
+							"aria-label": `${localeService.t(group.title)} ${groupItemIndex + 1}`,
+							"aria-pressed": selected,
+							className: (0, _univerjs_design.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__ */ (0, react_jsx_runtime.jsx)("a", {
-								className: "univer-flex univer-cursor-pointer univer-rounded hover:univer-bg-gray-100 dark:hover:!univer-bg-gray-700",
+							children: /* @__PURE__ */ (0, react_jsx_runtime.jsx)("span", {
+								className: "univer-flex univer-items-center univer-gap-0.5",
 								children: groupItem.list.map((base64, index) => /* @__PURE__ */ (0, react_jsx_runtime.jsx)("img", {
+									alt: "",
 									className: "univer-size-5",
 									src: base64,
 									draggable: false
@@ -3725,10 +3885,10 @@ const IconGroupList = (0, react.forwardRef)((props, ref) => {
 						}, groupItem.name);
 					})
 				})]
-			}, index);
+			}, group.title);
 		})
 	});
-});
+};
 const IconItemList = (props) => {
 	const { onClick } = props;
 	const localeService = (0, _univerjs_ui.useDependency)(_univerjs_core.LocaleService);
@@ -3880,7 +4040,7 @@ const IconSetRuleEdit = (props) => {
 										src: icon,
 										className: "univer-size-4",
 										draggable: false
-									}) : /* @__PURE__ */ (0, react_jsx_runtime.jsx)(_univerjs_icons.SlashDoubleIcon, { className: "univer-size-4" }), /* @__PURE__ */ (0, react_jsx_runtime.jsx)(_univerjs_icons.MoreDownIcon, {})]
+									}) : /* @__PURE__ */ (0, react_jsx_runtime.jsx)(_univerjs_icons.SlashDoubleIcon, {}), /* @__PURE__ */ (0, react_jsx_runtime.jsx)(_univerjs_icons.MoreDownIcon, {})]
 								})
 							})
 						}), !isEnd ? /* @__PURE__ */ (0, react_jsx_runtime.jsx)(_univerjs_design.Select, {
@@ -4066,9 +4226,6 @@ const IconSet = (props) => {
 		});
 		setConfigList([...configList]);
 	};
-	const layoutService = (0, _univerjs_ui.useDependency)(_univerjs_ui.ILayoutService);
-	const [iconGroupListEl, setIconGroupListEl] = (0, react.useState)();
-	(0, _univerjs_ui.useScrollYOverContainer)(iconGroupListEl, layoutService.rootContainerElement);
 	return /* @__PURE__ */ (0, react_jsx_runtime.jsxs)("div", {
 		"data-u-comp": "cf-icon-set-editor",
 		dir: direction,
@@ -4082,11 +4239,8 @@ const IconSet = (props) => {
 				children: /* @__PURE__ */ (0, react_jsx_runtime.jsx)(_univerjs_design.Dropdown, {
 					overlay: /* @__PURE__ */ (0, react_jsx_runtime.jsx)("div", {
 						dir: direction,
-						className: "univer-rounded-lg univer-p-4",
+						className: "univer-rounded-lg univer-p-3",
 						children: /* @__PURE__ */ (0, react_jsx_runtime.jsx)(IconGroupList, {
-							ref: (el) => {
-								!iconGroupListEl && el && setIconGroupListEl(el);
-							},
 							iconType: currentIconType,
 							onClick: handleClickIconList
 						})
@@ -4189,34 +4343,36 @@ const RankStyleEditor = (props) => {
 	const [style, setStyle] = (0, react.useState)({});
 	const getResult = (config) => {
 		const { type, isPercent, value, style } = config;
-		if (type === "isNotBottom") return {
-			type: _univerjs_sheets_conditional_formatting.CFRuleType.highlightCell,
-			subType: _univerjs_sheets_conditional_formatting.CFSubRuleType.rank,
-			isPercent,
-			isBottom: false,
-			value,
-			style
-		};
-		if (type === "isBottom") return {
-			type: _univerjs_sheets_conditional_formatting.CFRuleType.highlightCell,
-			subType: _univerjs_sheets_conditional_formatting.CFSubRuleType.rank,
-			isPercent,
-			isBottom: true,
-			value,
-			style
-		};
-		if (type === "greaterThanAverage") return {
-			type: _univerjs_sheets_conditional_formatting.CFRuleType.highlightCell,
-			subType: _univerjs_sheets_conditional_formatting.CFSubRuleType.average,
-			operator: _univerjs_sheets_conditional_formatting.CFNumberOperator.greaterThan,
-			style
-		};
-		if (type === "lessThanAverage") return {
-			type: _univerjs_sheets_conditional_formatting.CFRuleType.highlightCell,
-			subType: _univerjs_sheets_conditional_formatting.CFSubRuleType.average,
-			operator: _univerjs_sheets_conditional_formatting.CFNumberOperator.lessThan,
-			style
-		};
+		switch (type) {
+			case "isNotBottom": return {
+				type: _univerjs_sheets_conditional_formatting.CFRuleType.highlightCell,
+				subType: _univerjs_sheets_conditional_formatting.CFSubRuleType.rank,
+				isPercent,
+				isBottom: false,
+				value,
+				style
+			};
+			case "isBottom": return {
+				type: _univerjs_sheets_conditional_formatting.CFRuleType.highlightCell,
+				subType: _univerjs_sheets_conditional_formatting.CFSubRuleType.rank,
+				isPercent,
+				isBottom: true,
+				value,
+				style
+			};
+			case "greaterThanAverage": return {
+				type: _univerjs_sheets_conditional_formatting.CFRuleType.highlightCell,
+				subType: _univerjs_sheets_conditional_formatting.CFSubRuleType.average,
+				operator: _univerjs_sheets_conditional_formatting.CFNumberOperator.greaterThan,
+				style
+			};
+			case "lessThanAverage": return {
+				type: _univerjs_sheets_conditional_formatting.CFRuleType.highlightCell,
+				subType: _univerjs_sheets_conditional_formatting.CFSubRuleType.average,
+				operator: _univerjs_sheets_conditional_formatting.CFNumberOperator.lessThan,
+				style
+			};
+		}
 	};
 	(0, react.useEffect)(() => {
 		return interceptorManager.intercept(interceptorManager.getInterceptPoints().submit, { handler() {
@@ -4247,13 +4403,17 @@ 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__ */ (0, react_jsx_runtime.jsxs)("div", {
@@ -4579,7 +4739,7 @@ const getRuleDescribe = (rule, localeService) => {
 		}
 	}
 };
-const RuleList = (props) => {
+function RuleList(props) {
 	const { onClick } = props;
 	const conditionalFormattingRuleModel = (0, _univerjs_ui.useDependency)(_univerjs_sheets_conditional_formatting.ConditionalFormattingRuleModel);
 	const univerInstanceService = (0, _univerjs_ui.useDependency)(_univerjs_core.IUniverInstanceService);
@@ -4590,12 +4750,9 @@ const RuleList = (props) => {
 	const conditionalFormattingI18nController = (0, _univerjs_ui.useDependency)(ConditionalFormattingI18nController);
 	const workbook = (0, _univerjs_ui.useObservable)(() => univerInstanceService.getCurrentTypeOfUnit$(_univerjs_core.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] = (0, react.useState)([]);
 	const [selectValue, setSelectValue] = (0, react.useState)("2");
-	const [fetchRuleListId, setFetchRuleListId] = (0, react.useState)(0);
 	const [draggingId, setDraggingId] = (0, react.useState)("");
 	const selectOption = [{
 		label: localeService.t("sheets-conditional-formatting-ui.panel.workSheet"),
@@ -4617,25 +4774,8 @@ const RuleList = (props) => {
 		} else if (selectValue === "2") return [...ruleList];
 		return [];
 	};
-	const [ruleList, setRuleList] = (0, react.useState)(getRuleList);
-	(0, _univerjs_sheets_ui.useHighlightRange)(currentRuleRanges);
-	(0, react.useEffect)(() => {
-		const disposable = commandService.onCommandExecuted((commandInfo) => {
-			if (commandInfo.id === _univerjs_sheets.SetWorksheetActiveOperation.id) setFetchRuleListId(Math.random());
-		});
-		return () => disposable.dispose();
-	});
-	(0, react.useEffect)(() => {
-		setRuleList(getRuleList);
-	}, [
-		selectValue,
-		fetchRuleListId,
-		unitId,
-		subUnitId
-	]);
-	(0, react.useEffect)(() => {
-		if (selectValue === "2") return;
-		const subscription = new rxjs.Observable((commandSubscribe) => {
+	const ruleList = (0, _univerjs_ui.useObservable)(() => {
+		const commandEvent$ = new rxjs.Observable((subscriber) => {
 			const commandList = [
 				_univerjs_sheets.SetSelectionsOperation.id,
 				_univerjs_sheets_conditional_formatting.AddConditionalRuleMutation.id,
@@ -4644,29 +4784,25 @@ const RuleList = (props) => {
 				_univerjs_sheets_conditional_formatting.MoveConditionalRuleMutation.id
 			];
 			const disposable = commandService.onCommandExecuted((commandInfo) => {
-				const { id, params } = commandInfo;
-				const unitId = univerInstanceService.getCurrentUnitOfType(_univerjs_core.UniverInstanceType.UNIVER_SHEET).getUnitId();
-				if (commandList.includes(id) && params.unitId === unitId) commandSubscribe.next(null);
+				var _commandInfo$params;
+				if (commandInfo.id === _univerjs_sheets.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((0, rxjs.debounceTime)(16)).subscribe(() => {
-			setRuleList(getRuleList);
-		});
-		return () => {
-			subscription.unsubscribe();
-		};
-	}, [
-		univerInstanceService,
+		}).pipe((0, rxjs.share)());
+		return (0, rxjs.merge)(conditionalFormattingRuleModel.$ruleChange, commandEvent$.pipe((0, rxjs.filter)((event) => event === "immediate")), commandEvent$.pipe((0, rxjs.filter)((event) => event === "debounced"), (0, rxjs.debounceTime)(16))).pipe((0, rxjs.map)(getRuleList), (0, rxjs.startWith)(getRuleList()));
+	}, [], false, [
+		commandService,
+		conditionalFormattingRuleModel,
 		selectValue,
-		unitId,
-		subUnitId
+		subUnitId,
+		unitId
 	]);
-	(0, react.useEffect)(() => {
-		const dispose = conditionalFormattingRuleModel.$ruleChange.subscribe(() => {
-			setFetchRuleListId(Math.random());
-		});
-		return () => dispose.unsubscribe();
-	}, [conditionalFormattingRuleModel]);
+	(0, _univerjs_sheets_ui.useHighlightRange)(currentRuleRanges);
 	const handleDelete = (rule) => {
 		var _getActiveSheet;
 		const unitId = univerInstanceService.getCurrentUnitOfType(_univerjs_core.UniverInstanceType.UNIVER_SHEET).getUnitId();
@@ -4818,7 +4954,7 @@ const RuleList = (props) => {
 			]
 		})
 	}) })] });
-};
+}
 
 //#endregion
 //#region src/views/ConditionFormattingPanel.tsx
@@ -4878,7 +5014,11 @@ let ComponentsController = class ComponentsController extends _univerjs_core.Dis
 		this.disposeWithMe(this._componentManager.register(CF_PANEL_KEY, ConditionFormattingPanel));
 	}
 	_registerIcons() {
-		this.disposeWithMe(this._iconManager.register({ ConditionsDoubleIcon: _univerjs_icons.ConditionsDoubleIcon }));
+		this.disposeWithMe(this._iconManager.register({
+			ConditionsDoubleIcon: _univerjs_icons.ConditionsDoubleIcon,
+			DataBarIcon: _univerjs_icons.DataBarIcon,
+			RatingIcon: _univerjs_icons.RatingIcon
+		}));
 	}
 };
 ComponentsController = __decorate([__decorateParam(0, (0, _univerjs_core.Inject)(_univerjs_ui.ComponentManager)), __decorateParam(1, (0, _univerjs_core.Inject)(_univerjs_ui.IconManager))], ComponentsController);
@@ -4962,6 +5102,7 @@ let UniverSheetsConditionalFormattingUIPlugin = class UniverSheetsConditionalFor
 			AddAverageCfCommand,
 			AddColorScaleConditionalRuleCommand,
 			AddDataBarConditionalRuleCommand,
+			AddIconSetConditionalRuleCommand,
 			AddDuplicateValuesCfCommand,
 			AddNumberCfCommand,
 			AddRankCfCommand,
@@ -4990,6 +5131,7 @@ exports.AddAverageCfCommand = AddAverageCfCommand;
 exports.AddColorScaleConditionalRuleCommand = AddColorScaleConditionalRuleCommand;
 exports.AddDataBarConditionalRuleCommand = AddDataBarConditionalRuleCommand;
 exports.AddDuplicateValuesCfCommand = AddDuplicateValuesCfCommand;
+exports.AddIconSetConditionalRuleCommand = AddIconSetConditionalRuleCommand;
 exports.AddNumberCfCommand = AddNumberCfCommand;
 exports.AddRankCfCommand = AddRankCfCommand;
 exports.AddTextCfCommand = AddTextCfCommand;