Seems a bit overkill, but 👍🏻

I think we should always return an absolute path. Minor preference we move to pathlib internally but 🤷🏻 either way.

Switched to returning an absolute path. Like most people(?) I also prefer pathlib, but the return value immediately gets passed to add_toolitem, which is documented to take a str as input (and is technically something overridable by third-party backends) and I don't really want to bother changing that (especially as the filename will typically then get passed to a gui toolkit code to load the icon, and these typically don't actually support pathlib but plain str paths). So overall in this specific case I'm not convinced it's worth switching to returning a Path that'll immediately get converted back to a str (but I don't feel strongly either way).

I don't think it does? Both backends only implement the classic toolbar2, not mep22 toolmanager.

Thank you