Skip to content

Fix/designer#9

Merged
hackycy merged 9 commits intomainfrom
fix/designer
May 7, 2026
Merged

Fix/designer#9
hackycy merged 9 commits intomainfrom
fix/designer

Conversation

@hackycy
Copy link
Copy Markdown
Owner

@hackycy hackycy commented May 7, 2026

No description provided.

hackycy and others added 9 commits May 6, 2026 17:43
Introduce a GeometryService implementation for designer interactions and migrate code to use document/screen/local coordinate APIs. Update core types (PageGeometrySnapshot, LocalCoordinateOptions) and replace direct UnitManager screen<->document usage across designer composables, interactions, and material editors. Add geometry-service implementation and tests, adjust architecture docs and table-kernel to use document coordinates, and add an ESLint rule to forbid direct unitManager conversions in interaction code. This centralizes page offset/scroll/zoom handling and node transforms through a single service.
Introduce a writingMode prop (horizontal|vertical) for the text material and wire it through schema, defaults and AI descriptor. Refactor rendering logic into a new rendering.ts with helpers (getTextContainerStyles, getTextContentStyles) used by both designer and viewer, and export the module from index.ts. Update prop schemas, locale strings and mcp-server materials config to reflect the new enum and remove the legacy richText property; viewer now always escapes content (legacy richText treated as plain/escaped), and a unit test was added to verify vertical-mode styling and richText escaping.
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 7, 2026

Warning

Rate limit exceeded

@hackycy has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 59 minutes and 35 seconds before requesting another review.

To continue reviewing without waiting, purchase usage credits in the billing tab.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 9192cc96-4a29-4272-a722-85cede6c11de

📥 Commits

Reviewing files that changed from the base of the PR and between b23df68 and faabe48.

📒 Files selected for processing (74)
  • .github/architecture/10-designer-interaction.md
  • .github/architecture/11-element-system.md
  • .github/architecture/13-unit-system.md
  • .github/architecture/22-editing-behavior.md
  • .github/architecture/README.md
  • .gitignore
  • eslint.config.js
  • packages/core/src/behaviors/selection.ts
  • packages/core/src/commands/document.ts
  • packages/core/src/commands/index.ts
  • packages/core/src/editing-session.ts
  • packages/core/src/index.ts
  • packages/core/src/material-extension.ts
  • packages/core/src/page-planner.test.ts
  • packages/core/src/page-planner.ts
  • packages/core/src/selection.ts
  • packages/designer/src/components/CanvasContextMenu.vue
  • packages/designer/src/components/CanvasWorkspace.vue
  • packages/designer/src/components/EphemeralPanelHost.vue
  • packages/designer/src/components/GuideOverlay.vue
  • packages/designer/src/components/HistoryPanel.vue
  • packages/designer/src/components/PropertiesPanel.vue
  • packages/designer/src/components/StructureTree.vue
  • packages/designer/src/components/TopBarB.vue
  • packages/designer/src/composables/use-datasource-drop.test.ts
  • packages/designer/src/composables/use-datasource-drop.ts
  • packages/designer/src/composables/use-element-drag.ts
  • packages/designer/src/composables/use-element-resize.ts
  • packages/designer/src/composables/use-element-rotate.ts
  • packages/designer/src/composables/use-keyboard-shortcuts.ts
  • packages/designer/src/composables/use-marquee-select.test.ts
  • packages/designer/src/composables/use-marquee-select.ts
  • packages/designer/src/composables/use-material-drop.ts
  • packages/designer/src/editing/editing-session-manager.ts
  • packages/designer/src/editing/geometry-service.test.ts
  • packages/designer/src/editing/geometry-service.ts
  • packages/designer/src/interactions/canvas-interaction-controller.ts
  • packages/designer/src/interactions/clipboard-actions.ts
  • packages/designer/src/locale/zh-CN.ts
  • packages/designer/src/materials/prop-schemas.ts
  • packages/designer/src/snap/selection-box.ts
  • packages/designer/src/snap/snap-engine.test.ts
  • packages/designer/src/snap/snap-engine.ts
  • packages/designer/src/store/designer-store.ts
  • packages/materials/line/src/designer.ts
  • packages/materials/table-data/src/ai.ts
  • packages/materials/table-data/src/designer.ts
  • packages/materials/table-data/src/layout.ts
  • packages/materials/table-data/src/viewer.ts
  • packages/materials/table-kernel/src/commands.test.ts
  • packages/materials/table-kernel/src/editing/behaviors.ts
  • packages/materials/table-kernel/src/editing/geometry.test.ts
  • packages/materials/table-kernel/src/editing/geometry.ts
  • packages/materials/table-kernel/src/editing/selection-type.ts
  • packages/materials/table-kernel/src/geometry.ts
  • packages/materials/table-kernel/src/render.ts
  • packages/materials/text/src/ai.ts
  • packages/materials/text/src/designer.test.ts
  • packages/materials/text/src/designer.ts
  • packages/materials/text/src/index.ts
  • packages/materials/text/src/rendering.ts
  • packages/materials/text/src/schema.ts
  • packages/materials/text/src/viewer.test.ts
  • packages/materials/text/src/viewer.ts
  • packages/mcp-server/config/materials.json
  • packages/mcp-server/src/config/material-loader.test.ts
  • packages/mcp-server/src/prompts/system-builder.test.ts
  • packages/mcp-server/src/prompts/system-builder.ts
  • packages/samples/src/datasources.ts
  • packages/samples/src/index.ts
  • packages/samples/src/templates/supermarket-receipt.ts
  • packages/samples/src/templates/vertical-mixed-text.ts
  • packages/viewer/src/stack-flow-layout.test.ts
  • packages/viewer/src/stack-flow-layout.ts
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/designer

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

  • Generate code and open pull requests
  • Plan features and break down work
  • Investigate incidents and troubleshoot customer tickets together
  • Automate recurring tasks and respond to alerts with triggers
  • Summarize progress and report instantly

Built for teams:

  • Shared memory across your entire org—no repeating context
  • Per-thread sandboxes to safely plan and execute work
  • Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@hackycy hackycy merged commit cd29d42 into main May 7, 2026
0 of 2 checks passed
@hackycy hackycy deleted the fix/designer branch May 7, 2026 05:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant