> The OP seemed to keep using the current document so they would not have 
> changed anything IIUC

Even though that code does not change the tab, the return value should still be 
`true`.  According to the (erroneous) documentation, the only time it should 
return `false` is "if the tab could not be found".  That code clearly 
demonstrates that the document id *could* be found (so the tab should exist).  
There is also a demo script `info/list-open-files.lua` that is supposed to 
change tabs using `geany.activate()`, but does not.

> maybe there should be two different local variables, `lua_idx` and 
> `notebook_page` to avoid confusion

I'll poke at `glspi_activate()` later to see if I can find where it's 
misbehaving.  If I manage to find the problem, I'll try to rewrite it as you 
suggest.


-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany-plugins/issues/1229#issuecomment-1454421454
You are receiving this because you are subscribed to this thread.

Message ID: <geany/geany-plugins/issues/1229/[email protected]>

Reply via email to