Description
We want to remove the session picker from the variables pane because it has redundant behavior for console sessions.
The session picker in the variables pane has the exact same behavior as the interpreter picker for console session. When a user selects a console session from the list, the active session changes, which causes the interpreter picker and console pane to update.
When a user selects a notebook session from the list, the variables pane shows the notebook session data but no other panes update. Not seeing the interpreter pane update in this instance can be confusing for users.
The variables pane should only show the data for the active session. The active session changes because of user gestures such as opening a file, changing the focused editor, clicking on a console tab, etc.
Note: There is a setting positron.variables.followMode that was introduced in #5583 specifically for notebooks that we probably won't need anymore once we remove the session picker. We'll need to modify the event listening logic to make sure we rely on the foreground session only.
We'll also need to handle "exited" sessions - currently the variables pane doesn't show information for exited sessions that are the foreground session. If a user has multiple console sessions where one is exited, clicking on that session causes the foreground session to change but not the variables pane (it will continue to show the previous foreground session that is not in an exited state). We should show the variables in a "disabled" state if possible, or show nothing.
Description
We want to remove the session picker from the variables pane because it has redundant behavior for console sessions.
The session picker in the variables pane has the exact same behavior as the interpreter picker for console session. When a user selects a console session from the list, the active session changes, which causes the interpreter picker and console pane to update.
When a user selects a notebook session from the list, the variables pane shows the notebook session data but no other panes update. Not seeing the interpreter pane update in this instance can be confusing for users.
The variables pane should only show the data for the active session. The active session changes because of user gestures such as opening a file, changing the focused editor, clicking on a console tab, etc.
Note: There is a setting
positron.variables.followModethat was introduced in #5583 specifically for notebooks that we probably won't need anymore once we remove the session picker. We'll need to modify the event listening logic to make sure we rely on the foreground session only.We'll also need to handle "exited" sessions - currently the variables pane doesn't show information for exited sessions that are the foreground session. If a user has multiple console sessions where one is exited, clicking on that session causes the foreground session to change but not the variables pane (it will continue to show the previous foreground session that is not in an exited state). We should show the variables in a "disabled" state if possible, or show nothing.