Hacktoberfest 2026: los issues que los mantenedores marcaron para octubre, abiertos y aptos para principiantes. Explorar issues de Hacktoberfest

Bug: Undo History Not Working for Accept Changes in UnifiedMergeView

Abierto
  • ¿Algo no está bien?
#714 0 comentarios 4 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
4/5
Tiempo estimado
3-5 días
Aptitud para principiantes
35/100
Tipo de issue
Error
Claridad
Bastante claro
Estado de actividad
Estancado
Stack tecnológico
react, typescript
Área
frontend

Línea de trabajo

Empieza ejecutando la reproducción proporcionada de unifiedMergeView con mergeControls y la history extension; después, rastrea cómo Accept y Reject hacen dispatch de sus cambios. Confirma el comportamiento según las expectativas de undo indicadas e identifica el punto de integración relevante antes de añadir una prueba de regresión; el trabajo estará terminado cuando los cambios aceptados se puedan deshacer igual que los cambios rechazados.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

Description

When using unifiedMergeView with mergeControls: true, the undo history functionality (Ctrl+Z/Cmd+Z) only works for rejected changes but not for accepted changes. The expected behavior is that both accept and reject operations should be undoable.

Steps to Reproduce

  1. Create a basic setup with unifiedMergeView:
"use client";
import React, { useEffect, useRef } from "react";
import { EditorView, keymap  } from "@codemirror/view";
import { unifiedMergeView } from "@codemirror/merge";
import { javascript } from "@codemirror/lang-javascript";
import { oneDark } from "@codemirror/theme-one-dark";
import { defaultKeymap, history, historyKeymap } from "@codemirror/commands";

export default function DemoMergeIssue() {
  const editorRef = useRef(null);
  const commonExtensions = [
    javascript({ jsx: true, typescript: true }), 
    oneDark,
    history(),
    keymap.of(defaultKeymap),
    keymap.of(historyKeymap),
  ];
  useEffect(() => {
    if (editorRef.current) {

      // Create a unified merge view.
      const view = new EditorView({
        doc: "console.log('Modified Code');",
        extensions: [
          ...commonExtensions,
          oneDark,
          javascript(),
          unifiedMergeView({
            original: "console.log('Original Code');",
            highlightChanges: true,
            mergeControls: true,
          }),
        ],
        parent: editorRef.current,
      });
      return () => view.destroy();
    }
  }, []);

  return (
    <div
      ref={editorRef}
      style={{
        height: "400px",
        border: "1px solid #ccc",
        borderRadius: "4px",
        margin: "1rem 0",
      }}
    />
  );
}
});
  1. Click the "Accept" button in merge controls
  2. Try to undo the accept operation with Ctrl+Z/Cmd+Z
  3. Expected: Changes are undone, Actual: Nothing happens
  4. Click the "Reject" button in merge controls
  5. Try to undo the reject operation with Ctrl+Z/Cmd+Z
  6. Expected: Changes are undone, Actual: Changes are undone successfully

Analysis

The issue appears to be related to how the unified merge view handles history states for accept operations. While reject operations properly integrate with the history extension, accept operations seem to bypass the history mechanism.

Environment

    "@codemirror/commands": "^6.8.0",
    "@codemirror/lang-javascript": "^6.2.3",
    "@codemirror/merge": "^6.8.0",
    "@codemirror/state": "^6.5.2",
    "@codemirror/theme-one-dark": "^6.1.2",
    "@codemirror/view": "^6.36.3",
Lenguaje dominante
TypeScript
Estrellas
2.3k
Forks
162
Merge medio
2 h 41 min
PR fusionados (30 d)
1

Preparar el entorno

Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de uiwjs/react-codemirror

Todos los issues de uiwjs/react-codemirror

Issues similares

Más issues de TypeScript

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.