Skip to content

chore: v2: Add v1 style undo-redo controls to canvas#2264

Merged
camielvs merged 1 commit into
masterfrom
05-13-chore_add_v1_style_undo-redo_controls_to_canvas
May 14, 2026
Merged

chore: v2: Add v1 style undo-redo controls to canvas#2264
camielvs merged 1 commit into
masterfrom
05-13-chore_add_v1_style_undo-redo_controls_to_canvas

Conversation

@camielvs
Copy link
Copy Markdown
Collaborator

@camielvs camielvs commented May 13, 2026

Description

As per a recent discussion the v1 undo-redo controls have been brought over to the v2 canvas. Behaviour and UI is exactly the same as v1 editor

Related Issue and Pull requests

Type of Change

  • Improvement

Checklist

  • I have tested this does not break current pipelines / runs functionality
  • I have tested the changes on staging

Screenshots (if applicable)

image.png

Test Instructions

Additional Comments

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 13, 2026

🎩 Preview

A preview build has been created at: 05-13-chore_add_v1_style_undo-redo_controls_to_canvas/959c1e8

@camielvs camielvs force-pushed the 05-13-chore_add_v1_style_undo-redo_controls_to_canvas branch from 50baceb to a2c3e9d Compare May 13, 2026 21:52
@camielvs camielvs changed the title chore: Add v1 style undo-redo controls to canvas chore: v2: Add v1 style undo-redo controls to canvas May 13, 2026
@camielvs camielvs marked this pull request as ready for review May 14, 2026 00:48
@camielvs camielvs requested a review from a team as a code owner May 14, 2026 00:48
Comment thread src/routes/v2/pages/Editor/components/FlowCanvas/components/CanvasUndoRedo.tsx Outdated
const UNDO_SHORTCUT = isMac ? "⌘Z" : "Ctrl+Z";
const REDO_SHORTCUT = isMac ? "⌘⇧Z" : "Ctrl+Y";

export const CanvasUndoRedo = observer(function CanvasUndoRedo() {
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Slightly similar to https://app.graphite.com/github/pr/TangleML/tangle-ui/2269?ref=gt-pasteable-stack - I believe we can merge two components in future

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we can have a look at bringing them together

@camielvs camielvs force-pushed the 05-13-fix_order_pipeline_list_by_modifiedat branch from ac7738d to 9f3564f Compare May 14, 2026 23:30
@camielvs camielvs force-pushed the 05-13-chore_add_v1_style_undo-redo_controls_to_canvas branch from a2c3e9d to 7f89cd6 Compare May 14, 2026 23:30
@camielvs camielvs force-pushed the 05-13-fix_order_pipeline_list_by_modifiedat branch 2 times, most recently from 26b534a to c43a6fe Compare May 14, 2026 23:34
@camielvs camielvs force-pushed the 05-13-chore_add_v1_style_undo-redo_controls_to_canvas branch from 7f89cd6 to cabae0f Compare May 14, 2026 23:34
@camielvs camielvs changed the base branch from 05-13-fix_order_pipeline_list_by_modifiedat to graphite-base/2264 May 14, 2026 23:37
@camielvs camielvs force-pushed the 05-13-chore_add_v1_style_undo-redo_controls_to_canvas branch from cabae0f to 0655705 Compare May 14, 2026 23:38
@camielvs camielvs force-pushed the graphite-base/2264 branch from c43a6fe to 1b6ba4f Compare May 14, 2026 23:38
@graphite-app graphite-app Bot changed the base branch from graphite-base/2264 to master May 14, 2026 23:38
@camielvs camielvs force-pushed the 05-13-chore_add_v1_style_undo-redo_controls_to_canvas branch from 0655705 to 959c1e8 Compare May 14, 2026 23:38
Copy link
Copy Markdown
Collaborator Author

camielvs commented May 14, 2026

Merge activity

  • May 14, 11:42 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • May 14, 11:42 PM UTC: @camielvs merged this pull request with Graphite.

@camielvs camielvs merged commit 4410c4e into master May 14, 2026
17 checks passed
@camielvs camielvs deleted the 05-13-chore_add_v1_style_undo-redo_controls_to_canvas branch May 14, 2026 23:42
@camielvs camielvs mentioned this pull request May 15, 2026
3 tasks
@camielvs camielvs added the #gsd:50471 CSOM-X label May 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants