Given that the AnnotationState is
type AnnotationState = 'unattached' | 'not-showing' | 'showing';
and when we call the private attach() method, we first check if the state is 'unattached'
private attach() {
if (this._state === 'unattached' && this._e.parentElement) {
// ...
}
}I think the this.attach() statement in the 'not-showing' case is redundant.
show(): void {
switch (this._state) {
case 'unattached':
// ...
case 'showing':
// ...
case 'not-showing':
this.attach();
if (this._svg) {
this.render(this._svg, false);
}
break;
}
}Not sure if I'm missing anything : )
Reactions are currently unavailable
Given that the AnnotationState is
and when we call the private attach() method, we first check if the state is 'unattached'
I think the this.attach() statement in the 'not-showing' case is redundant.
Not sure if I'm missing anything : )