Skip to content

Commit

Permalink
Remove uiToggle from UiCurtain
Browse files Browse the repository at this point in the history
This was adding an extra 'hide' class in some cases which made
tooltips disappear when they were supposed to be visible.
  • Loading branch information
bhousel committed Dec 19, 2024
1 parent cb5c1be commit 375ea72
Showing 1 changed file with 1 addition and 7 deletions.
8 changes: 1 addition & 7 deletions modules/ui/intro/UiCurtain.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,6 @@ import { easeLinear } from 'd3-ease';
import { selection, select } from 'd3-selection';
import * as Polyclip from 'polyclip-ts';

import { uiToggle } from '../toggle.js';


/**
* UiCurtain
Expand Down Expand Up @@ -433,10 +431,6 @@ export class UiCurtain {
}
}

// if (opts.duration !== 0 || !this.$tooltip.classed(placement)) {
// this.$tooltip.call(uiToggle(true));
// }

this.$tooltip
.style('left', `${tipX}px`)
.style('top', `${tipY}px`)
Expand All @@ -456,7 +450,7 @@ export class UiCurtain {
.style('top', `${shiftY}px`);

} else {
this.$tooltip.classed('in', false).call(uiToggle(false));
this.$tooltip.classed('in', false);
this._tooltipDirty = false;
}
}
Expand Down

0 comments on commit 375ea72

Please sign in to comment.