Skip to content

Conversation

@dersimn
Copy link

@dersimn dersimn commented Sep 5, 2024

I changed some stuff regarding publisher, name etc. to make a working fork until you decide to pull the suff here. Just edit package.json back to your project.


Improvements:

  • Open multiple files at once with rmate file1 file2

  • Closing files with -w flag which is useful for using rmate in EDITOR-env variable.
    Example: EDITOR='rmate -w' sudoedit somefile

    Due to a limitation in VS Code API (currently there's no API that would notify our Extetnsion when the user closed a Tab) we implemented a buttton here to alternatively close the Tab and the rmate-Session behind:
    image

    Alternatively there's a command for closing single
    image
    Sessions or all Sessions at once:

ComfyDust and others added 30 commits October 24, 2018 18:05
…an array instead of a single instance variable
```
This activation event can be removed as VS Code generates these automatically from your package.json contribution declarations.
```
revert rootDir change and list config by string if this should cause any problems
Docs:

```
An array to which disposables can be added. When this extension is deactivated the disposables will be disposed.

Note that asynchronous dispose-functions aren't awaited.
```
- TextMate sends a separate `close` message for each closed tab (e.g. if several files were opened with `rmate file1 file2`).
- The token is appended to each `close` message.
- Sending without a token breaks the stdin function of at least [aurora/rmate](https://github.com/aurora/rmate)'s implementation.
- With the current VS Code workaround of having to close files manually because `vscode.workspace.onDidCloseTextDocument` is sometimes triggered after minutes, there is no real need to send a `close` message individually, but we will do so anyway so that it is clean.
Path on Windows looks like:

    /c:/Users/Simon/AppData/Local/Temp/rmate-vscode/4WFxHSBiGf/testserver/test
    C:\\Users\\Simon\\AppData\\Local\\Temp\\rmate-vscode\\4WFxHSBiGf\\testserver\\test

Simple string compare failed before
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.

2 participants