Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix REPL window opening in a redundant split #8

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

homedirectory
Copy link

If there are multiple tabs where each has its own REPL window (sharing the same buffer), then an issue occurs that opens a new split with a REPL window instead of reusing an existing one. This occurs because there is always only one "main window" which has a single associated REPL window, and there is no code that recalculates the current REPL window when the main one changes (e.g, when switching tabs).

Steps to reproduce:

  1. Start a server and connect to it.
  2. Send something to the REPL so that its window opens in a split in the current tab.
  3. Open a new tab and repeat step 2 there.
  4. Go back to the first tab, repeat step 2.

This change adds specific behaviour for REPL windows to try and reuse existing ones in the current tab.

If there are multiple tabs where each has its own REPL window (sharing
the same buffer), then an issue occurs that opens a new split with a
REPL window instead of reusing an existing one. This occurs because
there is always only one "main window" which has a single associated
REPL window, and there is no code that recalculates the current REPL
window when the main one changes (e.g, when switching tabs).

This change adds specific behaviour for REPL windows to try and reuse
existing ones in the current tab.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant