Skip to content

Commit

Permalink
add a specific warning about Windows in the README
Browse files Browse the repository at this point in the history
  • Loading branch information
gwenzek committed Dec 27, 2017
1 parent 2bcc9d1 commit 3738c64
Showing 1 changed file with 11 additions and 6 deletions.
17 changes: 11 additions & 6 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -21,13 +21,18 @@ documentation.

## Room for improvement

* Currently the bash script that modifies the window title needs the PID of the
target window. As there is no ST API for that, the script list all ST windows
and rename the one with a name matching the expected name.
If there was a reliable way to find the PID of the correct window then we could drop the `get_official_name` method, and simplify the script a lot.
* Currently the code that modifies the window title needs the PID of the
target window. As there is no ST API for that, we list all ST windows and
rename the one with a name matching the expected name.
If there was a reliable way to find the PID of the correct window then we
could drop the `get_official_name` method, and simplify the plugin a lot.

* For some reason some chars aren't displayed correctly (Linux).
* (Linux) For some reason some chars aren't displayed correctly.

* (Windows) The plugin is more brittle on Windows, some features have been
disabled as a consequence. If you have knowledge of the `user32` API we might
use your help !

## Thanks

Thanks to [cjdoris](https://github.com/cjdoris) for implementing Windows support.
Thanks to [cjdoris](https://github.com/cjdoris) for implementing Windows support.

0 comments on commit 3738c64

Please sign in to comment.