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

fix(datepicker): close with outside click (#386)

Browse source
This commit is contained in:
Erik Kroes 2019-11-27 14:19:59 +01:00 committed by Thomas Allmer
parent 3abb65da33
commit b34b399e17
2 changed files with 15 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
packages/input-datepicker/src/LionInputDatepicker.js
View file

@ -264,6 +264,7 @@ export class LionInputDatepicker extends OverlayMixin(LionInputDate) {
contentNode,
invokerNode,
elementToFocusAfterHide: invokerNode,
hidesOnOutsideClick: true,
});
return ctrl;
}

15
packages/input-datepicker/test/lion-input-datepicker.test.js
View file

@ -1,4 +1,4 @@
import { expect, fixture, defineCE } from '@open-wc/testing';
import { expect, fixture, defineCE, aTimeout } from '@open-wc/testing';
import sinon from 'sinon';
import { html, LitElement } from '@lion/core';
import { MaxDate, MinDate, MinMaxDate, IsDateDisabled } from '@lion/validate';
@ -99,6 +99,19 @@ describe('<lion-input-datepicker>', () => {
expect(elObj.overlayController.isShown).to.equal(false);
});
it('closes the calendar via outside click', async () => {
const el = await fixture(html`
<lion-input-datepicker></lion-input-datepicker>
`);
const elObj = new DatepickerInputObject(el);
await elObj.openCalendar();
expect(elObj.overlayController.isShown).to.equal(true);
document.body.click();
await aTimeout();
expect(elObj.overlayController.isShown).to.be.false;
});
/**
* Not in scope:
* - centralDate can be overridden