Skip to content
This repository was archived by the owner on Sep 10, 2019. It is now read-only.

Fix ModalFactory overlayClose=false option#717

Open
budiadiono wants to merge 1 commit intozurb:masterfrom
budiadiono:master
Open

Fix ModalFactory overlayClose=false option#717
budiadiono wants to merge 1 commit intozurb:masterfrom
budiadiono:master

Conversation

@budiadiono
Copy link

ModalFactory config -- should be able to read overlayClose that set to false

ModalFactory config -- should be able to read `overlayClose` that set to `false`
@budiadiono
Copy link
Author

I only changed one line, from this:
if(config[prop]) {
to be like this:
if(config[prop] !== undefined) {

but Travis giving a lot of scary error messages, was that by me?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant