Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added parameter for display #1

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Added parameter for display #1

wants to merge 2 commits into from

Conversation

jkorff
Copy link
Owner

@jkorff jkorff commented Nov 25, 2015

'displayType' is 'block' by default, emulating the usual CSS display property of the base version.

You can also set it to 'flex' if you want to restyle the popup.

When setting it to 'flex' you can use the following CSS (in addition to the existing CSS) to make the popup responsive:

#ouibounce-modal {
    justify-content: center;
    align-items: center;
    align-content: center;
}

Remove these styles:

#ouibounce-modal .modal {
    position: absolute;
    margin: auto;
    top: 0;
    right: 0;
    bottom: 0;
    left: 0;
}

'displayType' is 'block' by default, emulating the usual CSS display property of the base version.

You can also set it to 'flex' if you want to restyle the popup to be responsive.
Adds class 'ouibounce-display' when modal should be displayed. This allows for full CSS flexibility.

To avoid inline CSS, replace the jQuery "hide" methods with "removeClass('ouibounce-display')".
@jkorff
Copy link
Owner Author

jkorff commented Nov 26, 2015

This commit replaces CSS with classes which keeps the layers separate and allows for full flexibility.

@jkorff jkorff changed the title Added parameter 'displayType' Added parameter for display Nov 26, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant