I'm not convinced this._buttonInner needs to be exposed as a member. it is only accessed from _configureActionButton afaict. So let's make it a global variable there. I don't think you need to worry about perfs to retrieve the element as this code is not critical and getElementById is already pretty fast.