-
Notifications
You must be signed in to change notification settings - Fork 34
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
Running inside Oni #10
Comments
@CrossR Do you know if Oni implements buffer highlighting? After some quick tests, it seems calls like Semshi relies on that API for its highlighting, so without it the plugin wouldn't be of much use inside Oni anyway. Edit: Sorry, my bad. There seems to be another unrelated issue with Oni not recognizing Semshi's highlight groups, therefore not displaying the highlights. |
It could be the Oni is overriding some of the events Semshi is using somehow. There is a tonne of autocmds setup over here : https://github.com/onivim/oni/blob/master/vim/core/oni-core-interop/plugin/init.vim I'm going to try and remove them and see if that changes the behaviour of Semshi. |
Yep, I could make highlighting work after re-applying the highlight groups in my |
There is certainly a known issue where highlights from a users |
It looks like none of |
Sure! Would be good to have an issue tracking that. I'm intending to mess with the Python API so I'll hopefully clean up these bugs first. |
Just to let you know, I made a change so that Semshi ensures initialization irrespective of the event order (83f56bf) as this has also caused problems elsewhere (#11) - just in case you're still investigating and seeing changed behavior. Now, the only major difference when running inside Oni seems to be that it doesn't apply (or clears) Semshi's highlight group declarations which therefore need to be re-applied manually. |
As an addendum, the selected word higlight only works if you reload the colorscheme, or run |
As reported by @CrossR, running Semshi inside Oni raises an error when opening a Python file:
The cause seems to be that Semshi does some initialization on
BufEnter
, however Oni doesn't always trigger that event first, so Semshi may be uninitialized when a Python file is opened.The text was updated successfully, but these errors were encountered: