ayo/lion
1
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix(dialog): prevent opened-changed to be sent when it is not changing

Browse source
This commit is contained in:
Mathieu Puech 2020-12-17 15:23:42 -05:00
parent 125f606f89
commit 1e2f92b2f1
2 changed files with 4 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
packages/overlays/src/OverlayMixin.js
View file

@ -62,7 +62,7 @@ export const OverlayMixinImplementation = superclass =>
*/ */
requestUpdateInternal(name, oldValue) { requestUpdateInternal(name, oldValue) {
super.requestUpdateInternal(name, oldValue); super.requestUpdateInternal(name, oldValue);
if (name === 'opened') { if (name === 'opened' && this.opened !== oldValue) {
this.dispatchEvent(new Event('opened-changed')); this.dispatchEvent(new Event('opened-changed'));
} }
} }

3
packages/overlays/test-suites/OverlayMixin.suite.js
View file

@ -132,6 +132,9 @@ export function runOverlayMixinSuite({ tagString, tag, suffix = '' }) {
await el.updateComplete; await el.updateComplete;
expect(spy.callCount).to.equal(1); expect(spy.callCount).to.equal(1);
expect(el.opened).to.be.true; expect(el.opened).to.be.true;
el.opened = true;
await el.updateComplete;
expect(spy.callCount).to.equal(1);
await el._overlayCtrl.hide(); await el._overlayCtrl.hide();
await el.updateComplete; await el.updateComplete;
expect(spy.callCount).to.equal(2); expect(spy.callCount).to.equal(2);