Smart scrolling during script advancement implemented.

[helm.git] / helm / matita / matitaScript.ml

diff --git a/helm/matita/matitaScript.ml b/helm/matita/matitaScript.ml
index e31dd7a1540e5642fde6d525bad6f4d1aac10a3f..e0e4227dd1f5709d921b4bd57f3bf8b0df6594dd 100644 (file)
--- a/helm/matita/matitaScript.ml
+++ b/helm/matita/matitaScript.ml
@@ -383,13 +383,15 @@ let fresh_script_id =
   let i = ref 0 in
   fun () -> incr i; !i
 
-class script ~(buffer: GText.buffer) ~(init: MatitaTypes.status) 
+class script  ~(view: GText.view)
+              ~(init: MatitaTypes.status) 
               ~(mathviewer: MatitaTypes.mathViewer) 
               ~set_star
               ~ask_confirmation
               ~urichooser 
               ~develcreator 
               () =
+let buffer = view#buffer in
 object (self)
   val scriptId = fresh_script_id ()
   
@@ -464,14 +466,12 @@ List.iter (fun s -> prerr_endline ("'" ^ s ^ "'")) new_statements;
       end;
     self#moveMark (String.length new_text)
 
-  method private _retract () =
-    match statements, history with
-    | last_statement :: _, cur_status :: prev_status :: _ ->
-        MatitaSync.time_travel ~present:cur_status ~past:prev_status;
-        statements <- List.tl statements;
-        history <- List.tl history;
-        self#moveMark (- (String.length last_statement));
-    | _ -> raise Margin
+  method private _retract offset status new_statements new_history =
+   let cur_status = match history with s::_ -> s | [] -> assert false in
+    MatitaSync.time_travel ~present:cur_status ~past:status;
+    statements <- new_statements;
+    history <- new_history;
+    self#moveMark (- offset)
 
   method advance ?statement () =
     try
@@ -483,8 +483,15 @@ List.iter (fun s -> prerr_endline ("'" ^ s ^ "'")) new_statements;
 
   method retract () =
     try
-      self#_retract ();
-      self#notify
+      let cmp,new_statements,new_history,status =
+       match statements,history with
+          stat::statements, _::(status::_ as history) ->
+           String.length stat, statements, history, status
+       | [],[_] -> raise Margin
+       | _,_ -> assert false
+      in
+       self#_retract cmp status new_statements new_history;
+       self#notify
     with 
     | Margin -> self#notify
     | exc -> self#notify; raise exc
@@ -514,6 +521,12 @@ List.iter (fun s -> prerr_endline ("'" ^ s ^ "'")) new_statements;
         Incomplete_proof (_,goal) -> self#setGoal goal
       | _ -> ()
     end ;
+    let mark_position = buffer#get_iter_at_mark mark in
+    if view#move_mark_onscreen mark then
+     begin
+      buffer#move_mark mark mark_position;
+      view#scroll_to_mark ~use_align:true ~xalign:1.0 ~yalign:0.1 mark;
+     end;
     while Glib.Main.pending () do ignore(Glib.Main.iteration false); done
 
   method clean_dirty_lock =
@@ -599,6 +612,7 @@ List.iter (fun s -> prerr_endline ("'" ^ s ^ "'")) new_statements;
     | `Cursor ->
         let locked_iter () = buffer#get_iter_at_mark (`NAME "locked") in
         let cursor_iter () = buffer#get_iter_at_mark `INSERT in
+        let cmp = (locked_iter ())#offset - (cursor_iter ())#offset in
         let forward_until_cursor () = (* go forward until locked > cursor *)
           let rec aux oldpos =
             self#_advance ();
@@ -609,17 +623,19 @@ List.iter (fun s -> prerr_endline ("'" ^ s ^ "'")) new_statements;
           in
           aux (getpos ())
         in
-        let rec back_until_cursor () = (* go backward until locked < cursor *)
-          self#_retract ();
-          if (locked_iter ())#compare (cursor_iter ()) > 0 then
-            back_until_cursor ()
+        let rec back_until_cursor len = (* go backward until locked < cursor *)
+         function
+            statements, (status::_ as history) when len <= 0 ->
+             self#_retract (cmp - len) status statements history
+          | statement::tl1, _::tl2 ->
+             back_until_cursor (len - String.length statement) (tl1,tl2)
+          | _,_ -> assert false
         in
-        let cmp = (locked_iter ())#compare (cursor_iter ()) in
         (try
           if cmp < 0 then       (* locked < cursor *)
             (forward_until_cursor (); self#notify)
           else if cmp > 0 then  (* locked > cursor *)
-            (back_until_cursor (); self#notify)
+            (back_until_cursor cmp (statements,history); self#notify)
           else                  (* cursor = locked *)
               ()
         with 
@@ -673,10 +689,10 @@ end
 
 let _script = ref None
 
-let script ~buffer ~init ~mathviewer ~urichooser ~develcreator ~ask_confirmation ~set_star ()
+let script ~view ~init ~mathviewer ~urichooser ~develcreator ~ask_confirmation ~set_star ()
 =
   let s = new script 
-    ~buffer ~init ~mathviewer ~ask_confirmation ~urichooser ~develcreator ~set_star () 
+    ~view ~init ~mathviewer ~ask_confirmation ~urichooser ~develcreator ~set_star () 
   in
   _script := Some s;
   s