Oops, I'm not up2date with the relevant code and didn't check the diff closely enough. Sorry for the confusion, long time ago the Ctrl-<1-3> keybindings were hard-coded but luckily, this has changed in the time between. So it's all fine, unbound by default for the new keybindings sounds good to me. Let the user choose what to bind (e.g. I'm already using Ctrl-<4-5> for other keybindings).
And I agree with Lex to split the %s/%f feature from this PR.
--- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/792#issuecomment-164781012