"NNB" == Nidish Narayanaa Balaji<nidish.bal...@ila.uni-stuttgart.de>  writes:
Hi


A few pointers:

The main features that the new minor-mode enables are:
   * It redefines the page-break character to "%%" so an overline is
     drawn over those, making it visually look better.
   * The current cell that point is in is highlighted by being boldfaced
     (this is customizable).
   * Convenience functions for moving point forward and backward by one
     cell (also support being called with prefix arguments).
   * Convenience functions for moving a cell up or down by a block
     (number-prefixes supported too).
After a very very brief testing, some comments:

     1. Could you please include matlab-cell in the Makefile?

     2. So I understand your code does not generalise in any way
        matlab-shell-run-cell?

1.   Done. I've added it to the lisp_LISP variable in the Makefile and
   can confirm that it builds fine.

2. This is correct. I found no reason to rewrite matlab-shell-run-cell
   since it works fine as is. Let me know if you feel otherwise.

   * Some more non-interactive convenience functions to help development.
   * I've not bound any of the functions to any keymap within the package
     yet.
     In my config though, these are the bindings I'm using (in case
     someone's interested):
                  ("C-v" . 'matlab-cell-forward-cell)
                  ("M-v" . 'matlab-cell-backward-cell)
                  ("C-<return>" . matlab-shell-run-cell)
                  ("C-c C-b" . (lambda () (interactive)
                                 (save-window-excursion
                                   (matlab-shell-run-cell))
                                 (matlab-cell-forward-cell)))
                  ("C-x n c" . 'matlab-cell-narrow-to-cell)
                  ("M-[" . 'backward-paragraph)
                  ("M-]" . 'forward-paragraph)
                  ("C-c <down>" . 'matlab-cell-move-cell-down)
                  ("C-c <up>" . 'matlab-cell-move-cell-up)
I'll appreciate comments and suggestions! :)
Could you maybe enhance the documentation of matlab-cell-mode, for
example describe the navigation functions.
Okay - I've added a short writeup under the "Commentary" section in matlab-cell.el. Let me know if this is what you had in mind.
The keyb binding is something we might discuss later

Thanks for the code

Uwe
Thanks for taking a look!
Nidish
_______________________________________________
Matlab-emacs-discuss mailing list
Matlab-emacs-discuss@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/matlab-emacs-discuss

Reply via email to