|
From: Nidish N. B. <nid...@il...> - 2024-05-15 07:57:11
|
>>>> "NNB" == Nidish Narayanaa Balaji<nid...@il...> 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 |