No, that’s more or less how I would have done it (if that makes you feel any better? ). One thing though. Where you have:
+ I All (FvwmDock) CentreHorizontally
I suggest you change that to use ‘Next’:
+ I Next (FvwmDock) CentreHorizontally
Using All is fine, except that it is going to act on, well, all windows that bear the name ‘FvwmDock’. Not only is this pointless, but unnecessary since for other instances that already mapped, they’ll (presumably) already be in the correct position.
So, using Next ensures that the command only affects the next instance that gets created, i.e. the specific window.
– Thomas Adam