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

"create adjacent and carry" doesn't carry #163

Merged
merged 1 commit into from
Jul 2, 2021

Conversation

rpbaptist
Copy link
Contributor

@rpbaptist rpbaptist commented Jul 1, 2021

Resolves #161

When the new pane is created, the cursor ends up there as well. This is why the carry command does not work. I'm not sure when this feature broke or if it behaves differently in ST4 than it did in ST3.

Since the newly created pane becomes active, the file can be pulled from the opposite direction instead.

It can be tested without a code change by adding this keybind:

  {
    "keys": ["ctrl+space", "ctrl+u"],
    "command": "chain",
    "args": {
      "commands": [
        {"command": "create_pane", "args": {"direction": "up"}},
        {"command": "pull_file_from_pane", "args": {"direction": "down"}}
      ]
    }
  },

I also tested it locally by uninstalling the package, using my own branch and then using the original keybind to create_pane_with_file.

When the new pane is created, the cursor ends up there as well. This is
why the carry command does not work. Pulling it instead solves this.
@rchl
Copy link
Member

rchl commented Jul 2, 2021

LGTM. Thanks!

@rchl rchl changed the title create_pane_with_file creates and then pulls the file "create adjacent and carry" doesn't carry Jul 2, 2021
@rchl rchl merged commit 2811981 into SublimeText:master Jul 2, 2021
@rpbaptist rpbaptist deleted the fix/create_pane_and_carry branch July 2, 2021 22:11
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.

ST4: create_pane_with_file does not move the file
2 participants