Dmytro Maluka
Dmytro Maluka
Please squash the commits into one, there is no point in having a broken commit immediately followed by a commit that fixes it.
@JoeKar I also don't quite get your point. `CursorToViewCenter` moves the cursor to the view center, simple as that. `CursorToView{Top,Center,Down}` actions are independent of `Page{Up,Down}` (although of course can be...
IMHO not a bad feature, why not. The name `tabalwaysshow` may be a bit of a mouthful... Maybe just `tabalways`? (Better suggestions are welcome.) Also, this PR doesn't work quite...
> or when window is loosing focus Micro (or any other terminal-based editor) has no way to know if the terminal window it is running in is focused or not....
> That's not entirely true, some (many?) terminals support xterm's focus events [1](#user-content-fn-1-537f14e2cc3bb3072a97605c69a9a151) Ok, didn't know that (or forgot about that). Anyway, yeah, I see no advantages of doing that...
> If you "may be listening to file changes to do some tasks, and you expect the file to be valid," you should deactivate `autosave` if you are depending on...
> Possibly more realistic approach would be for micro to provide hooks for `onFocusIn` and `onFocusOut` events so the desired behavior could be easily implemented as a plugin Yep, I...
> If `onSetInactive` is implemented it will be called next to `onSetActive`, right? > Instead couldn't the "old pane" be an extra arg for `onSetActive`? Unfortunately we need to preserve...
Yes, reusing `onSetActive` seems better. > But IIRC there's already some sort of problem with `onSetActive` right? Which problem? (apart from its name)
Ok, those are problems with the implementation, not with the interface? I.e. we can just fix them.