feat(sheets-ui): optimize ref-range for hyper-link and thread-comment (…

…#2405)

* feat: update

* feat: optimize ref-range

* feat: ref-range optimize for comment

* feat: update

* feat: update

* fix: number error treated as link

* feat: update scroll

* fix: format-paint

* feat: update

* fix: update

* feat: update

* feat: update

* feat: u

* feat: update

* feat: update

* feat: update

* feat: update

* feat: update

* feat: update

* feat: update

* feat: update

* feat: update

* feat: update

* feat: dv-render

* feat: update

* feat: style update

* feat: update

* feat: update

* feat: lint

* feat: update

* fix: build

* feat: update

* feat: update

* feat: update

* feat: update

* feat: remove comments

* feat: update link popup

* feat: dispose popup

examples/src/data/sheets/demo/default-workbook-data-demo.ts

@@ -178,30 +178,13 @@ const dataValidation = [
 const dv2 = [
  {
  uid: 'xxx-2',
- type: 'checkbox',
- ranges: [
- {
- startRow: 1,
- endRow: 2,
- startColumn: 1,
- endColumn: 2,
- },
- ],
- },
- {
- uid: 'adN9-O',
- type: 'list',
- formula1: "='sheet-0005'!F4:F8",
- ranges: [
- {
- startRow: 4,
- startColumn: 5,
- endRow: 14,
- endColumn: 8,
- rangeType: 0,
- },
- ],
- formula2: '',
+ type: DataValidationType.CHECKBOX,
+ ranges: [{
+ startRow: 1,
+ endRow: 2,
+ startColumn: 1,
+ endColumn: 2,
+ }],
  },
 ];
 
@@ -23681,7 +23664,7 @@ export const DEFAULT_WORKBOOK_DATA_DEMO: IWorkbookData = {
  ref: 'C6',
  personId: 'mockId',
  unitId: 'workbook-01',
- subUnitId: 'dv-test',
+ subUnitId: 'sheet-0011',
  },
  {
  text: {
@@ -23696,7 +23679,7 @@ export const DEFAULT_WORKBOOK_DATA_DEMO: IWorkbookData = {
  ref: 'B6',
  personId: 'mockId',
  unitId: 'workbook-01',
- subUnitId: 'dv-test',
+ subUnitId: 'sheet-0011',
  },
  {
  text: {
@@ -23711,7 +23694,7 @@ export const DEFAULT_WORKBOOK_DATA_DEMO: IWorkbookData = {
  ref: 'B7',
  personId: 'mockId',
  unitId: 'workbook-01',
- subUnitId: 'dv-test',
+ subUnitId: 'sheet-0011',
  },
  {
  text: {
@@ -23726,7 +23709,7 @@ export const DEFAULT_WORKBOOK_DATA_DEMO: IWorkbookData = {
  ref: 'B8',
  personId: 'mockId',
  unitId: 'workbook-01',
- subUnitId: 'dv-test',
+ subUnitId: 'sheet-0011',
  },
  {
  text: {
@@ -23741,7 +23724,7 @@ export const DEFAULT_WORKBOOK_DATA_DEMO: IWorkbookData = {
  ref: 'B9',
  personId: 'mockId',
  unitId: 'workbook-01',
- subUnitId: 'dv-test',
+ subUnitId: 'sheet-0011',
  },
  ],
  }),

packages/sheets-hyper-link/package.json

@@ -64,6 +64,9 @@
  "@univerjs/sheets-hyper-link": "workspace:*",
  "@wendellhu/redi": "0.15.4",
  "rxjs": ">=7.0.0"
+ },
+ "dependencies": {
+
  },
  "devDependencies": {
  "@univerjs/core": "workspace:*",

packages/sheets-ui/src/commands/commands/set-format-painter.command.ts

@@ -72,6 +72,7 @@ export interface IApplyFormatPainterCommandParams {
 export const ApplyFormatPainterCommand: ICommand = {
  type: CommandType.COMMAND,
  id: 'sheet.command.apply-format-painter',
+ // eslint-disable-next-line max-lines-per-function
  handler: async (accessor: IAccessor, params: IApplyFormatPainterCommandParams) => {
  const formatPainterService = accessor.get(IFormatPainterService);
  return formatPainterService.applyFormatPainter(params.unitId, params.subUnitId, params.range);