Manim-Notebook / manim-notebook

Simple commands to replicate the manim dev workflow in VSCode
MIT License
5 stars 0 forks source link

Start scene should warn before killing active session #36

Closed bhoov closed 5 days ago

bhoov commented 6 days ago

(see this comment)

Re: If the start scene command is invoked while a preview is already running, close the current session and invoke a new one -- I find this a bit jarring that we automatically kill, even if the hotkey is accidentally pressed. From a user-experience perspective, I would prefer a pop-up question "Would you like to kill the existing manim shell to start a new shell at the cursor? [y/N]"

Right now, a user can run the "Start scene at cursor" command which will automatically kill the existing terminal and start a new session at the specified line. It will benefit new users who are learning the commands and hotkeys to warn if we are killing their session

Splines commented 5 days ago

Notifications with actions example code | UX guidelines