@tintou any tips for updating the vapi? Completely manually or using vapigen and taking the diff? Or do you have a script somewhere? I tried to do a vapigen for Mutter clutter but I got stuck at it thinking Atk.Implementor is a class and saying can't derive from two classes :/

I've been testing some additional changes on top of this branch and gala seems working fine with them. I'd like to contribute them back and I was wondering what would be the preferred approach: opening a PR against this branch, or against main with this branch as a base?

@garaevdi not sure what the rule of thumb here is, but personally would go with the latter option. Using this PR as base against the main branch.

@garaevdi a PR against main would be great, thanks!

@tintou any tips for updating the vapi? Completely manually or using vapigen and taking the diff? Or do you have a script somewhere? I tried to do a vapigen for Mutter clutter but I got stuck at it thinking Atk.Implementor is a class and saying can't derive from two classes :/

Sorry, I somehow missed your message, what I'm usually doing is adding vapi generation support first in the vapi/meson.build and using the .vapi generated from there and manually changing the files, it takes usually a few hours to do 😅

Closing since this was incorporated into a merged branch :)