Nit: it would be nice to change id-holder to locator-holder. But you'd have to make sure no CSS or anything else depends on the name id-holder.