> I'm assuming the other tabs also use strings to check which tab is active so ok with this to maintain consistency.

Yup, exactly (e.g. [here](https://github.com/spree/spree/blob/60e617a/backend/app/views/spree/admin/images/index.html.erb#L3) and collectively [here](https://github.com/spree/spree/blob/60e617a/backend/app/views/spree/admin/shared/_product_tabs.html.erb#L12-L24)).  Cool, will leave as-is for consistency then.