Skip to content
This repository was archived by the owner on Oct 7, 2020. It is now read-only.

[#99] Fix hide event for smartbanners which option.layer is false (iO…#132

Open
grusenka wants to merge 1 commit intojasny:masterfrom
grusenka:bugfix-#99-close-button-fix-for-banner-without-layer
Open

[#99] Fix hide event for smartbanners which option.layer is false (iO…#132
grusenka wants to merge 1 commit intojasny:masterfrom
grusenka:bugfix-#99-close-button-fix-for-banner-without-layer

Conversation

@grusenka
Copy link

Default option for layer is false. And as commented in issue #99, problem is in this string:

      $('#smartbanner')
        .css('position', this.options.layer ? 'absolute' : 'static');

Solution for me is animate or margin-top.

@jdalton jdalton added the bug label Jun 23, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants